[PATCH v3 2/2] nvme: Enable autonomous power state transitions

Christoph Hellwig hch at lst.de
Mon Feb 6 00:58:05 PST 2017


On Sat, Feb 04, 2017 at 07:46:45PM -0800, Andy Lutomirski wrote:
> The only decent argument I have for not just putting
> nvme_configure_apst() into nvme_init_identify() is the code in pci.c's
> nvme_setup_io_queues() that tears down the admin queue interrupt and
> the whole BAR mapping after nvme_init_identify().  This shouldn't be a
> problem, but it makes me a tiny bit nervous to have APST on when we do
> this -- is it plausible that something bad would happen if the NVMe
> device took a nap while we this happens?
> 
> Christoph, Keith, and/or Jens, any insight here?  Maybe I should stop
> worrying about this and just move everything into
> nvme_init_identify().

I would much prefer to move the code to nvme_init_identify.  The
NVMe spec has no prohibition on taking down interrupts after setting
up features.  In fact there are all kinds of drivers that run NVMe
devices without any interrupts at all.



More information about the Linux-nvme mailing list