diff options
author | Vaidyanathan Srinivasan <svaidy@linux.vnet.ibm.com> | 2008-12-08 10:22:49 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-12-08 10:48:00 -0500 |
commit | efbe027e95dc13ac343b6130948418d7ead7ddf1 (patch) | |
tree | 96666b3d26027656183a64e96ffa3babcf8bd184 /kernel/sched.c | |
parent | 5436499e6098759c2340f8b906ea52f993dc4efb (diff) |
sched: idle_balance() does not call load_balance_newidle()
Impact: fix SD_BALANCE_NEWIDLEand broaden its use
load_balance_newidle() does not get called if SD_BALANCE_NEWIDLE is
set at higher level domain (3-CPU) and not in low level domain (2-MC).
pulled_task is initialised to -1 and checked for non-zero which is
always true if the lowest level sched_domain does not have
SD_BALANCE_NEWIDLE flag set.
Signed-off-by: Vaidyanathan Srinivasan <svaidy@linux.vnet.ibm.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/sched.c')
-rw-r--r-- | kernel/sched.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/sched.c b/kernel/sched.c index 74498c840f93..bb9c6384d077 100644 --- a/kernel/sched.c +++ b/kernel/sched.c | |||
@@ -3685,7 +3685,7 @@ out_balanced: | |||
3685 | static void idle_balance(int this_cpu, struct rq *this_rq) | 3685 | static void idle_balance(int this_cpu, struct rq *this_rq) |
3686 | { | 3686 | { |
3687 | struct sched_domain *sd; | 3687 | struct sched_domain *sd; |
3688 | int pulled_task = -1; | 3688 | int pulled_task = 0; |
3689 | unsigned long next_balance = jiffies + HZ; | 3689 | unsigned long next_balance = jiffies + HZ; |
3690 | cpumask_t tmpmask; | 3690 | cpumask_t tmpmask; |
3691 | 3691 | ||