Messages in this thread | | | Date | Thu, 9 May 2019 11:25:14 +0200 | From | Christoph Hellwig <> | Subject | Re: [PATCH] nvme-pci: Use non-operational power state instead of D3 on Suspend-to-Idle |
| |
On Thu, May 09, 2019 at 11:19:37AM +0200, Rafael J. Wysocki wrote: > Right, the choice of the target system state has already been made > when their callbacks get invoked (and it has been made by user space, > not by the platform).
From a previous discussion I remember the main problem here is that a lot of consumer NVMe use more power when put into D3hot than just letting the device itself manage the power state transitions themselves. Based on this patch there also might be some other device that want an explicit power state transition from the host, but still not be put into D3hot.
The avoid D3hot at all cost thing seems to be based on the Windows broken^H^H^H^H^H^Hmodern standby principles. So for platforms that follow the modern standby model we need to avoid putting NVMe devices that support power management into D3hot somehow. This patch doesa a few more things, but at least for the device where I was involved in the earlier discussion those are not needed, and from the Linux point of view many of them seem wrong too.
How do you think we best make that distinction? Are the pm_ops enough if we don't use the simple version?
| |