Re: [PATCH 2/2] powerpc/{32,book3e}: kcsan: Extend KCSAN Support
Christophe Leroy writes: > Le 17/02/2023 à 00:23, Rohan McLure a écrit : >>> On 16 Feb 2023, at 6:14 pm, Christophe Leroy >>> wrote: >>> Le 16/02/2023 à 06:09, Rohan McLure a écrit : 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 --- 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
Re: [PATCH 2/2] powerpc/{32,book3e}: kcsan: Extend KCSAN Support
Le 17/02/2023 à 00:23, Rohan McLure a écrit : >> On 16 Feb 2023, at 6:14 pm, Christophe Leroy >> wrote: >> >> >> >> Le 16/02/2023 à 06:09, Rohan McLure a écrit : >>> 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 >>> --- >>> 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. > > Link: > https://patchwork.ozlabs.org/project/linuxppc-dev/cover/20230206021801.105268-1-rmcl...@linux.ibm.com/ > >> >>>select HAVE_ARCH_KFENCE if ARCH_SUPPORTS_DEBUG_PAGEALLOC >>>select HAVE_ARCH_RANDOMIZE_KSTACK_OFFSET >>>select HAVE_ARCH_KGDB > >
Re: [PATCH 2/2] powerpc/{32,book3e}: kcsan: Extend KCSAN Support
> On 16 Feb 2023, at 6:14 pm, Christophe Leroy > wrote: > > > > Le 16/02/2023 à 06:09, Rohan McLure a écrit : >> 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 >> --- >> 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. Link: https://patchwork.ozlabs.org/project/linuxppc-dev/cover/20230206021801.105268-1-rmcl...@linux.ibm.com/ > >> select HAVE_ARCH_KFENCE if ARCH_SUPPORTS_DEBUG_PAGEALLOC >> select HAVE_ARCH_RANDOMIZE_KSTACK_OFFSET >> select HAVE_ARCH_KGDB
Re: [PATCH 2/2] powerpc/{32,book3e}: kcsan: Extend KCSAN Support
Le 16/02/2023 à 06:09, Rohan McLure a écrit : > 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 > --- > 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 ? > select HAVE_ARCH_KFENCE if ARCH_SUPPORTS_DEBUG_PAGEALLOC > select HAVE_ARCH_RANDOMIZE_KSTACK_OFFSET > select HAVE_ARCH_KGDB