diff options
author | Tim Blechmann <tim@klingt.org> | 2009-11-24 05:55:45 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-11-24 06:18:12 -0500 |
commit | 36ace27e3e60d44ea69ce394b2e45386ae98d9d9 (patch) | |
tree | 059cf2b286d14b90e6a2515e7014eff82b5c73c6 /kernel | |
parent | 429947248f814e90f416ab4f68a871ab628000c3 (diff) |
sched: Optimize branch hint in pick_next_task_fair()
Branch hint profiling on my nehalem machine showed 90%
incorrect branch hints:
15728471 158903754 90 pick_next_task_fair
sched_fair.c 1555
Signed-off-by: Tim Blechmann <tim@klingt.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
LKML-Reference: <4B0BBBB1.2050100@klingt.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/sched_fair.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/sched_fair.c b/kernel/sched_fair.c index f28a2671a1a6..24086e7e7593 100644 --- a/kernel/sched_fair.c +++ b/kernel/sched_fair.c | |||
@@ -1704,7 +1704,7 @@ static struct task_struct *pick_next_task_fair(struct rq *rq) | |||
1704 | struct cfs_rq *cfs_rq = &rq->cfs; | 1704 | struct cfs_rq *cfs_rq = &rq->cfs; |
1705 | struct sched_entity *se; | 1705 | struct sched_entity *se; |
1706 | 1706 | ||
1707 | if (unlikely(!cfs_rq->nr_running)) | 1707 | if (!cfs_rq->nr_running) |
1708 | return NULL; | 1708 | return NULL; |
1709 | 1709 | ||
1710 | do { | 1710 | do { |