| From 69497593510257a1dc10829aa5addacda69ca412 Mon Sep 17 00:00:00 2001 |
| From: Viresh Kumar <viresh.kumar@linaro.org> |
| Date: Tue, 21 Mar 2017 11:36:06 +0530 |
| Subject: [PATCH] cpufreq: Restore policy min/max limits on CPU online |
| |
| commit ff010472fb75670cb5c08671e820eeea3af59c87 upstream. |
| |
| On CPU online the cpufreq core restores the previous governor (or |
| the previous "policy" setting for ->setpolicy drivers), but it does |
| not restore the min/max limits at the same time, which is confusing, |
| inconsistent and real pain for users who set the limits and then |
| suspend/resume the system (using full suspend), in which case the |
| limits are reset on all CPUs except for the boot one. |
| |
| Fix this by making cpufreq_online() restore the limits when an inactive |
| policy is brought online. |
| |
| The commit log and patch are inspired from Rafael's earlier work. |
| |
| Reported-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> |
| Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> |
| Cc: 4.3+ <stable@vger.kernel.org> # 4.3+ |
| Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> |
| Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> |
| |
| diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c |
| index ffba27cefebf..05ebd43f0355 100644 |
| --- a/drivers/cpufreq/cpufreq.c |
| +++ b/drivers/cpufreq/cpufreq.c |
| @@ -1235,6 +1235,9 @@ static int cpufreq_online(unsigned int cpu) |
| for_each_cpu(j, policy->related_cpus) |
| per_cpu(cpufreq_cpu_data, j) = policy; |
| write_unlock_irqrestore(&cpufreq_driver_lock, flags); |
| + } else { |
| + policy->min = policy->user_policy.min; |
| + policy->max = policy->user_policy.max; |
| } |
| |
| if (cpufreq_driver->get && !cpufreq_driver->setpolicy) { |
| -- |
| 2.12.0 |
| |