diff options
| author | venkatesh.pallipadi@intel.com <venkatesh.pallipadi@intel.com> | 2008-07-30 22:21:44 -0400 |
|---|---|---|
| committer | Andi Kleen <ak@linux.intel.com> | 2008-08-15 15:25:35 -0400 |
| commit | 06d9e908b2248f983b186aaf569c58e1430db85d (patch) | |
| tree | a72b6608c6f787df72fe34d1d58f02b4b8d1bef7 | |
| parent | 320eee776357db52d6fcfb11cff985b1976a4595 (diff) | |
cpuidle: Make ladder governor honor latency requirements fully
ladder governor only honored latency requirement when promoting C-states.
Instead. it should check for latency requirement on each idle call,
and demote to appropriate C-state when there is a latency requirement change.
Signed-off-by: Venkatesh Pallipadi <venkatesh.pallipadi@intel.com>
Signed-off-by: Andi Kleen <ak@linux.intel.com>
| -rw-r--r-- | drivers/cpuidle/governors/ladder.c | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/drivers/cpuidle/governors/ladder.c b/drivers/cpuidle/governors/ladder.c index 27ab3bfe375b..a4bec3f919aa 100644 --- a/drivers/cpuidle/governors/ladder.c +++ b/drivers/cpuidle/governors/ladder.c | |||
| @@ -99,6 +99,18 @@ static int ladder_select_state(struct cpuidle_device *dev) | |||
| 99 | 99 | ||
| 100 | /* consider demotion */ | 100 | /* consider demotion */ |
| 101 | if (last_idx > CPUIDLE_DRIVER_STATE_START && | 101 | if (last_idx > CPUIDLE_DRIVER_STATE_START && |
| 102 | dev->states[last_idx].exit_latency > latency_req) { | ||
| 103 | int i; | ||
| 104 | |||
| 105 | for (i = last_idx - 1; i > CPUIDLE_DRIVER_STATE_START; i--) { | ||
| 106 | if (dev->states[i].exit_latency <= latency_req) | ||
| 107 | break; | ||
| 108 | } | ||
| 109 | ladder_do_selection(ldev, last_idx, i); | ||
| 110 | return i; | ||
| 111 | } | ||
| 112 | |||
| 113 | if (last_idx > CPUIDLE_DRIVER_STATE_START && | ||
| 102 | last_residency < last_state->threshold.demotion_time) { | 114 | last_residency < last_state->threshold.demotion_time) { |
| 103 | last_state->stats.demotion_count++; | 115 | last_state->stats.demotion_count++; |
| 104 | last_state->stats.promotion_count = 0; | 116 | last_state->stats.promotion_count = 0; |
