Hi Uros, On Sun, Dec 08, 2024 at 09:45:15PM +0100, Uros Bizjak wrote: > Enable strict percpu address space checks via x86 named address space > qualifiers. Percpu variables are declared in __seg_gs/__seg_fs named > AS and kept named AS qualified until they are dereferenced via percpu > accessor. This approach enables various compiler checks for > cross-namespace variable assignments. > > Please note that current version of sparse doesn't know anything about > __typeof_unqual__() operator. Avoid the usage of __typeof_unqual__() > when sparse checking is active to prevent sparse errors with unknowing > keyword. The proposed patch by Dan Carpenter to implement > __typeof_unqual__() handling in sparse is located at: > > https://lore.kernel.org/lkml/[email protected]/ > > v2: - Add comment to remove test for __CHECKER__ once sparse learns > about __typeof_unqual__. > - Add Acked-by: tags. > v3: - Rename __per_cpu_qual to __percpu_qual. > - Add more Acked-by: tags. > > Cc: Thomas Gleixner <[email protected]> > Cc: Dennis Zhou <[email protected]> > Cc: Tejun Heo <[email protected]> > Cc: Christoph Lameter <[email protected]> > Cc: Linus Torvalds <[email protected]> > Cc: Andy Lutomirski <[email protected]> > Cc: Ingo Molnar <[email protected]> > Cc: Nadav Amit <[email protected]> > Cc: Brian Gerst <[email protected]> > Cc: Dan Carpenter <[email protected]> > Cc: H. Peter Anvin <[email protected]> > Cc: Peter Zijlstra <[email protected]> > > Uros Bizjak (6): > x86/kgdb: Use IS_ERR_PCPU() macro > compiler.h: Introduce TYPEOF_UNQUAL() macro > percpu: Use TYPEOF_UNQUAL() in variable declarations > percpu: Use TYPEOF_UNQUAL() in *_cpu_ptr() accessors > percpu: Repurpose __percpu tag as a named address space qualifier > percpu/x86: Enable strict percpu checks via named AS qualifiers > > arch/x86/include/asm/percpu.h | 38 ++++++++++++++++++++++++--------- > arch/x86/kernel/kgdb.c | 2 +- > fs/bcachefs/util.h | 2 +- > include/asm-generic/percpu.h | 39 ++++++++++++++++++++++------------ > include/linux/compiler.h | 13 ++++++++++++ > include/linux/compiler_types.h | 2 +- > include/linux/part_stat.h | 2 +- > include/linux/percpu-defs.h | 6 +++--- > include/net/snmp.h | 5 ++--- > init/Kconfig | 3 +++ > kernel/locking/percpu-rwsem.c | 2 +- > net/mpls/internal.h | 4 ++-- > 12 files changed, 82 insertions(+), 36 deletions(-) > > -- > 2.42.0 >
This all looks good to me. Acked-by: Dennis Zhou <[email protected]> Thanks, Dennis
