diff options
author | venkatesh.pallipadi@intel.com <venkatesh.pallipadi@intel.com> | 2008-07-30 19:21:43 -0700 |
---|---|---|
committer | Andi Kleen <ak@linux.intel.com> | 2008-08-15 21:25:25 +0200 |
commit | 320eee776357db52d6fcfb11cff985b1976a4595 (patch) | |
tree | 29859b2e6209a35fc6a735fa4c195e7daeaa1b31 /drivers/cpuidle/cpuidle.c | |
parent | a2bd92023357e47f22a34d4cb1635453546662bc (diff) |
cpuidle: Menu governor fix wrong usage of measured_us
There is a bug in menu governor where we have
if (data->elapsed_us < data->elapsed_us + measured_us)
with measured_us already having elapsed_us added in tickless case here
unsigned int measured_us =
cpuidle_get_last_residency(dev) + data->elapsed_us;
Also, it should be last_residency, not measured_us, that need to be used to
do comparing and distinguish between expected & non-expected events.
Refactor menu_reflect() to fix these two problems.
Signed-off-by: Venkatesh Pallipadi <venkatesh.pallipadi@intel.com>
Signed-off-by: Wei Gang <gang.wei@intel.com>
Signed-off-by: Andi Kleen <ak@linux.intel.com>
Diffstat (limited to 'drivers/cpuidle/cpuidle.c')
0 files changed, 0 insertions, 0 deletions