Re: [PATCH v6 3/5] thermal/cpufreq: Remove arch_update_thermal_pressure()

From: Dhruva Gole
Date: Tue Mar 26 2024 - 07:47:29 EST


Hi,

On Mar 26, 2024 at 10:16:14 +0100, 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@xxxxxxxxxx>
> Reviewed-by: Lukasz Luba <lukasz.luba@xxxxxxx>
> Acked-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx>
> Tested-by: Lukasz Luba <lukasz.luba@xxxxxxx>
> ---

Makes sense,
Reviewed-by: Dhruva Gole <d-gole@xxxxxx>


--
Best regards,
Dhruva