Commit ebb6d35a authored by Arnd Bergmann's avatar Arnd Bergmann Committed by Linus Torvalds

kasan: remove clang version check for KASAN_STACK

asan-stack mode still uses dangerously large kernel stacks of tens of
kilobytes in some drivers, and it does not seem that anyone is working
on the clang bug.

Turn it off for all clang versions to prevent users from accidentally
enabling it once they update to clang-9, and to help automated build
testing with clang-9.

Link: https://bugs.llvm.org/show_bug.cgi?id=38809
Link: http://lkml.kernel.org/r/20190719200347.2596375-1-arnd@arndb.de
Fixes: 6baec880 ("kasan: turn off asan-stack for clang-8 and earlier")
Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
Acked-by: default avatarNick Desaulniers <ndesaulniers@google.com>
Reviewed-by: default avatarMark Brown <broonie@kernel.org>
Reviewed-by: default avatarAndrey Ryabinin <aryabinin@virtuozzo.com>
Cc: Qian Cai <cai@lca.pw>
Cc: Andrey Konovalov <andreyknvl@google.com>
Cc: Vasily Gorbik <gor@linux.ibm.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 670105a2
...@@ -106,7 +106,6 @@ endchoice ...@@ -106,7 +106,6 @@ endchoice
config KASAN_STACK_ENABLE config KASAN_STACK_ENABLE
bool "Enable stack instrumentation (unsafe)" if CC_IS_CLANG && !COMPILE_TEST bool "Enable stack instrumentation (unsafe)" if CC_IS_CLANG && !COMPILE_TEST
default !(CLANG_VERSION < 90000)
depends on KASAN depends on KASAN
help help
The LLVM stack address sanitizer has a know problem that The LLVM stack address sanitizer has a know problem that
...@@ -115,11 +114,11 @@ config KASAN_STACK_ENABLE ...@@ -115,11 +114,11 @@ config KASAN_STACK_ENABLE
Disabling asan-stack makes it safe to run kernels build Disabling asan-stack makes it safe to run kernels build
with clang-8 with KASAN enabled, though it loses some of with clang-8 with KASAN enabled, though it loses some of
the functionality. the functionality.
This feature is always disabled when compile-testing with clang-8 This feature is always disabled when compile-testing with clang
or earlier to avoid cluttering the output in stack overflow to avoid cluttering the output in stack overflow warnings,
warnings, but clang-8 users can still enable it for builds without but clang users can still enable it for builds without
CONFIG_COMPILE_TEST. On gcc and later clang versions it is CONFIG_COMPILE_TEST. On gcc it is assumed to always be safe
assumed to always be safe to use and enabled by default. to use and enabled by default.
config KASAN_STACK config KASAN_STACK
int int
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment