]> git.baikalelectronics.ru Git - kernel.git/commit
cpufreq: intel_pstate: Fix EPP setting via sysfs in active mode
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>
Tue, 28 Jul 2020 17:09:32 +0000 (19:09 +0200)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Thu, 30 Jul 2020 16:20:23 +0000 (18:20 +0200)
commit93c3097588bfb70a8a2b4c0078743ed6f6e62f16
tree3f9bca4261e41622950446bbd46e7649a394b686
parentb5e84ba548a1d411f17353e3b6a6e12d2975979b
cpufreq: intel_pstate: Fix EPP setting via sysfs in active mode

Because intel_pstate_set_energy_pref_index() reads and writes the
MSR_HWP_REQUEST register without using the cached value of it used by
intel_pstate_hwp_boost_up() and intel_pstate_hwp_boost_down(), those
functions may overwrite the value written by it and so the EPP value
set via sysfs may be lost.

To avoid that, make intel_pstate_set_energy_pref_index() take the
cached value of MSR_HWP_REQUEST just like the other two routines
mentioned above and update it with the new EPP value coming from
user space in addition to updating the MSR.

Note that the MSR itself still needs to be updated too in case
hwp_boost is unset or the boosting mechanism is not active at the
EPP change time.

Fixes: 977566f0190b ("cpufreq: intel_pstate: Add HWP boost utility and sched util hooks")
Reported-by: Francisco Jerez <currojerez@riseup.net>
Cc: 4.18+ <stable@vger.kernel.org> # 4.18+: 3da97d4db8ee cpufreq: intel_pstate: Rearrange ...
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Reviewed-by: Francisco Jerez <currojerez@riseup.net>
drivers/cpufreq/intel_pstate.c