Re: [PATCH v2 1/4] hw/nvme: claim NVMe 2.0 compliance
On Jun 27 10:59, Christoph Hellwig wrote: > On Mon, Jun 27, 2022 at 01:47:28PM +0200, Niklas Cassel via wrote: > > CRMS.CRWMS bit shall be set to 1 on controllers compliant with versions > > later than NVMe 1.4. > > > > The first version later than NVMe 1.4 is NVMe 2.0 > > > > Let's claim compliance with NVMe 2.0 such that a follow up patch can > > set the CRMS.CRWMS bit. > > > > This is needed since CC.CRIME is only writable when both CRMS.CRIMS > > and CRMS.CRWMS is set. > > You can also always support newer features without claiming > compliance for the new version. I'd suggest to go through the > mandatory changes list first before upgrading the compliance. Agreed. > (And one day it would be neat if someone tried to run the official > but commercial compliance tests on qemu a well..) Ouch! signature.asc Description: PGP signature
Re: [PATCH v2 1/4] hw/nvme: claim NVMe 2.0 compliance
On Mon, Jun 27, 2022 at 01:47:28PM +0200, Niklas Cassel via wrote: > CRMS.CRWMS bit shall be set to 1 on controllers compliant with versions > later than NVMe 1.4. > > The first version later than NVMe 1.4 is NVMe 2.0 > > Let's claim compliance with NVMe 2.0 such that a follow up patch can > set the CRMS.CRWMS bit. > > This is needed since CC.CRIME is only writable when both CRMS.CRIMS > and CRMS.CRWMS is set. You can also always support newer features without claiming compliance for the new version. I'd suggest to go through the mandatory changes list first before upgrading the compliance. (And one day it would be neat if someone tried to run the official but commercial compliance tests on qemu a well..)
[PATCH v2 1/4] hw/nvme: claim NVMe 2.0 compliance
CRMS.CRWMS bit shall be set to 1 on controllers compliant with versions later than NVMe 1.4. The first version later than NVMe 1.4 is NVMe 2.0 Let's claim compliance with NVMe 2.0 such that a follow up patch can set the CRMS.CRWMS bit. This is needed since CC.CRIME is only writable when both CRMS.CRIMS and CRMS.CRWMS is set. Signed-off-by: Niklas Cassel --- hw/nvme/ctrl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/nvme/ctrl.c b/hw/nvme/ctrl.c index f8ec4a7be3..8ca824ea14 100644 --- a/hw/nvme/ctrl.c +++ b/hw/nvme/ctrl.c @@ -204,7 +204,7 @@ #define NVME_MAX_IOQPAIRS 0x #define NVME_DB_SIZE 4 -#define NVME_SPEC_VER 0x00010400 +#define NVME_SPEC_VER 0x0002 #define NVME_CMB_BIR 2 #define NVME_PMR_BIR 4 #define NVME_TEMPERATURE 0x143 -- 2.36.1
