diff options
author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2025-09-05 16:52:03 +0300 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2025-09-09 13:58:54 +0300 |
commit | 69e5d50fcf4093fb3f9f41c4f931f12c2ca8c467 (patch) | |
tree | f178e6361001703a4884819deed215bcdfae3277 /tools/testing/shared/xarray-shared.c | |
parent | 42c74f6b1c3694892ab92750a60aeb6212d73ac1 (diff) | |
download | linux-69e5d50fcf4093fb3f9f41c4f931f12c2ca8c467.tar.xz |
cpufreq: intel_pstate: Fix object lifecycle issue in update_qos_request()
The cpufreq_cpu_put() call in update_qos_request() takes place too early
because the latter subsequently calls freq_qos_update_request() that
indirectly accesses the policy object in question through the QoS request
object passed to it.
Fortunately, update_qos_request() is called under intel_pstate_driver_lock,
so this issue does not matter for changing the intel_pstate operation
mode, but it theoretically can cause a crash to occur on CPU device hot
removal (which currently can only happen in virt, but it is formally
supported nevertheless).
Address this issue by modifying update_qos_request() to drop the
reference to the policy later.
Fixes: da5c504c7aae ("cpufreq: intel_pstate: Implement QoS supported freq constraints")
Cc: 5.4+ <stable@vger.kernel.org> # 5.4+
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Reviewed-by: Zihuan Zhang <zhangzihuan@kylinos.cn>
Link: https://patch.msgid.link/2255671.irdbgypaU6@rafael.j.wysocki
Diffstat (limited to 'tools/testing/shared/xarray-shared.c')
0 files changed, 0 insertions, 0 deletions