diff options
author | Frederic Weisbecker <fweisbec@gmail.com> | 2009-04-10 21:17:17 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-04-12 10:10:51 -0400 |
commit | 9eeba6138cefc0435695463ddadb0d95e0a6bcd2 (patch) | |
tree | daba646e5ddd27132e9679aa3eabec389bf4722e /include | |
parent | 066123a535927b3f17cac2305258cc71abdb0d92 (diff) |
lockdep: warn about lockdep disabling after kernel taint
Impact: provide useful missing info for developers
Kernel taint can occur in several situations such as warnings,
load of prorietary or staging modules, bad page, etc...
But when such taint happens, a developer might still be working on
the kernel, expecting that lockdep is still enabled. But a taint
disables lockdep without ever warning about it.
Such a kernel behaviour doesn't really help for kernel development.
This patch adds this missing warning.
Since the taint is done most of the time after the main message that
explain the real source issue, it seems safe to warn about it inside
add_taint() so that it appears at last, without hurting the main
information.
v2: Use a generic helper to disable lockdep instead of an
open coded xchg().
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Peter Zijlstra <peterz@infradead.org>
LKML-Reference: <1239412638-6739-1-git-send-email-fweisbec@gmail.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/debug_locks.h | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/include/linux/debug_locks.h b/include/linux/debug_locks.h index 096476f1fb35..493dedb7a67b 100644 --- a/include/linux/debug_locks.h +++ b/include/linux/debug_locks.h | |||
@@ -2,12 +2,19 @@ | |||
2 | #define __LINUX_DEBUG_LOCKING_H | 2 | #define __LINUX_DEBUG_LOCKING_H |
3 | 3 | ||
4 | #include <linux/kernel.h> | 4 | #include <linux/kernel.h> |
5 | #include <asm/atomic.h> | ||
5 | 6 | ||
6 | struct task_struct; | 7 | struct task_struct; |
7 | 8 | ||
8 | extern int debug_locks; | 9 | extern int debug_locks; |
9 | extern int debug_locks_silent; | 10 | extern int debug_locks_silent; |
10 | 11 | ||
12 | |||
13 | static inline int __debug_locks_off(void) | ||
14 | { | ||
15 | return xchg(&debug_locks, 0); | ||
16 | } | ||
17 | |||
11 | /* | 18 | /* |
12 | * Generic 'turn off all lock debugging' function: | 19 | * Generic 'turn off all lock debugging' function: |
13 | */ | 20 | */ |