From mboxrd@z Thu Jan 1 00:00:00 1970 From: hch@lst.de (Christoph Hellwig) Date: Thu, 9 May 2019 08:52:23 +0200 Subject: [PATCH] nvme-pci: Use non-operational power state instead of D3 on Suspend-to-Idle In-Reply-To: <064701C3-2BD4-4D93-891D-B7FBB5040FC4@canonical.com> References: <20190508185955.11406-1-kai.heng.feng@canonical.com> <20190508191624.GA8365@localhost.localdomain> <3CDA9F13-B17C-456F-8CE1-3A63C6E0DC8F@canonical.com> <20190508195159.GA1530@lst.de> <20190509061237.GA15229@lst.de> <064701C3-2BD4-4D93-891D-B7FBB5040FC4@canonical.com> Message-ID: <20190509065223.GA15984@lst.de> On Thu, May 09, 2019@02:48:59PM +0800, Kai-Heng Feng wrote: > Not really, for hibernation pm_suspend_via_s2idle() evaluates to false so > the old code path will be taken. > >> >> And more to the points - if these "modern MS standby" systems are >> becoming common, which it looks they are, we need support in the PM core >> for those instead of working around the decisions in low-level drivers. > > Rafael, what do you think about this? > Including this patch, there are five drivers that use > pm_suspend_via_{firmware,s2idle}() to differentiate between S2I and S3. > So I think maybe it?s time to introduce a new suspend callback for S2I? We also really need something like that to avoid the PCI_DEV_FLAGS_NO_D3 abuse - that flag is a quirk statically set on a device at probe time to prevent any entering of D3 state. >> per definition, although they might not be too useful. I suspect checking >> APSTA might be safer, but if we don't want to rely on APST we should >> check for a power state supporting the condition that the MS document >> quoted in the original document supports. > > If Modern Standby or Connected Standby is not supported by servers, I > don?t think the design documents mean much here. > We probably should check if the platform firmware really supports S2I > instead. That too. As said this really is a platform decision, and needs to be managed by the platform code through the PM core. Individual drivers like nvme can just implement the behavior, but are the absolute wrong place to make decisions on what kinds of suspend to enter.