commit f227c2f2d111a3e7cee3abf429fedb99ff00f89f
author Rafael J. Wysocki <rafael.j.wysocki@intel.com> Mon Mar 25 18:04:24 2024 +0100
committer Greg Kroah-Hartman <gregkh@linuxfoundation.org> Tue Sep 09 18:54:24 2025 +0200
tree 8e359556e2651d4ed22485c66e4841c66f57629a
parent 51a579023c9f9b8b382cffb9d4f4301446aea31a

cpufreq: intel_pstate: Read global.no_turbo under READ_ONCE()

[ Upstream commit 9558fae8ce97b3b320b387dd7c88309df2c36d4d ]

Because global.no_turbo is generally not read under intel_pstate_driver_lock
make store_no_turbo() use WRITE_ONCE() for updating it (this is the only
place at which it is updated except for the initialization) and make the
majority of places reading it use READ_ONCE().

Also remove redundant global.turbo_disabled checks from places that
depend on the 'true' value of global.no_turbo because it can only be
'true' if global.turbo_disabled is also 'true'.

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Acked-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Stable-dep-of: 350cbb5d2f67 ("cpufreq: intel_pstate: Check turbo_is_disabled() in store_no_turbo()")
Signed-off-by: Sasha Levin <sashal@kernel.org>