Messages in this thread | | | From | Viresh Kumar <> | Date | Sat, 12 Nov 2016 10:49:41 +0530 | Subject | Re: [PATCH 1/3] cpufreq: schedutil: enable fast switch earlier |
| |
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
| |