Commit 8130b05c authored by Vincent Guittot's avatar Vincent Guittot Committed by Rafael J. Wysocki

PM: EM: fix wrong utilization estimation in em_cpu_energy()

Commit 1b600da5 ("PM: EM: Optimize em_cpu_energy() and remove division")
has added back map_util_perf() in em_cpu_energy() computation which has
been removed with the rework of scheduler/cpufreq interface.
This is wrong because sugov_effective_cpu_perf() already takes care of
mapping the utilization to a performance level.

Fixes: 1b600da5 ("PM: EM: Optimize em_cpu_energy() and remove division")
Signed-off-by: default avatarVincent Guittot <vincent.guittot@linaro.org>
Reviewed-by: default avatarLukasz Luba <lukasz.luba@arm.com>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent 39cd87c4
...@@ -245,7 +245,6 @@ static inline unsigned long em_cpu_energy(struct em_perf_domain *pd, ...@@ -245,7 +245,6 @@ static inline unsigned long em_cpu_energy(struct em_perf_domain *pd,
* max utilization to the allowed CPU capacity before calculating * max utilization to the allowed CPU capacity before calculating
* effective performance. * effective performance.
*/ */
max_util = map_util_perf(max_util);
max_util = min(max_util, allowed_cpu_cap); max_util = min(max_util, allowed_cpu_cap);
/* /*
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment