cpuidle: menu: Avoid taking spinlock for accessing QoS values
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>
Fri, 24 Feb 2017 12:25:14 +0000 (13:25 +0100)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Mon, 27 Feb 2017 14:07:38 +0000 (15:07 +0100)
commit6dbf5cea05a7098a69f294c96b6d76f08562cae5
treeb82971769598b518c88773bbb88874875a91aa62
parent37efa4b41ffb31dcdfc3beb97d47992bb2a083e5
cpuidle: menu: Avoid taking spinlock for accessing QoS values

After commit 9908859acaa9 (cpuidle/menu: add per CPU PM QoS resume
latency consideration) the cpuidle menu governor calls
dev_pm_qos_read_value() on CPU devices to read the current resume
latency QoS constraint values for them.  That function takes a spinlock
to prevent the device's power.qos pointer from becoming NULL during
the access which is a problem for the RT patchset where spinlocks are
converted into mutexes and the idle loop stops working.

However, it is not even necessary for the menu governor to take
that spinlock, because the power.qos pointer accessed under it
cannot be modified during the access anyway.

For this reason, introduce a "raw" routine for accessing device
QoS resume latency constraints without locking and use it in the
menu governor.

Fixes: 9908859acaa9 (cpuidle/menu: add per CPU PM QoS resume latency consideration)
Acked-by: Alex Shi <alex.shi@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/base/power/qos.c
drivers/cpuidle/governors/menu.c
include/linux/pm_qos.h