From: Akihiko Odaki <akihiko.odaki@daynix.com>
To: "Michael S. Tsirkin" <mst@redhat.com>,
"Marcel Apfelbaum" <marcel.apfelbaum@gmail.com>,
"Alex Williamson" <alex.williamson@redhat.com>,
"Cédric Le Goater" <clg@redhat.com>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Daniel P. Berrangé" <berrange@redhat.com>,
"Eduardo Habkost" <eduardo@habkost.net>,
"Sriram Yagnaraman" <sriram.yagnaraman@est.tech>,
"Jason Wang" <jasowang@redhat.com>,
"Keith Busch" <kbusch@kernel.org>,
"Klaus Jensen" <its@irrelevant.dk>
Cc: qemu-devel@nongnu.org, qemu-block@nongnu.org,
Akihiko Odaki <akihiko.odaki@daynix.com>
Subject: [PATCH 0/6] hw/pci: SR-IOV related fixes and improvements
Date: Sat, 03 Feb 2024 18:31:54 +0900 [thread overview]
Message-ID: <20240203-reuse-v1-0-5be8c5ce6338@daynix.com> (raw)
I submitted a RFC series[1] to add support for SR-IOV emulation to
virtio-net-pci. During the development of the series, I fixed some
trivial bugs and made improvements that I think are independently
useful. This series extracts those fixes and improvements from the RFC
series. Below is an explanation of the patches:
Patch 1 adds a function to check if ROM BAR is explicitly enabled. It
is used in the RFC series to report an error if the user requests to
enable ROM BAR for SR-IOV VF. Patch 2 and 3 use it for vfio to remove
hacky device option dictionary inspection.
Patch 4 adds SR-IOV NumVFs validation to fix potential buffer overflow.
Patch 5 changes to realize SR-IOV VFs when the PF is being realized to
validate VF configuration.
Patch 6 fixes memory leak that occurs if a SR-IOV VF fails to realize.
[1]: https://patchew.org/QEMU/20231210-sriov-v2-0-b959e8a6dfaf@daynix.com/
Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
---
Akihiko Odaki (6):
hw/pci: Determine if rombar is explicitly enabled
vfio: Avoid inspecting option QDict for rombar
hw/qdev: Remove opts member
pcie_sriov: Validate NumVFs
pcie_sriov: Reuse SR-IOV VF device instances
pcie_sriov: Release VFs failed to realize
docs/pcie_sriov.txt | 8 ++--
include/hw/pci/pci.h | 2 +-
include/hw/pci/pci_device.h | 7 ++-
include/hw/pci/pcie_sriov.h | 6 +--
include/hw/qdev-core.h | 4 --
hw/core/qdev.c | 1 -
hw/net/igb.c | 13 ++++--
hw/nvme/ctrl.c | 29 ++++++------
hw/pci/pci.c | 20 +++++----
hw/pci/pci_host.c | 4 +-
hw/pci/pcie.c | 4 +-
hw/pci/pcie_sriov.c | 105 +++++++++++++++++++++-----------------------
hw/vfio/pci.c | 3 +-
system/qdev-monitor.c | 12 ++---
14 files changed, 116 insertions(+), 102 deletions(-)
---
base-commit: 4a4efae44f19528589204581e9e2fab69c5d39aa
change-id: 20240129-reuse-faae22b11934
Best regards,
--
Akihiko Odaki <akihiko.odaki@daynix.com>
next reply other threads:[~2024-02-03 9:33 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-03 9:31 Akihiko Odaki [this message]
2024-02-03 9:31 ` [PATCH 1/6] hw/pci: Determine if rombar is explicitly enabled Akihiko Odaki
2024-02-03 9:31 ` [PATCH 2/6] vfio: Avoid inspecting option QDict for rombar Akihiko Odaki
2024-02-03 9:31 ` [PATCH 3/6] hw/qdev: Remove opts member Akihiko Odaki
2024-02-03 9:31 ` [PATCH 4/6] pcie_sriov: Validate NumVFs Akihiko Odaki
2024-02-03 9:31 ` [PATCH 5/6] pcie_sriov: Reuse SR-IOV VF device instances Akihiko Odaki
2024-02-03 9:32 ` [PATCH 6/6] pcie_sriov: Release VFs failed to realize Akihiko Odaki
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240203-reuse-v1-0-5be8c5ce6338@daynix.com \
--to=akihiko.odaki@daynix.com \
--cc=alex.williamson@redhat.com \
--cc=berrange@redhat.com \
--cc=clg@redhat.com \
--cc=eduardo@habkost.net \
--cc=its@irrelevant.dk \
--cc=jasowang@redhat.com \
--cc=kbusch@kernel.org \
--cc=marcel.apfelbaum@gmail.com \
--cc=mst@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=sriram.yagnaraman@est.tech \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).