@@ -71,6 +71,10 @@ static int ladder_select_state(struct cpuidle_driver *drv,
int last_residency, last_idx = ldev->last_state_idx;
int latency_req = pm_qos_request(PM_QOS_CPU_DMA_LATENCY);
+ /* qos ignored, pick the deepest idle state */
+ if (unlikely(latency_req == PM_QOS_DEFAULT_VALUE))
+ return drv->state_count - 1;
+
/* Special case when user has set very strict latency requirement */
if (unlikely(latency_req == 0)) {
ladder_do_selection(ldev, last_idx, 0);
@@ -297,12 +297,21 @@ static int menu_select(struct cpuidle_driver *drv, struct cpuidle_device *dev)
data->needs_update = 0;
}
+ /* qos ignored, pick the deepest idle state */
+ if (unlikely(latency_req == PM_QOS_DEFAULT_VALUE)) {
+ struct cpuidle_state *s = &drv->states[drv->state_count - 1];
+
+ data->last_state_idx = drv->state_count - 1;
+ data->exit_us = s->exit_latency;
+ goto exit_done;
+ }
+
data->last_state_idx = 0;
data->exit_us = 0;
/* Special case when user has set very strict latency requirement */
if (unlikely(latency_req == 0))
- return 0;
+ goto exit_done;
/* determine the expected residency time, round up */
t = ktime_to_timespec(tick_nohz_get_sleep_length());
@@ -361,7 +370,7 @@ static int menu_select(struct cpuidle_driver *drv, struct cpuidle_device *dev)
data->last_state_idx = i;
data->exit_us = s->exit_latency;
}
-
+exit_done:
return data->last_state_idx;
}
If a QOS class is disabled due to platform thermal or power capping needs, idle governor shall pick the deepest idle state to conserve power. Power saving should be prioritized over QOS in this case in that hardware based catastrophic measure is more intrusive than managed idle. Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com> --- drivers/cpuidle/governors/ladder.c | 4 ++++ drivers/cpuidle/governors/menu.c | 13 +++++++++++-- 2 files changed, 15 insertions(+), 2 deletions(-)