OSDN Git Service

PM: runtime: Defer suspending suppliers
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>
Fri, 19 Mar 2021 14:47:31 +0000 (15:47 +0100)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Mon, 22 Mar 2021 14:21:38 +0000 (15:21 +0100)
commit5244f5e2d801259af877ee759e8c22364c607072
tree4377ad10363aa3375701a3f373ecf41512e38476
parent0d02ec6b3136c73c09e7859f0d0e4e2c4c07b49b
PM: runtime: Defer suspending suppliers

Because the PM-runtime status of the device is not updated in
__rpm_callback(), attempts to suspend the suppliers of the given
device triggered by the rpm_put_suppliers() call in there may
cause a supplier to be suspended completely before the status of
the consumer is updated to RPM_SUSPENDED, which is confusing.

To avoid that (1) modify __rpm_callback() to only decrease the
PM-runtime usage counter of each supplier and (2) make rpm_suspend()
try to suspend the suppliers after changing the consumer's status to
RPM_SUSPENDED, in analogy with the device's parent.

Link: https://lore.kernel.org/linux-pm/CAPDyKFqm06KDw_p8WXsM4dijDbho4bb6T4k50UqqvR1_COsp8g@mail.gmail.com/
Fixes: 21d5c57b3726 ("PM / runtime: Use device links")
Reported-by: elaine.zhang <zhangqing@rock-chips.com>
Diagnosed-by: Ulf Hansson <ulf.hansson@linaro.org>
Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/base/power/runtime.c