Re: [PATCH] nvme: Use NN for max_namespaces if MNAN is zero

From: Christoph Hellwig
Date: Tue May 25 2021 - 03:22:42 EST


On Tue, May 25, 2021 at 09:12:59AM +0200, Daniel Wagner wrote:
> On Mon, May 24, 2021 at 09:37:03AM +0200, Christoph Hellwig wrote:
> > I think we should add a sanity check for that and reject the broken
> > controller if that is not the case rather than working around it.
>
> Alright. I understood from the spec, that in the non ANA case the NN
> field should still be case though?

For non-ANA MNAN doesn't have to be set indeed. But we also don't use
the value at all either.