diff options
author | Mel Gorman <mgorman@techsingularity.net> | 2021-08-04 13:58:57 +0200 |
---|---|---|
committer | Peter Zijlstra <peterz@infradead.org> | 2021-08-04 15:16:44 +0200 |
commit | 56498cfb045d7147cdcba33795d19429afcd1d00 (patch) | |
tree | 71409bf88b04686a90d8050cda527bf5f31065e2 /kernel/sched | |
parent | sched/fair: Use prev instead of new target as recent_used_cpu (diff) | |
download | linux-56498cfb045d7147cdcba33795d19429afcd1d00.tar.xz linux-56498cfb045d7147cdcba33795d19429afcd1d00.zip |
sched/fair: Avoid a second scan of target in select_idle_cpu
When select_idle_cpu starts scanning for an idle CPU, it starts with
a target CPU that has already been checked by select_idle_sibling.
This patch starts with the next CPU instead.
Signed-off-by: Mel Gorman <mgorman@techsingularity.net>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lkml.kernel.org/r/20210804115857.6253-3-mgorman@techsingularity.net
Diffstat (limited to 'kernel/sched')
-rw-r--r-- | kernel/sched/fair.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index 7ee394b4b4cd..47a0fbf224b2 100644 --- a/kernel/sched/fair.c +++ b/kernel/sched/fair.c @@ -6220,7 +6220,7 @@ static int select_idle_cpu(struct task_struct *p, struct sched_domain *sd, bool time = cpu_clock(this); } - for_each_cpu_wrap(cpu, cpus, target) { + for_each_cpu_wrap(cpu, cpus, target + 1) { if (has_idle_core) { i = select_idle_core(p, cpu, cpus, &idle_cpu); if ((unsigned int)i < nr_cpumask_bits) |