On 12 November 2016 at 03:28, Rafael J. Wysocki rafael@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