Re: [PATCH] cpufreq: use last policy after online

Viresh Kumar
2015-12-02T02:53:36Z

On 01-12-15, 16:52, Srinivas Pandruvada wrote:
> For cpufreq drivers which use setpolicy interface, after offline->online
> the policy is set to default. This can be reproduced by setting the
> default policy of intel_pstate or longrun to ondemand and then change to
> "performance". After offline and online, the setpolicy will be called with
> the policy=ondemand.
> For drivers using governors this condition is handled by storing
> last_governor, during offline and restoring during online. The same should
> be done for drivers using setpolicy interface. Storing last_policy during
> offline and restoring during online.
>
> Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
> ---
> drivers/cpufreq/cpufreq.c | 14 ++++++++++----
> include/linux/cpufreq.h | 1 +
> 2 files changed, 11 insertions(+), 4 deletions(-)

Okay, the problem looks real. But I think it should be solved a bit
differently.

Try partly reverting the below commit, so that we can use user_policy
instead.

88dc43849587 ("cpufreq: remove redundant 'policy' field from
user_policy")

--
viresh