[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1464657098-24880-2-git-send-email-yuyang.du@intel.com>
Date: Tue, 31 May 2016 09:11:37 +0800
From: Yuyang Du <yuyang.du@...el.com>
To: peterz@...radead.org, mingo@...nel.org,
linux-kernel@...r.kernel.org
Cc: umgwanakikbuti@...il.com, bsegall@...gle.com, pjt@...gle.com,
morten.rasmussen@....com, vincent.guittot@...aro.org,
dietmar.eggemann@....com, Yuyang Du <yuyang.du@...el.com>
Subject: [RFC PATCH 1/2] sched: Clean up SD_BALANCE_WAKE flags in sched domain build-up
The SD_BALANCE_WAKE is irrelevant in the contexts of these two removals,
and in addition SD_BALANCE_WAKE is not and should not be set in any
sched_domain flags so far.
Signed-off-by: Yuyang Du <yuyang.du@...el.com>
---
kernel/sched/core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/kernel/sched/core.c b/kernel/sched/core.c
index 7f2cae4..e00b8ea 100644
--- a/kernel/sched/core.c
+++ b/kernel/sched/core.c
@@ -6229,10 +6229,10 @@ static void set_domain_attribute(struct sched_domain *sd,
request = attr->relax_domain_level;
if (request < sd->level) {
/* turn off idle balance on this domain */
- sd->flags &= ~(SD_BALANCE_WAKE|SD_BALANCE_NEWIDLE);
+ sd->flags &= ~(SD_BALANCE_NEWIDLE);
} else {
/* turn on idle balance on this domain */
- sd->flags |= (SD_BALANCE_WAKE|SD_BALANCE_NEWIDLE);
+ sd->flags |= (SD_BALANCE_NEWIDLE);
}
}
--
1.7.9.5
Powered by blists - more mailing lists