diff options
author | Chris Redpath <chris.redpath@arm.com> | 2015-02-03 16:33:30 +0900 |
---|---|---|
committer | Seung-Woo Kim <sw0312.kim@samsung.com> | 2016-12-14 13:41:44 +0900 |
commit | 6af38d004a4e626e50ed6591d43194e16d831e66 (patch) | |
tree | 0f2f5ab8af8a0a0ab0b6867b476cd71e6fd3d466 /kernel | |
parent | be1fd8f1b73d94d88343ebda53aa361869e75da5 (diff) |
sched: HMP change nr_running offload metric
rq->nr_running was better than cfs.nr_running, since it includes
all tasks actually on the CPU. However, it includes RT tasks which
we would rather ignore at this point.
Switching to cfs.h_nr_running includes all the CFS tasks but no
RT tasks.
Signed-off-by: Chris Redpath <chris.redpath@arm.com>
Signed-off-by: Liviu Dudau <liviu.dudau@arm.com>
Signed-off-by: Jon Medhurst <tixy@linaro.org>
[k.kozlowski: rebased on 4.1, no signed-off-by of previous committer]
Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Diffstat (limited to 'kernel')
-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 91ac25a943d1..bdd0b8263cd8 100644 --- a/kernel/sched/fair.c +++ b/kernel/sched/fair.c @@ -5528,7 +5528,7 @@ static inline unsigned int hmp_offload_down(int cpu, struct sched_entity *se) return NR_CPUS; /* Is the task alone on the cpu? */ - if (cpu_rq(cpu)->cfs.nr_running < 2) + if (cpu_rq(cpu)->cfs.h_nr_running < 2) return NR_CPUS; /* Is the task actually starving? */ |