diff options
author | Lin Ming <ming.m.lin@intel.com> | 2009-01-15 17:17:15 +0100 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-01-15 21:07:57 +0100 |
commit | 6272d68cc6a5f90c6b1a2228cf0f67b895305d17 (patch) | |
tree | 9f7e6635d4ffb9d713d3b2be83a4cd4bd4749080 /kernel/sched_fair.c | |
parent | e17036dac189dd034c092a91df56aa740db7146d (diff) | |
download | linux-6272d68cc6a5f90c6b1a2228cf0f67b895305d17.tar.gz linux-6272d68cc6a5f90c6b1a2228cf0f67b895305d17.tar.bz2 linux-6272d68cc6a5f90c6b1a2228cf0f67b895305d17.zip |
sched: sched_slice() fixlet
Mike's change: 0a582440f "sched: fix sched_slice())" broke group
scheduling by forgetting to reload cfs_rq on each loop.
This patch fixes aim7 regression and specjbb2005 regression becomes
less than 1.5% on 8-core stokley.
Signed-off-by: Lin Ming <ming.m.lin@intel.com>
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Tested-by: Jayson King <dev@jaysonking.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/sched_fair.c')
-rw-r--r-- | kernel/sched_fair.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/kernel/sched_fair.c b/kernel/sched_fair.c index 16b419bb8b0a..5cc1c162044f 100644 --- a/kernel/sched_fair.c +++ b/kernel/sched_fair.c @@ -429,7 +429,10 @@ static u64 sched_slice(struct cfs_rq *cfs_rq, struct sched_entity *se) u64 slice = __sched_period(cfs_rq->nr_running + !se->on_rq); for_each_sched_entity(se) { - struct load_weight *load = &cfs_rq->load; + struct load_weight *load; + + cfs_rq = cfs_rq_of(se); + load = &cfs_rq->load; if (unlikely(!se->on_rq)) { struct load_weight lw = cfs_rq->load; |