[PATCH 17/18] nvme: move namespace scanning to common code

Christoph Hellwig hch at lst.de
Thu Oct 22 09:30:00 PDT 2015


On Thu, Oct 22, 2015 at 01:48:30PM +0000, Busch, Keith wrote:
> > Yes, someone should sit down and think about the concurrency model
> > for namespace scanning.  Also we really should look into using Identify
> > for the active name space list as that will be a lot more efficient for
> > controllers with lots of namespaces.
> 
> Here's a namespace list proposal:
> 
> http://lists.infradead.org/pipermail/linux-nvme/2015-September/002325.html
> 
> I'll rebase it on the latest and resend.

Looks good.  I suspect we might want a fallback if the controller
doesn't support Identify 0x2 - NVMe 1.1 doesn't mention anything about
Identify subcommand being optional or not and 1.2 says "Controllers that
support specification revision 1.1 or later shall support this
capability." which isn't as a strong as a must.  And some lazy people
like me implemented NVMe targets systes that don't support it but claim
to be version 1.2, although I'll try to get that fixed ASAP.



More information about the Linux-nvme mailing list