Messages in this thread | | | Date | Fri, 15 Dec 2023 15:38:46 +0000 | Subject | Re: [PATCH 3/4] thermal/cpufreq: Remove arch_update_thermal_pressure() | From | Lukasz Luba <> |
| |
On 12/12/23 14:27, Vincent Guittot wrote: > arch_update_thermal_pressure() aims to update fast changing signal which > should be averaged using PELT filtering before being provided to the > scheduler which can't make smart use of fast changing signal. > cpufreq now provides the maximum freq_qos pressure on the capacity to the > scheduler, which includes cpufreq cooling device. Remove the call to > arch_update_thermal_pressure() in cpufreq cooling device as this is > handled by cpufreq_get_pressure(). > > Signed-off-by: Vincent Guittot <vincent.guittot@linaro.org> > --- > drivers/thermal/cpufreq_cooling.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/thermal/cpufreq_cooling.c b/drivers/thermal/cpufreq_cooling.c > index e2cc7bd30862..e77d3b44903e 100644 > --- a/drivers/thermal/cpufreq_cooling.c > +++ b/drivers/thermal/cpufreq_cooling.c > @@ -448,7 +448,6 @@ static int cpufreq_set_cur_state(struct thermal_cooling_device *cdev, > unsigned long state) > { > struct cpufreq_cooling_device *cpufreq_cdev = cdev->devdata; > - struct cpumask *cpus; > unsigned int frequency; > int ret; > > @@ -465,8 +464,6 @@ static int cpufreq_set_cur_state(struct thermal_cooling_device *cdev, > ret = freq_qos_update_request(&cpufreq_cdev->qos_req, frequency); > if (ret >= 0) { > cpufreq_cdev->cpufreq_state = state; > - cpus = cpufreq_cdev->policy->related_cpus; > - arch_update_thermal_pressure(cpus, frequency); > ret = 0; > } >
Reviewed-by: Lukasz Luba <lukasz.luba@arm.com>
| |