public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] sched.c: call debug_show_all_locks when dumping all tasks
@ 2009-11-25 13:23 Shmulik Ladkani
  2009-11-25 13:30 ` [tip:sched/core] sched.c: Call debug_show_all_locks() " tip-bot for Shmulik Ladkani
  0 siblings, 1 reply; 2+ messages in thread
From: Shmulik Ladkani @ 2009-11-25 13:23 UTC (permalink / raw)
  To: mingo, peterz, linux-kernel

show_state_filter() failed to call debug_show_all_locks() when specified to
dump all tasks.
Fix: call debug_show_all_locks() when !state_filter (since !state_filter
means "all tasks").

Signed-off-by: Shmulik Ladkani <shmulik.ladkani@gmail.com>
---
In commit 39bc89fd4019b164002adaacef92c4140e37955a the interface of
show_state_filter() was changed: zero valued 'state_filter' specifies "dump all
tasks" (instead of -1).
However, the condition for calling debug_show_all_locks() ("show locks if all
tasks are dumped") was not updated accordingly.

diff --git a/kernel/sched.c b/kernel/sched.c
index 3c11ae0..6e2b936 100644
--- a/kernel/sched.c
+++ b/kernel/sched.c
@@ -6935,7 +6935,7 @@ void show_state_filter(unsigned long state_filter)
 	/*
 	 * Only show locks if all tasks are dumped:
 	 */
-	if (state_filter == -1)
+	if (!state_filter)
 		debug_show_all_locks();
 }

-- 
Shmulik Ladkani

^ permalink raw reply related	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2009-11-25 13:31 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-11-25 13:23 [PATCH] sched.c: call debug_show_all_locks when dumping all tasks Shmulik Ladkani
2009-11-25 13:30 ` [tip:sched/core] sched.c: Call debug_show_all_locks() " tip-bot for Shmulik Ladkani

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox