qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v4 06/14] hw/block/nvme: Add support for active/inactive name


From: Klaus Jensen
Subject: Re: [PATCH v4 06/14] hw/block/nvme: Add support for active/inactive namespaces
Date: Thu, 24 Sep 2020 20:55:24 +0200

On Sep 24 18:17, Niklas Cassel wrote:
> On Thu, Sep 24, 2020 at 02:12:03PM +0200, Klaus Jensen wrote:
> > On Sep 24 03:20, Dmitry Fomichev wrote:
> > > From: Niklas Cassel <niklas.cassel@wdc.com>
> > > 
> > > E.g., if the user sets CC.CSS to Admin Only, NVM namespaces should be
> > > marked as inactive.
> > > 
> > 
> > Hmm. I'm not convinced that this is correct. Can you reference the spec?
> > 
> 
> CC.CSS can only be changed when the controller is disabled.

Right. I think I see you point. While the controller is disabled, the
host obviously cannot even see what namespaces are available, so the
controller is free to only expose (aka, attach) the namespaces that
makes sense for the value of CC.CSS.

OK then, the patch is good :)

Attachment: signature.asc
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]