Commit f9122ee4 authored by Gautham R. Shenoy's avatar Gautham R. Shenoy Committed by Michael Ellerman

cpuidle-powernv: Allow Deep stop states that don't stop time

The current code in the cpuidle-powernv intialization only allows deep
stop states (indicated by OPAL_PM_STOP_INST_DEEP) which lose timebase
(indicated by OPAL_PM_TIMEBASE_STOP). This assumption goes back to
POWER8 time where deep states used to lose the timebase. However, on
POWER9, we do have stop states that are deep (they lose hypervisor
state) but retain the timebase.

Fix the initialization code in the cpuidle-powernv driver to allow
such deep states.

Further, there is a bug in cpuidle-powernv driver with
CONFIG_TICK_ONESHOT=n where we end up incrementing the nr_idle_states
even if a platform idle state which loses time base was not added to
the cpuidle table.

Fix this by ensuring that the nr_idle_states variable gets incremented
only when the platform idle state was added to the cpuidle table.
Signed-off-by: default avatarGautham R. Shenoy <ego@linux.vnet.ibm.com>
Signed-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
parent 22c6663d
...@@ -354,6 +354,7 @@ static int powernv_add_idle_states(void) ...@@ -354,6 +354,7 @@ static int powernv_add_idle_states(void)
for (i = 0; i < dt_idle_states; i++) { for (i = 0; i < dt_idle_states; i++) {
unsigned int exit_latency, target_residency; unsigned int exit_latency, target_residency;
bool stops_timebase = false;
/* /*
* If an idle state has exit latency beyond * If an idle state has exit latency beyond
* POWERNV_THRESHOLD_LATENCY_NS then don't use it * POWERNV_THRESHOLD_LATENCY_NS then don't use it
...@@ -381,6 +382,9 @@ static int powernv_add_idle_states(void) ...@@ -381,6 +382,9 @@ static int powernv_add_idle_states(void)
} }
} }
if (flags[i] & OPAL_PM_TIMEBASE_STOP)
stops_timebase = true;
/* /*
* For nap and fastsleep, use default target_residency * For nap and fastsleep, use default target_residency
* values if f/w does not expose it. * values if f/w does not expose it.
...@@ -392,8 +396,7 @@ static int powernv_add_idle_states(void) ...@@ -392,8 +396,7 @@ static int powernv_add_idle_states(void)
add_powernv_state(nr_idle_states, "Nap", add_powernv_state(nr_idle_states, "Nap",
CPUIDLE_FLAG_NONE, nap_loop, CPUIDLE_FLAG_NONE, nap_loop,
target_residency, exit_latency, 0, 0); target_residency, exit_latency, 0, 0);
} else if ((flags[i] & OPAL_PM_STOP_INST_FAST) && } else if (has_stop_states && !stops_timebase) {
!(flags[i] & OPAL_PM_TIMEBASE_STOP)) {
add_powernv_state(nr_idle_states, names[i], add_powernv_state(nr_idle_states, names[i],
CPUIDLE_FLAG_NONE, stop_loop, CPUIDLE_FLAG_NONE, stop_loop,
target_residency, exit_latency, target_residency, exit_latency,
...@@ -405,8 +408,8 @@ static int powernv_add_idle_states(void) ...@@ -405,8 +408,8 @@ static int powernv_add_idle_states(void)
* within this config dependency check. * within this config dependency check.
*/ */
#ifdef CONFIG_TICK_ONESHOT #ifdef CONFIG_TICK_ONESHOT
if (flags[i] & OPAL_PM_SLEEP_ENABLED || else if (flags[i] & OPAL_PM_SLEEP_ENABLED ||
flags[i] & OPAL_PM_SLEEP_ENABLED_ER1) { flags[i] & OPAL_PM_SLEEP_ENABLED_ER1) {
if (!rc) if (!rc)
target_residency = 300000; target_residency = 300000;
/* Add FASTSLEEP state */ /* Add FASTSLEEP state */
...@@ -414,14 +417,15 @@ static int powernv_add_idle_states(void) ...@@ -414,14 +417,15 @@ static int powernv_add_idle_states(void)
CPUIDLE_FLAG_TIMER_STOP, CPUIDLE_FLAG_TIMER_STOP,
fastsleep_loop, fastsleep_loop,
target_residency, exit_latency, 0, 0); target_residency, exit_latency, 0, 0);
} else if ((flags[i] & OPAL_PM_STOP_INST_DEEP) && } else if (has_stop_states && stops_timebase) {
(flags[i] & OPAL_PM_TIMEBASE_STOP)) {
add_powernv_state(nr_idle_states, names[i], add_powernv_state(nr_idle_states, names[i],
CPUIDLE_FLAG_TIMER_STOP, stop_loop, CPUIDLE_FLAG_TIMER_STOP, stop_loop,
target_residency, exit_latency, target_residency, exit_latency,
psscr_val[i], psscr_mask[i]); psscr_val[i], psscr_mask[i]);
} }
#endif #endif
else
continue;
nr_idle_states++; nr_idle_states++;
} }
out: out:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment