Re: [PATCH] cpufreq: schedutil: Move max CPU capacity to sugov_policy

From: Lukasz Luba
Date: Fri Jul 15 2022 - 07:47:46 EST




On 7/15/22 12:44, Rafael J. Wysocki wrote:
On Fri, Jul 15, 2022 at 10:47 AM Lukasz Luba <lukasz.luba@xxxxxxx> wrote:

Hi Rafael,

gentle ping.

On 7/11/22 13:42, Lukasz Luba wrote:
There is no need to keep the max CPU capacity in the per_cpu instance.
Furthermore, there is no need to check and update that variable
(sg_cpu->max) everytime in the frequency change request, which is part
of hot path. Instead use struct sugov_policy to store that information.
Initialize the max CPU capacity during the setup and start callback.
We can do that since all CPUs in the same frequency domain have the same
max capacity (capacity setup and thermal pressure are based on that).

Signed-off-by: Lukasz Luba <lukasz.luba@xxxxxxx>
---
kernel/sched/cpufreq_schedutil.c | 30 +++++++++++++++---------------
1 file changed, 15 insertions(+), 15 deletions(-)

The patch got Ack from Viresh.
Could you take it?

Yes, it's there in my queue. Same for the EM changes.

Thank you Rafael!