[v2] virtio-iommu-pci: Advertise the device as modern-only

[PATCH v2 0/2] virtio-iommu-pci: Advertise the device as modern-only

Eric Auger posted 2 patches 3 years, 7 months ago

Diff against v1
Download series mbox

Test docker-quick@centos7 failed

Test docker-mingw@fedora failed

Test checkpatch failed

Test FreeBSD failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200908193309.20569-1-eric.auger@redhat.com

Maintainers: "Michael S. Tsirkin" <mst@redhat.com>, Eric Auger <eric.auger@redhat.com>

hw/virtio/virtio-iommu-pci.c | 2 +-
hw/virtio/virtio-iommu.c     | 8 ++++++--
2 files changed, 7 insertions(+), 3 deletions(-)

Expand all Fold all

[PATCH v2 0/2] virtio-iommu-pci: Advertise the device as modern-only

Posted by Eric Auger 3 years, 7 months ago

Since commit 9b3a35ec82 ("virtio: verify that legacy support is not
accidentally on"), we are forced to use disable-legacy=on when
instantiating the virtio-iommu-pci device. This also revealed that
the unrealize() function is likely to call g_tree_destroy() on
NULL gtrees, which triggers assertions.

Best Regards

Eric

History:
v1 -> v2:
- Fix patch #1 commit message typos
- Add Connie's R-b on both patches

Eric Auger (2):
  virtio-iommu: Check gtrees are non null before destroying them
  virtio-iommu-pci: force virtio version 1

 hw/virtio/virtio-iommu-pci.c | 2 +-
 hw/virtio/virtio-iommu.c     | 8 ++++++--
 2 files changed, 7 insertions(+), 3 deletions(-)

-- 
2.21.3