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_KCSANSo 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