Thread (9 messages) 9 messages, 4 authors, 2023-02-20

Re: [PATCH 2/2] powerpc/{32,book3e}: kcsan: Extend KCSAN Support

From: Michael Ellerman <mpe@ellerman.id.au>
Date: 2023-02-20 06:11:33

Christophe Leroy [off-list ref] writes:
Le 17/02/2023 à 00:23, Rohan McLure a écrit :
quoted
quoted
On 16 Feb 2023, at 6:14 pm, Christophe Leroy [off-list ref] wrote:
Le 16/02/2023 à 06:09, Rohan McLure a écrit :
quoted
Enable HAVE_ARCH_KCSAN on all powerpc platforms, permitting use of the
kernel concurrency sanitiser through the CONFIG_KCSAN_* kconfig options.

Boots and passes selftests on 32-bit and 64-bit platforms. See
documentation in Documentation/dev-tools/kcsan.rst for more information.

Signed-off-by: Rohan McLure <redacted>
---
New patch
---
  arch/powerpc/Kconfig | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
index 2c9cdf1d8761..45771448d47a 100644
--- a/arch/powerpc/Kconfig
+++ b/arch/powerpc/Kconfig
@@ -197,7 +197,7 @@ config PPC
   select HAVE_ARCH_KASAN if PPC_RADIX_MMU
   select HAVE_ARCH_KASAN if PPC_BOOK3E_64
   select HAVE_ARCH_KASAN_VMALLOC if HAVE_ARCH_KASAN
- select HAVE_ARCH_KCSAN             if PPC_BOOK3S_64
+ select HAVE_ARCH_KCSAN
So that's a followup of a not yet posted version v5 of the other series ?
Why not just add patch 1 in that series and have KCSAN for all powerpc
at once ?
So the v3 was accepted upstream, likely to appear in 6.3. This patch series is
just to extend support to other platforms, once kcsan supports us.
Hum ... Ok.

I checked in checkpatch before writting that mail and saw that v4 was 
flagged "changes requested", so I didn't notice v3 was accepted.
Sorry that's my fault. I talked to Rohan on chat and decided that taking
v3 was the best way to get something in before the merge window closed.

cheers
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help