From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751396AbdJXFWb (ORCPT ); Tue, 24 Oct 2017 01:22:31 -0400 Received: from mail-it0-f46.google.com ([209.85.214.46]:50574 "EHLO mail-it0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751296AbdJXFW0 (ORCPT ); Tue, 24 Oct 2017 01:22:26 -0400 X-Google-Smtp-Source: ABhQp+T0a2mvZAqTjpYwuL1uTL1zIps8XDOUgKxrnpFQotP2cpfOts4qLON8cZdbdJDMHHedFGM8uBHDcEcCoSKjMAI= MIME-Version: 1.0 In-Reply-To: <2481137.OYGUrPQscL@aspire.rjw.lan> References: <3806130.B2KCK0tvef@aspire.rjw.lan> <2481137.OYGUrPQscL@aspire.rjw.lan> From: Ulf Hansson Date: Tue, 24 Oct 2017 07:22:25 +0200 Message-ID: Subject: Re: [PATCH 04/12] PM / core: Add SMART_SUSPEND driver flag To: "Rafael J. Wysocki" Cc: Linux PM , Bjorn Helgaas , Alan Stern , Greg Kroah-Hartman , LKML , Linux ACPI , Linux PCI , Linux Documentation , Mika Westerberg , Andy Shevchenko , Kevin Hilman , Wolfram Sang , "linux-i2c@vger.kernel.org" , Lee Jones Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 16 October 2017 at 03:29, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > Define and document a SMART_SUSPEND flag to instruct bus types and PM > domains that the system suspend callbacks provided by the driver can > cope with runtime-suspended devices, so from the driver's perspective > it should be safe to leave devices in runtime suspend during system > suspend. > > Setting that flag also causes the PM core to skip the "late" and > "noirq" phases of device suspend for devices that remain in runtime > suspend at the beginning of the "late" phase (when runtime PM has > been disabled for them) under the assumption that their state cannot > (and should not) change after that point until the system suspend > transition is complete. Moreover, the PM core prevents runtime PM > from acting on devices with DPM_FLAG_SMART_SUSPEND during system > resume by setting their runtime PM status to "active" at the end of > the "early" phase (right prior to enabling runtime PM for them). > That allows system resume callbacks to do whatever is necessary to > resume the device without worrying about runtime PM possibly > running in parallel with them. After some sleep, I woke up and realized that the hole thing of making the PM core skip invoking system sleep callbacks, is not compatible with devices being attached to the genpd. Sorry. The reason is because genpd may not power off the PM domain, even if all devices attached to it are runtime suspended. For example, due to a subdomain holding it powered or because a PM QoS constraints prevents to power off it in runtime. Then to understand whether it shall power off/on the PM domain, during system-wide PM it requires the system sleep callbacks to be invoked. So, even if the driver can cope with the behavior from DPM_FLAG_SMART_SUSPEND, then what happens when the PM domain (genpd) can not? Taking this into account, this feels like solution entirely specific to ACPI and PCI. That is fine by me, however then we still have those cross SoC drivers, the i2c-designware driver, which may have its device attached to an ACPI PM domain or perhaps a genpd. > > However, that doesn't apply to transitions involving ->thaw_noirq, > ->thaw_early and ->thaw callbacks during hibernation, as they > generally are not expected to change the power states of devices. > Consequently, if a device is in runtime suspend at the beginning > of such a transition, it must stay in runtime suspend until the > "complete" phase of it (since the callbacks may not change its > power state). > [...] Kind regards Uffe