On Wed, 2020-04-15 at 15:01 +0200, Klaus Jensen wrote:
> From: Klaus Jensen <k.jen...@samsung.com>
>
> 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