On May 16 17:25, Lukasz Maniak wrote: > On Mon, May 09, 2022 at 04:16:08PM +0200, Lukasz Maniak wrote: > > Changes since v7: > > - Fixed description of hw/acpi: Make the PCI hot-plug aware of SR-IOV > > - Added description to docs: Add documentation for SR-IOV and > > Virtualization Enhancements > > - Added Reviewed-by and Acked-by tags > > - Rebased on master > > > > Lukasz Maniak (4): > > hw/nvme: Add support for SR-IOV > > hw/nvme: Add support for Primary Controller Capabilities > > hw/nvme: Add support for Secondary Controller List > > docs: Add documentation for SR-IOV and Virtualization Enhancements > > > > Łukasz Gieryk (8): > > hw/nvme: Implement the Function Level Reset > > hw/nvme: Make max_ioqpairs and msix_qsize configurable in runtime > > hw/nvme: Remove reg_size variable and update BAR0 size calculation > > hw/nvme: Calculate BAR attributes in a function > > hw/nvme: Initialize capability structures for primary/secondary > > controllers > > hw/nvme: Add support for the Virtualization Management command > > hw/nvme: Update the initalization place for the AER queue > > hw/acpi: Make the PCI hot-plug aware of SR-IOV > > > > docs/system/devices/nvme.rst | 82 +++++ > > hw/acpi/pcihp.c | 6 +- > > hw/nvme/ctrl.c | 673 ++++++++++++++++++++++++++++++++--- > > hw/nvme/ns.c | 2 +- > > hw/nvme/nvme.h | 55 ++- > > hw/nvme/subsys.c | 75 +++- > > hw/nvme/trace-events | 6 + > > include/block/nvme.h | 65 ++++ > > include/hw/pci/pci_ids.h | 1 + > > 9 files changed, 909 insertions(+), 56 deletions(-) > > > > -- > > 2.25.1 > > > > Hi Klaus, > > Should we consider this series ready to merge? >
Hi Lukasz and Lukasz :) Yes, I'm queing this up. I found a problem when used with SPDK introduced by the "hw/nvme: Add support for the Virtualization Management command" patch. However, it's not really a problem in your patch, its related to the general handling of CSTS and CC in nvme_write_bar(). I'll follow up with a patch on top of this series and when reviewed, I'll apply this series and that patch to nvme-next together. Thanks for following through on this major feature! :) Klaus
signature.asc
Description: PGP signature
