summaryrefslogtreecommitdiff
path: root/scripts/Makefile.kcsan
diff options
context:
space:
mode:
authorMarco Elver <elver@google.com>2020-05-21 16:20:38 +0200
committerThomas Gleixner <tglx@linutronix.de>2020-06-11 20:04:00 +0200
commit17168f5c1bef4ec7c239e2dd5e3e60b862727dd4 (patch)
tree6f4b678199cac97c93a55922dec568b60b7798bc /scripts/Makefile.kcsan
parentea91a1d45d19469001a4955583187b0d75915759 (diff)
downloadlwn-17168f5c1bef4ec7c239e2dd5e3e60b862727dd4.tar.gz
lwn-17168f5c1bef4ec7c239e2dd5e3e60b862727dd4.zip
kcsan: Avoid inserting __tsan_func_entry/exit if possible
To avoid inserting __tsan_func_{entry,exit}, add option if supported by compiler. Currently only Clang can be told to not emit calls to these functions. It is safe to not emit these, since KCSAN does not rely on them. Note that, if we disable __tsan_func_{entry,exit}(), we need to disable tail-call optimization in sanitized compilation units, as otherwise we may skip frames in the stack trace; in particular when the tail called function is one of the KCSAN's runtime functions, and a report is generated, we might miss the function where the actual access occurred. Since __tsan_func_{entry,exit}() insertion effectively disabled tail-call optimization, there should be no observable change. This was caught and confirmed with kcsan-test & UNWINDER_ORC. Signed-off-by: Marco Elver <elver@google.com> Signed-off-by: Borislav Petkov <bp@suse.de> Signed-off-by: Thomas Gleixner <tglx@linutronix.de> Acked-by: Will Deacon <will@kernel.org> Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org> Link: https://lkml.kernel.org/r/20200521142047.169334-3-elver@google.com
Diffstat (limited to 'scripts/Makefile.kcsan')
-rw-r--r--scripts/Makefile.kcsan11
1 files changed, 10 insertions, 1 deletions
diff --git a/scripts/Makefile.kcsan b/scripts/Makefile.kcsan
index caf1111a28ae..20337a7ecf54 100644
--- a/scripts/Makefile.kcsan
+++ b/scripts/Makefile.kcsan
@@ -1,6 +1,15 @@
# SPDX-License-Identifier: GPL-2.0
ifdef CONFIG_KCSAN
-CFLAGS_KCSAN := -fsanitize=thread
+# GCC and Clang accept backend options differently. Do not wrap in cc-option,
+# because Clang accepts "--param" even if it is unused.
+ifdef CONFIG_CC_IS_CLANG
+cc-param = -mllvm -$(1)
+else
+cc-param = --param -$(1)
+endif
+
+CFLAGS_KCSAN := -fsanitize=thread \
+ $(call cc-option,$(call cc-param,tsan-instrument-func-entry-exit=0) -fno-optimize-sibling-calls)
endif # CONFIG_KCSAN