diff options
| author |   Peter Zijlstra <a.p.zijlstra@chello.nl> | 2011-04-05 17:23:44 +0200 |
|---|---|---|
| committer |   Ingo Molnar <mingo@elte.hu> | 2011-04-14 08:52:35 +0200 |
| commit | fd2f4419b4cbe8fe90796df9617c355762afd6a4 (patch) | |
| tree | 9eefa7f46c7163f00adb416ff4b9db97653d2665 /kernel/sched_debug.c | |
| parent | sched: Clean up ttwu() stats (diff) | |
| download | linux-dev-fd2f4419b4cbe8fe90796df9617c355762afd6a4.tar.xz linux-dev-fd2f4419b4cbe8fe90796df9617c355762afd6a4.zip | |
sched: Provide p->on_rq
Provide a generic p->on_rq because the p->se.on_rq semantics are
unfavourable for lockless wakeups but needed for sched_fair.
In particular, p->on_rq is only cleared when we actually dequeue the
task in schedule() and not on any random dequeue as done by things
like __migrate_task() and __sched_setscheduler().
This also allows us to remove p->se usage from !sched_fair code.
Reviewed-by: Frank Rowand <frank.rowand@am.sony.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Nick Piggin <npiggin@kernel.dk>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/20110405152728.949545047@chello.nl
Diffstat (limited to 'kernel/sched_debug.c')
| -rw-r--r-- | kernel/sched_debug.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/sched_debug.c b/kernel/sched_debug.c index 7bacd83a4158..3669bec6e130 100644 --- a/kernel/sched_debug.c +++ b/kernel/sched_debug.c @@ -152,7 +152,7 @@ static void print_rq(struct seq_file *m, struct rq *rq, int rq_cpu) read_lock_irqsave(&tasklist_lock, flags); do_each_thread(g, p) { - if (!p->se.on_rq || task_cpu(p) != rq_cpu) + if (!p->on_rq || task_cpu(p) != rq_cpu) continue; print_task(m, rq, p); |