Thread (9 messages) 9 messages, 4 authors, 2021-07-06

RE: [EXT] Re: [PATCH] nvme: Add abrupt shutdown support

From: Shivamurthy Shastri (sshivamurthy) <hidden>
Date: 2021-07-06 08:14:50
Also in: lkml

Micron Confidential

Hi,

On Mon, Jul 05, 2021 at 01:53:49PM +0200, Christoph Hellwig wrote:
quoted
On Mon, Jul 05, 2021 at 11:15:51AM +0000, Shivamurthy Shastri
(sshivamurthy) wrote:
quoted
quoted
For systems with passive electronic components to handle power-
loss/battery-removal,
quoted
quoted
CC.SHN=10b will help to achieve safer shutdown with relatively lower power
back-up
quoted
quoted
requirements (at lower cost).

The module parameter gives flexibility to the host to decide shutdown type
even during
quoted
quoted
boot failure.
If you have a device that always prefers an abrupt please introduce a TP
in the NVMe technical working group so that the hosts automatically
do the right thing.  Requiring sysadmin intervention using a global
kernel parameter is not the way to do this.
The abrupt shutdown notification is already part of NVMe Specification.
Is it the device that wants an abrupt shutdown or the platform? If the
platform's power is running on limited back-up supply, and could inform
the kernel's power management subsystem of this, then a driver could use
that to determine if the quick shutdown is appropriate.
Yes, the platform is running on limited back-up supply. In this case, abrupt shutdown
option will help to achieve shutdown with relatively low back-up requirements.

Please let me know if you want to include the abrupt shutdown as part of 

static const struct dev_pm_ops nvme_dev_pm_ops = {
	.suspend	= nvme_suspend,
               .....


Thanks,
Shiva



Micron Confidential

_______________________________________________
Linux-nvme mailing list
Linux-nvme@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-nvme
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help