[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 00/16] nvme: refactoring and cleanups
From: |
Maxim Levitsky |
Subject: |
Re: [PATCH v2 00/16] nvme: refactoring and cleanups |
Date: |
Tue, 21 Apr 2020 19:24:05 +0300 |
On Wed, 2020-04-15 at 15:01 +0200, Klaus Jensen wrote:
> From: Klaus Jensen <address@hidden>
>
> Changes since v1
> ~~~~~~~~~~~~~~~~
> * nvme: fix pci doorbell size calculation
> - added some defines and a better comment (Philippe)
>
> * nvme: rename trace events to pci_nvme
> - changed the prefix from nvme_dev to pci_nvme (Philippe)
>
> * nvme: add max_ioqpairs device parameter
> - added a deprecation comment. I doubt this will go in until 5.1, so
> changed it to "deprecated from 5.1" (Philippe)
>
> * nvme: factor out property/constraint checks
> * nvme: factor out block backend setup
> - changed to return void and propagate errors in proper QEMU style
> (Philippe)
>
> * nvme: add namespace helpers
> - use the helper immediately (Philippe)
>
> * nvme: factor out pci setup
> - removed setting of vendor and device id which is already inherited
> from nvme_class_init() (Philippe)
>
> * nvme: factor out cmb setup
> - add lost comment (Philippe)
>
>
> Klaus Jensen (16):
> nvme: fix pci doorbell size calculation
> nvme: rename trace events to pci_nvme
> nvme: remove superfluous breaks
> nvme: move device parameters to separate struct
> nvme: use constants in identify
> nvme: refactor nvme_addr_read
> nvme: add max_ioqpairs device parameter
> nvme: remove redundant cmbloc/cmbsz members
> nvme: factor out property/constraint checks
> nvme: factor out device state setup
> nvme: factor out block backend setup
> nvme: add namespace helpers
> nvme: factor out namespace setup
> nvme: factor out pci setup
> nvme: factor out cmb setup
> nvme: factor out controller identify setup
>
> hw/block/nvme.c | 433 ++++++++++++++++++++++++------------------
> hw/block/nvme.h | 36 +++-
> hw/block/trace-events | 172 ++++++++---------
> include/block/nvme.h | 8 +
> 4 files changed, 372 insertions(+), 277 deletions(-)
>
Should I also review the V7 series or I should wait for V8 which will not
include these cleanups?
Best regards,
Maxim Levitsky
- [PATCH v2 14/16] nvme: factor out pci setup, (continued)
- [PATCH v2 14/16] nvme: factor out pci setup, Klaus Jensen, 2020/04/15
- [PATCH v2 15/16] nvme: factor out cmb setup, Klaus Jensen, 2020/04/15
- Re: [PATCH v2 00/16] nvme: refactoring and cleanups, no-reply, 2020/04/15
- Re: [PATCH v2 00/16] nvme: refactoring and cleanups, Klaus Birkelund Jensen, 2020/04/20
- Re: [PATCH v2 00/16] nvme: refactoring and cleanups, Keith Busch, 2020/04/20
- Re: [PATCH v2 00/16] nvme: refactoring and cleanups,
Maxim Levitsky <=