Re: [PATCH 5/5] cpufreq: Get rid of ->governor_enabled and its lock

From: Rafael J. Wysocki
Date: Tue Feb 02 2016 - 16:57:38 EST


On Tue, Feb 2, 2016 at 11:57 AM, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
> Invalid state-transitions is verified by governor core now

What about the governors that don't use cpufreq_governor_dbs()?

> and there is no need to replicate that in cpufreq core. Also we don't drop
> policy->rwsem anymore, which makes rest of the races go away.
>
> Simplify code a bit now.
>
> Signed-off-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx>

I won't make this change just yet. At least not in 4.5 (provided that
the other patches go into it).

Thanks,
Rafael