tracing: branch tracer, fix vdso crash
authorIngo Molnar <mingo@elte.hu>
Wed, 12 Nov 2008 12:17:38 +0000 (13:17 +0100)
committerIngo Molnar <mingo@elte.hu>
Wed, 12 Nov 2008 12:26:38 +0000 (13:26 +0100)
Impact: fix bootup crash

the branch tracer missed arch/x86/vdso/vclock_gettime.c from
disabling tracing, which caused such bootup crashes:

  [  201.840097] init[1]: segfault at 7fffed3fe7c0 ip 00007fffed3fea2e sp 000077

also clean up the ugly ifdefs in arch/x86/kernel/vsyscall_64.c by
creating DISABLE_UNLIKELY_PROFILE facility for code to turn off
instrumentation on a per file basis.

Signed-off-by: Ingo Molnar <mingo@elte.hu>
arch/x86/kernel/vsyscall_64.c
arch/x86/vdso/vclock_gettime.c
include/linux/compiler.h

index 2f90202e59b33b4a014ac33297a5a5faeedcd14a..ece02932ea5748875b864099a6c87a90ec0dec8c 100644 (file)
  *  want per guest time just set the kernel.vsyscall64 sysctl to 0.
  */
 
-/* Protect userspace from profiling */
-#ifdef CONFIG_TRACE_UNLIKELY_PROFILE
-# undef likely
-# undef unlikely
-# define likely(x)             likely_notrace(x)
-# define unlikely(x)           unlikely_notrace(x)
-#endif
+/* Disable profiling for userspace code: */
+#define DISABLE_UNLIKELY_PROFILE
 
 #include <linux/time.h>
 #include <linux/init.h>
index 1ef0f90813d626ed6be436b93d3d5b6550dbb392..6e667631e7dc23f2b9c4b74daf6889339ac6dbb3 100644 (file)
@@ -9,6 +9,9 @@
  * Also alternative() doesn't work.
  */
 
+/* Disable profiling for userspace code: */
+#define DISABLE_UNLIKELY_PROFILE
+
 #include <linux/kernel.h>
 #include <linux/posix-timers.h>
 #include <linux/time.h>
index 935e30cfaf3c1635c67ae2f197a1ac4445038406..63b7d9089d6e1642a10481d319a960172c9c531e 100644 (file)
@@ -59,7 +59,11 @@ extern void __chk_io_ptr(const volatile void __iomem *);
  * specific implementations come from the above header files
  */
 
-#ifdef CONFIG_TRACE_UNLIKELY_PROFILE
+/*
+ * Note: DISABLE_UNLIKELY_PROFILE can be used by special lowlevel code
+ * to disable branch tracing on a per file basis.
+ */
+#if defined(CONFIG_TRACE_UNLIKELY_PROFILE) && !defined(DISABLE_UNLIKELY_PROFILE)
 struct ftrace_likely_data {
        const char *func;
        const char *file;