diff options
author | Nikhil Rao <ncrao@google.com> | 2010-11-17 14:42:04 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2010-11-18 07:12:33 -0500 |
commit | d5ad140bc1505a98c0f040937125bfcbb508078f (patch) | |
tree | adc1581eb61159675f9163515fd2402f81b586e2 /kernel | |
parent | b5482cfa1c95a188b3054fa33274806add91bbe5 (diff) |
sched: Fix idle balancing
An earlier commit reverts idle balancing throttling reset to fix a 30%
regression in volanomark throughput. We still need to reset idle_stamp
when we pull a task in newidle balance.
Reported-by: Alex Shi <alex.shi@intel.com>
Signed-off-by: Nikhil Rao <ncrao@google.com>
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
LKML-Reference: <1290022924-3548-1-git-send-email-ncrao@google.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/sched_fair.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/kernel/sched_fair.c b/kernel/sched_fair.c index ba0556dc7c06..00ebd7686676 100644 --- a/kernel/sched_fair.c +++ b/kernel/sched_fair.c | |||
@@ -3215,8 +3215,10 @@ static void idle_balance(int this_cpu, struct rq *this_rq) | |||
3215 | interval = msecs_to_jiffies(sd->balance_interval); | 3215 | interval = msecs_to_jiffies(sd->balance_interval); |
3216 | if (time_after(next_balance, sd->last_balance + interval)) | 3216 | if (time_after(next_balance, sd->last_balance + interval)) |
3217 | next_balance = sd->last_balance + interval; | 3217 | next_balance = sd->last_balance + interval; |
3218 | if (pulled_task) | 3218 | if (pulled_task) { |
3219 | this_rq->idle_stamp = 0; | ||
3219 | break; | 3220 | break; |
3221 | } | ||
3220 | } | 3222 | } |
3221 | 3223 | ||
3222 | raw_spin_lock(&this_rq->lock); | 3224 | raw_spin_lock(&this_rq->lock); |