Commit 816bb611 authored by Pallipadi, Venkatesh's avatar Pallipadi, Venkatesh Committed by Len Brown

cpuidle: Add decaying history logic to menu idle predictor

Add decaying history of predicted idle time, instead of using the last early
wakeup. This logic helps menu governor do better job of predicting idle time.

With this change, we also measured noticable (~8%) power savings on
a DP server system with CPUs supporting deep C states, when system
was lightly loaded. There was no change to power or perf on other load
conditions.
Signed-off-by: default avatarVenkatesh Pallipadi <venkatesh.pallipadi@intel.com>
Signed-off-by: default avatarLen Brown <len.brown@intel.com>
parent 4a6908a3
...@@ -15,12 +15,14 @@ ...@@ -15,12 +15,14 @@
#include <linux/tick.h> #include <linux/tick.h>
#define BREAK_FUZZ 4 /* 4 us */ #define BREAK_FUZZ 4 /* 4 us */
#define PRED_HISTORY_PCT 50
struct menu_device { struct menu_device {
int last_state_idx; int last_state_idx;
unsigned int expected_us; unsigned int expected_us;
unsigned int predicted_us; unsigned int predicted_us;
unsigned int current_predicted_us;
unsigned int last_measured_us; unsigned int last_measured_us;
unsigned int elapsed_us; unsigned int elapsed_us;
}; };
...@@ -47,6 +49,12 @@ static int menu_select(struct cpuidle_device *dev) ...@@ -47,6 +49,12 @@ static int menu_select(struct cpuidle_device *dev)
data->expected_us = data->expected_us =
(u32) ktime_to_ns(tick_nohz_get_sleep_length()) / 1000; (u32) ktime_to_ns(tick_nohz_get_sleep_length()) / 1000;
/* Recalculate predicted_us based on prediction_history_pct */
data->predicted_us *= PRED_HISTORY_PCT;
data->predicted_us += (100 - PRED_HISTORY_PCT) *
data->current_predicted_us;
data->predicted_us /= 100;
/* find the deepest idle state that satisfies our constraints */ /* find the deepest idle state that satisfies our constraints */
for (i = CPUIDLE_DRIVER_STATE_START + 1; i < dev->state_count; i++) { for (i = CPUIDLE_DRIVER_STATE_START + 1; i < dev->state_count; i++) {
struct cpuidle_state *s = &dev->states[i]; struct cpuidle_state *s = &dev->states[i];
...@@ -97,7 +105,7 @@ static void menu_reflect(struct cpuidle_device *dev) ...@@ -97,7 +105,7 @@ static void menu_reflect(struct cpuidle_device *dev)
measured_us = -1; measured_us = -1;
/* Predict time until next break event */ /* Predict time until next break event */
data->predicted_us = max(measured_us, data->last_measured_us); data->current_predicted_us = max(measured_us, data->last_measured_us);
if (last_idle_us + BREAK_FUZZ < if (last_idle_us + BREAK_FUZZ <
data->expected_us - target->exit_latency) { data->expected_us - target->exit_latency) {
......
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