diff options
| author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2018-08-16 12:55:01 +0200 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2018-08-16 23:05:43 +0200 |
| commit | 50f7ccc64750610f57983720613713b2c14f0e9c (patch) | |
| tree | 57f9ae7d176444b82245ed474de55af05f712e96 /drivers/cpuidle | |
| parent | x86/ACPI/cstate: Make APCI C1 FFH MWAIT C-state description vendor-neutral (diff) | |
| download | linux-50f7ccc64750610f57983720613713b2c14f0e9c.tar.xz linux-50f7ccc64750610f57983720613713b2c14f0e9c.zip | |
cpuidle: menu: Update stale polling override comment
The comment to explain why the menu governor uses idle state 1
instead of idle state 0 as the first one sometimes is stale (among
other things it mentions a user setting not present any more),
so update it.
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/cpuidle')
| -rw-r--r-- | drivers/cpuidle/governors/menu.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/cpuidle/governors/menu.c b/drivers/cpuidle/governors/menu.c index 5700464baad5..1560a9e76f06 100644 --- a/drivers/cpuidle/governors/menu.c +++ b/drivers/cpuidle/governors/menu.c @@ -328,9 +328,8 @@ static int menu_select(struct cpuidle_driver *drv, struct cpuidle_device *dev, unsigned int polling_threshold; /* - * We want to default to C1 (hlt), not to busy polling - * unless the timer is happening really really soon, or - * C1's exit latency exceeds the user configured limit. + * Default to a physical idle state, not to busy polling, unless + * a timer is going to trigger really really soon. */ polling_threshold = max_t(unsigned int, 20, s->target_residency); if (data->next_timer_us > polling_threshold && |