Re: [PATCH V5 1/3] cpufreq: governor: No need to manage state machine now

From: Rafael J. Wysocki
Date: Thu Feb 11 2016 - 04:59:08 EST


On Wed, Feb 10, 2016 at 8:00 AM, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
> The cpufreq core now guarantees that policy->rwsem won't be dropped
> while running the ->governor callback for the CPUFREQ_GOV_POLICY_EXIT
> event and will be held acquired until the complete sequence of governor
> state changes has finished.
>
> This allows governor state machine checks to be dropped from multiple
> functions in cpufreq_governor.c.
>
> This also means that policy_dbs->policy can be initialized upfront, so
> the entire initialization of struct policy_dbs is carried out in one
> place.
>
> Signed-off-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx>
> Tested-by: Juri Lelli <juri.lelli@xxxxxxx>
> Tested-by: Shilpasri G Bhat <shilpa.bhat@xxxxxxxxxxxxxxxxxx>
> ---
> Rafael,
>
> The 4th patch is dropped and 5-7 are updated on top of that.

There are some comments to address on the changelogs of patches
[1-2/7], so can you address them and resend the whole series, please?

Thanks,
Rafael