[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v7 00/11] hw/block/nvme: Support Namespace Types and Zoned Na
From: |
Niklas Cassel |
Subject: |
Re: [PATCH v7 00/11] hw/block/nvme: Support Namespace Types and Zoned Namespace Command Set |
Date: |
Mon, 19 Oct 2020 07:32:02 +0000 |
On Mon, Oct 19, 2020 at 11:17:15AM +0900, Dmitry Fomichev wrote:
(snip)
>
> Dmitry Fomichev (9):
> hw/block/nvme: Add Commands Supported and Effects log
> hw/block/nvme: Generate namespace UUIDs
> hw/block/nvme: Support Zoned Namespace Command Set
> hw/block/nvme: Introduce max active and open zone limits
> hw/block/nvme: Support Zone Descriptor Extensions
> hw/block/nvme: Add injection of Offline/Read-Only zones
> hw/block/nvme: Document zoned parameters in usage text
> hw/block/nvme: Separate read and write handlers
> hw/block/nvme: Merge nvme_write_zeroes() with nvme_write()
>
> Niklas Cassel (2):
> hw/block/nvme: Add support for Namespace Types
> hw/block/nvme: Support allocated CNS command variants
>
> block/nvme.c | 2 +-
> hw/block/nvme-ns.c | 295 ++++++++
> hw/block/nvme-ns.h | 109 +++
> hw/block/nvme.c | 1550 ++++++++++++++++++++++++++++++++++++++---
> hw/block/nvme.h | 9 +
> hw/block/trace-events | 36 +-
> include/block/nvme.h | 201 +++++-
> 7 files changed, 2078 insertions(+), 124 deletions(-)
>
> --
> 2.21.0
>
Thank you Dmitry, this version was easier to review.
Except for a missing
/* fall through */ comment in nvme_cmd_effects().
(in the "hw/block/nvme: Add Commands Supported and Effects log" patch.)
For the whole series:
Reviewed-by: Niklas Cassel <niklas.cassel@wdc.com>