All of lore.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.