lockdep: warn about lockdep disabling after kernel taint
authorFrederic Weisbecker <fweisbec@gmail.com>
Sat, 11 Apr 2009 01:17:17 +0000 (03:17 +0200)
committerIngo Molnar <mingo@elte.hu>
Sun, 12 Apr 2009 14:10:51 +0000 (16:10 +0200)
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>
include/linux/debug_locks.h
kernel/panic.c
lib/debug_locks.c

index 096476f1fb356a2c17ab5d267be9a2295f55295a..493dedb7a67ba17a67c47540bc1f188fc097c44e 100644 (file)
@@ -2,12 +2,19 @@
 #define __LINUX_DEBUG_LOCKING_H
 
 #include <linux/kernel.h>
+#include <asm/atomic.h>
 
 struct task_struct;
 
 extern int debug_locks;
 extern int debug_locks_silent;
 
+
+static inline int __debug_locks_off(void)
+{
+       return xchg(&debug_locks, 0);
+}
+
 /*
  * Generic 'turn off all lock debugging' function:
  */
index 3fd8c5bf8b394dd94fee9ce22fa8c20135c76c4f..940ca14f6dbfa211ad0f2768248f371a2f5e36d7 100644 (file)
@@ -213,8 +213,14 @@ unsigned long get_taint(void)
 
 void add_taint(unsigned flag)
 {
-       /* can't trust the integrity of the kernel anymore: */
-       debug_locks = 0;
+       /*
+        * Can't trust the integrity of the kernel anymore.
+        * We don't call directly debug_locks_off() because the issue
+        * is not necessarily serious enough to set oops_in_progress to 1
+        */
+       if (__debug_locks_off())
+               printk(KERN_WARNING "Disabling lockdep due to kernel taint\n");
+
        set_bit(flag, &tainted_mask);
 }
 EXPORT_SYMBOL(add_taint);
index 0218b4693dd8c250f83c741bf6ffbfbbca31f94b..bc3b11731b9ce4160ff770058f9ebc440c441351 100644 (file)
@@ -36,7 +36,7 @@ int debug_locks_silent;
  */
 int debug_locks_off(void)
 {
-       if (xchg(&debug_locks, 0)) {
+       if (__debug_locks_off()) {
                if (!debug_locks_silent) {
                        oops_in_progress = 1;
                        console_verbose();