On 8/17/21 8:06 AM, Viresh Kumar wrote:

On 16-08-21, 16:19, Rafael J. Wysocki wrote:

quoted

On Tue, Aug 10, 2021 at 8:13 AM Viresh Kumar [off-list ref] wrote:

quoted

  I understand  that this was done to not do the efficient stuff in case of
   userspace/powersave/performance governors.

   What about reusing following for finding all such cases ?

         policy->governor->flags & CPUFREQ_GOV_DYNAMIC_SWITCHING

   The driver can set a flag to tell if it wants efficient frequencies
   or not, and at runtime we apply efficient algorithm only if the
   current governor does DVFS, which will leave out
   userspace/performance/powersave.

As long as this can be done without actually accessing
policy->governor->flags on every transition, it sounds like a good
idea.

Great.

Vincent, I hope you will be taking this forward then ?

Vincent has been on vacations for quite a while, but he will be back
next week.

Thank you guys for the comments.