Re: [PATCH v3] sched/topology: remove sysctl_sched_energy_aware depending on the architecture

[Shrikanth Hegde]

On 9/14/23 9:51 PM, Valentin Schneider wrote:
> On 13/09/23 17:18, Shrikanth Hegde wrote:
>> sysctl_sched_energy_aware is available for the admin to disable/enable
>> energy aware scheduling(EAS). EAS is enabled only if few conditions are
>> met by the platform. They are, asymmetric CPU capacity, no SMT,
>> valid cpufreq policy, frequency invariant load tracking. It is possible
>> platform when booting may not have EAS capability, but can do that after.
>> For example, changing/registering the cpufreq policy.
>>
>> At present, though platform doesn't support EAS, this sysctl is still
>> present and it ends up calling rebuild of sched domain on write to 1 and
>> NOP when writing to 0. That is confusing and un-necessary.
>>
> 

Hi Valentin, Thanks for taking a look at this patch.

> But why would you write to it in the first place? Or do you mean to use
> this as an indicator for userspace that EAS is supported?
> 

Since this sysctl is present and its value being 1, it gives the 
impression to the user that EAS is supported when it is not. 
So its an attempt to correct that part. 

So, yes, this can be an indicator whether platform can do EAS at 
the moment or platform can do EAS and admin has explicitly disabled it.