Skip to content

Commit 6676fd3

Browse files
committed
kstack_erase: Add -mgeneral-regs-only to silence Clang warnings
Once CONFIG_KSTACK_ERASE is enabled with Clang on i386, the build warns: kernel/kstack_erase.c:168:2: warning: function with attribute 'no_caller_saved_registers' should only call a function with attribute 'no_caller_saved_registers' or be compiled with '-mgeneral-regs-only' [-Wexcessive-regsave] Add -mgeneral-regs-only for the kstack_erase handler, to make Clang feel better (it is effectively a no-op flag for the kernel). No binary changes encountered. Build & boot tested with Clang 21 on x86_64, and i386. Build tested with GCC 14.2.0 on x86_64, i386, arm64, and arm. Reported-by: Nathan Chancellor <[email protected]> Closes: https://lore.kernel.org/all/20250726004313.GA3650901@ax162 Tested-by: Nathan Chancellor <[email protected]> Signed-off-by: Kees Cook <[email protected]>
1 parent 381a38e commit 6676fd3

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

kernel/Makefile

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -140,6 +140,7 @@ obj-$(CONFIG_RESOURCE_KUNIT_TEST) += resource_kunit.o
140140
obj-$(CONFIG_SYSCTL_KUNIT_TEST) += sysctl-test.o
141141

142142
CFLAGS_kstack_erase.o += $(DISABLE_KSTACK_ERASE)
143+
CFLAGS_kstack_erase.o += $(call cc-option,-mgeneral-regs-only)
143144
obj-$(CONFIG_KSTACK_ERASE) += kstack_erase.o
144145
KASAN_SANITIZE_kstack_erase.o := n
145146
KCSAN_SANITIZE_kstack_erase.o := n

0 commit comments

Comments
 (0)