Commit 381a38ea authored by Kees Cook's avatar Kees Cook
Browse files

init.h: Disable sanitizer coverage for __init and __head 

While __noinstr already contained __no_sanitize_coverage, it needs to
be added to __init and __head section markings to support the Clang
implementation of CONFIG_KSTACK_ERASE. This is to make sure the stack
depth tracking callback is not executed in unsupported contexts.

The other sanitizer coverage options (trace-pc and trace-cmp) aren't
needed in __head nor __init either ("We are interested in code coverage
as a function of a syscall inputs"[1]), so this is fine to disable for
them as well.

Link: https://web.git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/kernel/kcov.c?h=v6.14#n179

 [1]
Acked-by: default avatarMarco Elver <elver@google.com>
Link: https://lore.kernel.org/r/20250724055029.3623499-3-kees@kernel.org


Signed-off-by: default avatarKees Cook <kees@kernel.org>
parent 431a380f

arch/x86/include/asm/init.h

+1 −1
Original line number Diff line number Diff line
@@ -5,7 +5,7 @@ 
#if defined(CONFIG_CC_IS_CLANG) && CONFIG_CLANG_VERSION < 170000

#define __head	__section(".head.text") __no_sanitize_undefined __no_stack_protector

#else

#define __head	__section(".head.text") __no_sanitize_undefined

#define __head	__section(".head.text") __no_sanitize_undefined __no_sanitize_coverage

#endif



struct x86_mapping_info {

include/linux/init.h

+3 −1
Original line number Diff line number Diff line
@@ -49,7 +49,9 @@ 


/* These are for everybody (although not all archs will actually

   discard it in modules) */

#define __init		__section(".init.text") __cold  __latent_entropy __noinitretpoline

#define __init		__section(".init.text") __cold __latent_entropy	\

						__noinitretpoline	\

						__no_sanitize_coverage

#define __initdata	__section(".init.data")

#define __initconst	__section(".init.rodata")

#define __exitdata	__section(".exit.data")