diff options
| author | Akshay Adiga <akshay.adiga@linux.vnet.ibm.com> | 2016-11-14 17:29:27 +0530 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2016-11-14 21:32:31 +0100 |
| commit | c9a81e6864d48f81797397bbd65d299bf20c6148 (patch) | |
| tree | 6a0c091f2d23f42574fa25146c66efb08b96747c /drivers/cpufreq/powernv-cpufreq.c | |
| parent | 7f7a516ee30a931fa2d257b39f5ef7b9196c56fc (diff) | |
| download | linux-c9a81e6864d48f81797397bbd65d299bf20c6148.tar.gz linux-c9a81e6864d48f81797397bbd65d299bf20c6148.tar.bz2 linux-c9a81e6864d48f81797397bbd65d299bf20c6148.zip | |
cpufreq: powernv: Fix uninitialized lpstate_idx in gpstates_timer_handler()
lpstate_idx remains uninitialized in the case when elapsed_time
is greater than MAX_RAMP_DOWN_TIME. At the end of rampdown the
global pstate should be equal to the local pstate.
Fixes: 20b15b766354 (cpufreq: powernv: Use PMCR to verify global and localpstate)
Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: Akshay Adiga <akshay.adiga@linux.vnet.ibm.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/cpufreq/powernv-cpufreq.c')
| -rw-r--r-- | drivers/cpufreq/powernv-cpufreq.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/cpufreq/powernv-cpufreq.c b/drivers/cpufreq/powernv-cpufreq.c index c82304b7a4c0..c5c5bc35eece 100644 --- a/drivers/cpufreq/powernv-cpufreq.c +++ b/drivers/cpufreq/powernv-cpufreq.c @@ -624,6 +624,7 @@ void gpstate_timer_handler(unsigned long data) if (gpstates->elapsed_time > MAX_RAMP_DOWN_TIME) { gpstate_idx = pstate_to_idx(freq_data.pstate_id); + lpstate_idx = gpstate_idx; reset_gpstates(policy); gpstates->highest_lpstate_idx = gpstate_idx; } else { |