]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
cpufreq: intel_pstate: Fix energy_performance_preference for passive
authorTero Kristo <tero.kristo@linux.intel.com>
Wed, 21 Jun 2023 06:58:39 +0000 (09:58 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 11 Jul 2023 17:39:24 +0000 (19:39 +0200)
[ Upstream commit 03f44ffb3d5be2fceda375d92c70ab6de4df7081 ]

If the intel_pstate driver is set to passive mode, then writing the
same value to the energy_performance_preference sysfs twice will fail.
This is caused by the wrong return value used (index of the matched
energy_perf_string), instead of the length of the passed in parameter.
Fix by forcing the internal return value to zero when the same
preference is passed in by user. This same issue is not present when
active mode is used for the driver.

Fixes: f6ebbcf08f37 ("cpufreq: intel_pstate: Implement passive mode with HWP enabled")
Reported-by: Niklas Neronin <niklas.neronin@intel.com>
Signed-off-by: Tero Kristo <tero.kristo@linux.intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/cpufreq/intel_pstate.c

index 48a4613cef1e1e14a56cc2f07f0e40f4fcfba7f7..ee9e96a1893c60afcf4938b93f358248fc1061a9 100644 (file)
@@ -824,6 +824,8 @@ static ssize_t store_energy_performance_preference(
                        err = cpufreq_start_governor(policy);
                        if (!ret)
                                ret = err;
+               } else {
+                       ret = 0;
                }
        }