diff options
| author | venkatesh.pallipadi@intel.com <venkatesh.pallipadi@intel.com> | 2008-07-31 04:21:43 +0200 |
|---|---|---|
| committer | Andi Kleen <ak@linux.intel.com> | 2008-08-15 21:25:25 +0200 |
| commit | 320eee776357db52d6fcfb11cff985b1976a4595 (patch) | |
| tree | 29859b2e6209a35fc6a735fa4c195e7daeaa1b31 /lib/iomap_copy.c | |
| parent | cpuidle: Do not use poll_idle unless user asks for it (diff) | |
| download | linux-320eee776357db52d6fcfb11cff985b1976a4595.tar.xz linux-320eee776357db52d6fcfb11cff985b1976a4595.zip | |
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 'lib/iomap_copy.c')
0 files changed, 0 insertions, 0 deletions