summaryrefslogtreecommitdiffstats
path: root/kernel/lockdep.c
diff options
context:
space:
mode:
authorIngo Molnar <mingo@elte.hu>2007-12-05 15:46:09 +0100
committerIngo Molnar <mingo@elte.hu>2007-12-05 15:46:09 +0100
commit856848737bd944c1db3ce0a66bbf67e56bd6f77d (patch)
tree4fce302f29225f7d1d558559e40e73dadc40e60e /kernel/lockdep.c
parent41a2d6cfa3f77ec469e7e5f06b4d7ffd031f9c0e (diff)
downloadop-kernel-dev-856848737bd944c1db3ce0a66bbf67e56bd6f77d.zip
op-kernel-dev-856848737bd944c1db3ce0a66bbf67e56bd6f77d.tar.gz
lockdep: fix debug_show_all_locks()
fix the oops that can be seen in: http://bugzilla.kernel.org/attachment.cgi?id=13828&action=view it is not safe to print the locks of running tasks. (even with this fix we have a small race - but this is a debug function after all.) Signed-off-by: Ingo Molnar <mingo@elte.hu> Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Diffstat (limited to 'kernel/lockdep.c')
-rw-r--r--kernel/lockdep.c7
1 files changed, 7 insertions, 0 deletions
diff --git a/kernel/lockdep.c b/kernel/lockdep.c
index ed38bbf..7e2ca7c 100644
--- a/kernel/lockdep.c
+++ b/kernel/lockdep.c
@@ -3173,6 +3173,13 @@ retry:
printk(" locked it.\n");
do_each_thread(g, p) {
+ /*
+ * It's not reliable to print a task's held locks
+ * if it's not sleeping (or if it's not the current
+ * task):
+ */
+ if (p->state == TASK_RUNNING && p != current)
+ continue;
if (p->lockdep_depth)
lockdep_print_held_locks(p);
if (!unlock)
OpenPOWER on IntegriCloud