diff options
author | Vaidyanathan Srinivasan <svaidy@linux.vnet.ibm.com> | 2008-12-08 20:52:49 +0530 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-12-08 16:48:00 +0100 |
commit | efbe027e95dc13ac343b6130948418d7ead7ddf1 (patch) | |
tree | 96666b3d26027656183a64e96ffa3babcf8bd184 /kernel | |
parent | 5436499e6098759c2340f8b906ea52f993dc4efb (diff) | |
download | op-kernel-dev-efbe027e95dc13ac343b6130948418d7ead7ddf1.zip op-kernel-dev-efbe027e95dc13ac343b6130948418d7ead7ddf1.tar.gz |
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')
-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 74498c8..bb9c638 100644 --- a/kernel/sched.c +++ b/kernel/sched.c @@ -3685,7 +3685,7 @@ out_balanced: static void idle_balance(int this_cpu, struct rq *this_rq) { struct sched_domain *sd; - int pulled_task = -1; + int pulled_task = 0; unsigned long next_balance = jiffies + HZ; cpumask_t tmpmask; |