Re: [PATCH v2 1/2] sched/fair: Take thermal pressure into account while estimating energy

[Lukasz Luba]

On 6/10/21 1:40 PM, Vincent Guittot wrote:
> On Thu, 10 Jun 2021 at 14:30, Lukasz Luba <lukasz.luba@xxxxxxx> wrote:

[snip]

> > So for this scenario, where we want to just align EAS with SchedUtil
> > frequency decision, which is instantaneous and has 'raw' value
> > of capping from policy->max, shouldn't we use:
> >
> > thermal_pressure = arch_scale_thermal_pressure(cpu_id)
>
> Yes you should probably use arch_scale_thermal_pressure(cpu) instead
> of thermal_load_avg(rq) in this case

Thank you Vincent for valuable opinions!
I will rewrite it and experiment with a new approach,
then send a v3.

Regards,
Lukasz