diff options
author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2017-03-21 22:19:07 +0100 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2017-03-21 22:19:07 +0100 |
commit | 64897b20eed29cee2b998fb5ba362e65523dba3c (patch) | |
tree | 31b8e9fdddad8ca4b50fbb285a5feed777c406a1 /README | |
parent | 7de32556dfc62b9e1203730cc26b71292da8a244 (diff) | |
download | lwn-64897b20eed29cee2b998fb5ba362e65523dba3c.tar.gz lwn-64897b20eed29cee2b998fb5ba362e65523dba3c.zip |
cpufreq: intel_pstate: Fix policy data management in passive mode
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>
Diffstat (limited to 'README')
0 files changed, 0 insertions, 0 deletions