On 12 November 2016 at 03:28, Rafael J. Wysocki <raf...@kernel.org> wrote:
>> @@ -478,8 +484,6 @@ static void sugov_exit(struct cpufreq_policy *policy) >> struct sugov_tunables *tunables = sg_policy->tunables; >> unsigned int count; >> >> - cpufreq_disable_fast_switch(policy); >> - > > ->but why is this change necessary? > > sugov_stop() has been called already, so the ordering here shouldn't matter. Because sugov_policy_free() would be using the flag fast_switch_enabled. -- viresh