On Thu, Sep 08, 2016 at 11:29:48PM +0200, Rafael J. Wysocki wrote: > Introduce a new flag in struct dev_pm_info, pm_sleep_in_progress, to > indicate that runtime PM has been disabled because of a PM sleep > transition in progress. [...] > That will allow helpers like pm_runtime_get_sync() to be called > during system sleep transitions without worrying about possible > error codes they may return because runtime PM is disabled at > that point.
I have a suspicion that this patch papers over the direct_complete bug I reported Sep 10 and that the patch is unnecessary once that bug is fixed. AFAICS, runtime PM is only disabled in two places during the system sleep process: In __device_suspend() for devices using direct_complete, and __device_suspend_late() for all devices. In both of these phases (dpm_suspend() and dpm_suspend_late()), the device tree is walked bottom-up. Since we've reordered consumers to the back of dpm_list, they will be treated *before* their suppliers. Thus, runtime PM is disabled on the consumers first, and only later on the suppliers. Then how can it be that runtime PM is already disabled on the supplier? The only scenario I can imagine is that the supplier chose to exercise direct_complete, thus was pm_runtime_disabled() in the __device_suspend() phase, and the consumer did *not* choose to exercise direct_complete and later tried to runtime resume its suppliers and itself. I assume this patch is a replacement for Marek's [v2 08/10]. @Marek, does this scenario match with what you witnessed? Best regards, Lukas
