summaryrefslogtreecommitdiff
path: root/include
diff options
context:
space:
mode:
authorVincent Guittot <vincent.guittot@linaro.org>2024-04-04 13:42:00 +0300
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2024-04-04 20:27:53 +0300
commit8130b05c559d1aa83d0c8971b422ba0da18ef24a (patch)
treef18a8a4b48202f7ff73ca7f2839361c4de5a48cf /include
parent39cd87c4eb2b893354f3b850f916353f2658ae6f (diff)
downloadlinux-8130b05c559d1aa83d0c8971b422ba0da18ef24a.tar.xz
PM: EM: fix wrong utilization estimation in em_cpu_energy()
Commit 1b600da51073 ("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: 1b600da51073 ("PM: EM: Optimize em_cpu_energy() and remove division") Signed-off-by: Vincent Guittot <vincent.guittot@linaro.org> Reviewed-by: Lukasz Luba <lukasz.luba@arm.com> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'include')
-rw-r--r--include/linux/energy_model.h1
1 files changed, 0 insertions, 1 deletions
diff --git a/include/linux/energy_model.h b/include/linux/energy_model.h
index 770755df852f..70cd7258cd29 100644
--- a/include/linux/energy_model.h
+++ b/include/linux/energy_model.h
@@ -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
* effective performance.
*/
- max_util = map_util_perf(max_util);
max_util = min(max_util, allowed_cpu_cap);
/*