Revision 64897b20eed29cee2b998fb5ba362e65523dba3c authored by Rafael J. Wysocki on 21 March 2017, 21:19:07 UTC, committed by Rafael J. Wysocki on 21 March 2017, 21:19:07 UTC
The policy->cpuinfo.max_freq and policy->max updates in intel_cpufreq_turbo_update() are excessive as they are done for no good reason and may lead to problems in principle, so they should be dropped. However, after dropping them intel_cpufreq_turbo_update() becomes almost entirely pointless, because the check made by it is made again down the road in intel_pstate_prepare_request(). The only thing in it that still needs to be done is the call to update_turbo_state(), so drop intel_cpufreq_turbo_update() altogether and make its callers invoke update_turbo_state() directly instead of it. In addition to that, fix intel_cpufreq_verify_policy() so that it checks global.no_turbo in addition to global.turbo_disabled when updating policy->cpuinfo.max_freq to make it consistent with intel_pstate_verify_policy(). Fixes: 001c76f05b01 (cpufreq: intel_pstate: Generic governors support) Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
1 parent 7de3255
File | Mode | Size |
---|---|---|
Kconfig | -rw-r--r-- | 2.6 KB |
Makefile | -rw-r--r-- | 496 bytes |
affinity.c | -rw-r--r-- | 3.9 KB |
autoprobe.c | -rw-r--r-- | 4.5 KB |
chip.c | -rw-r--r-- | 30.2 KB |
cpuhotplug.c | -rw-r--r-- | 2.1 KB |
debug.h | -rw-r--r-- | 1.2 KB |
devres.c | -rw-r--r-- | 5.6 KB |
dummychip.c | -rw-r--r-- | 1.3 KB |
generic-chip.c | -rw-r--r-- | 15.7 KB |
handle.c | -rw-r--r-- | 5.3 KB |
internals.h | -rw-r--r-- | 6.9 KB |
ipi.c | -rw-r--r-- | 9.2 KB |
irqdesc.c | -rw-r--r-- | 21.0 KB |
irqdomain.c | -rw-r--r-- | 41.3 KB |
manage.c | -rw-r--r-- | 54.1 KB |
migration.c | -rw-r--r-- | 2.0 KB |
msi.c | -rw-r--r-- | 10.9 KB |
pm.c | -rw-r--r-- | 5.1 KB |
proc.c | -rw-r--r-- | 12.2 KB |
resend.c | -rw-r--r-- | 2.4 KB |
settings.h | -rw-r--r-- | 4.4 KB |
spurious.c | -rw-r--r-- | 11.9 KB |
![swh spinner](/static/img/swh-spinner.gif)
Computing file changes ...