Messages in this thread | | | Subject | Re: [PATCH v2 1/2] sched/fair: Take thermal pressure into account while estimating energy | From | Lukasz Luba <> | Date | Thu, 10 Jun 2021 13:53:15 +0100 |
| |
On 6/10/21 1:40 PM, Vincent Guittot wrote: > On Thu, 10 Jun 2021 at 14:30, Lukasz Luba <lukasz.luba@arm.com> 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
| |