[libvirt] [PATCH 0/6] qemu: PCI-related bug fixes and cleanups

Andrea Bolognani posted 6 patches 7 years, 1 month ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/1487707064-29602-1-git-send-email-abologna@redhat.com
src/conf/domain_addr.c                             |  17 +-
src/conf/domain_conf.c                             |  12 +-
src/qemu/qemu_command.c                            |  23 +--
src/qemu/qemu_domain_address.c                     |  10 +-
.../qemuxml2argv-boot-floppy-q35.args              |   5 +-
.../qemuxml2argv-boot-floppy-q35.xml               |  10 +-
.../qemuxml2argv-bootindex-floppy-q35.args         |   5 +-
.../qemuxml2argv-bootindex-floppy-q35.xml          |  10 +-
.../qemuxml2argv-intel-iommu-machine.args          |   5 +-
.../qemuxml2argv-intel-iommu-machine.xml           |  13 +-
.../qemuxml2argvdata/qemuxml2argv-intel-iommu.args |   5 +-
.../qemuxml2argvdata/qemuxml2argv-intel-iommu.xml  |  13 +-
.../qemuxml2argv-pcie-expander-bus-bad-bus.xml     |  18 +-
.../qemuxml2argv-pcie-expander-bus.args            | 136 ++++++-------
.../qemuxml2argv-pcie-expander-bus.xml             |  85 ++++----
.../qemuxml2argv-pcie-root-port-too-many.xml       |   6 +-
.../qemuxml2argv-pcie-root-port.args               |   6 +-
.../qemuxml2argv-pcie-root-port.xml                |   6 +-
.../qemuxml2argv-pcie-switch-downstream-port.args  |  22 +-
.../qemuxml2argv-pcie-switch-downstream-port.xml   |  18 +-
.../qemuxml2argv-pcie-switch-upstream-port.args    |  10 +-
.../qemuxml2argv-pcie-switch-upstream-port.xml     |  10 +-
.../qemuxml2argv-pcihole64-q35.args                |   2 -
.../qemuxml2argv-pcihole64-q35.xml                 |   2 -
.../qemuxml2argv-q35-dmi-bad-address1.xml          |   5 +-
.../qemuxml2argv-q35-dmi-bad-address2.xml          |  21 +-
.../qemuxml2argv-q35-wrong-root.xml                |   7 -
tests/qemuxml2argvdata/qemuxml2argv-q35.args       |   2 -
tests/qemuxml2argvdata/qemuxml2argv-q35.xml        |   7 -
.../qemuxml2argv-usb-controller-default-q35.args   |   2 -
.../qemuxml2argv-usb-controller-default-q35.xml    |   5 -
.../qemuxml2argv-usb-controller-explicit-q35.args  |   2 -
.../qemuxml2argv-usb-controller-explicit-q35.xml   |   5 -
.../qemuxml2argv-vcpu-placement-static.xml         |   6 -
tests/qemuxml2argvtest.c                           |  83 ++------
.../qemuxml2xmlout-intel-iommu.xml                 |  13 +-
.../qemuxml2xmlout-pcie-expander-bus.xml           | 225 ++++++++++-----------
.../qemuxml2xmlout-pcie-root-port.xml              |  15 +-
.../qemuxml2xmlout-pcie-switch-downstream-port.xml |  61 +++---
.../qemuxml2xmlout-pcie-switch-upstream-port.xml   |  25 +--
.../qemuxml2xmlout-pcihole64-q35.xml               |   9 -
tests/qemuxml2xmloutdata/qemuxml2xmlout-q35.xml    |   9 -
.../qemuxml2xmlout-usb-controller-default-q35.xml  |   9 -
.../qemuxml2xmlout-usb-controller-explicit-q35.xml |   9 -
.../qemuxml2xmlout-vcpu-placement-static.xml       |   9 -
tests/qemuxml2xmltest.c                            |  56 ++---
46 files changed, 354 insertions(+), 680 deletions(-)
[libvirt] [PATCH 0/6] qemu: PCI-related bug fixes and cleanups
Posted by Andrea Bolognani 7 years, 1 month ago
Patches 1-3 are straightforward code cleanups.

Patch 4 is a massive reduction in the size of our test suite,
achieved by removing legacy PCI controllers from test cases
that don't need them.

Patch 5 removes some incorrect assumptions that, for all
intents and purposes, made the availablilty of pci-bridge
mandatory even for guests that only used PCIe controllers
and devices.

Patch 6 reduces the test suite in size further by dropping
the capabilities that are no longer required after patch 5:
if you apply this patch before the previous one and run the
test suite, you'll get a lot of failures.


Andrea Bolognani (6):
  conf: Remove dead code
  qemu: Make switch statements more strict
  conf: Make switch statements more strict
  tests: Reduce usage of legacy PCI controllers on PCIe machines
  qemu: Allow multiple bridges when pci-bridges is not available
  tests: Reduce QEMU_CAPS_DEVICE_{DMI_TO_,}PCI_BRIDGE usage

 src/conf/domain_addr.c                             |  17 +-
 src/conf/domain_conf.c                             |  12 +-
 src/qemu/qemu_command.c                            |  23 +--
 src/qemu/qemu_domain_address.c                     |  10 +-
 .../qemuxml2argv-boot-floppy-q35.args              |   5 +-
 .../qemuxml2argv-boot-floppy-q35.xml               |  10 +-
 .../qemuxml2argv-bootindex-floppy-q35.args         |   5 +-
 .../qemuxml2argv-bootindex-floppy-q35.xml          |  10 +-
 .../qemuxml2argv-intel-iommu-machine.args          |   5 +-
 .../qemuxml2argv-intel-iommu-machine.xml           |  13 +-
 .../qemuxml2argvdata/qemuxml2argv-intel-iommu.args |   5 +-
 .../qemuxml2argvdata/qemuxml2argv-intel-iommu.xml  |  13 +-
 .../qemuxml2argv-pcie-expander-bus-bad-bus.xml     |  18 +-
 .../qemuxml2argv-pcie-expander-bus.args            | 136 ++++++-------
 .../qemuxml2argv-pcie-expander-bus.xml             |  85 ++++----
 .../qemuxml2argv-pcie-root-port-too-many.xml       |   6 +-
 .../qemuxml2argv-pcie-root-port.args               |   6 +-
 .../qemuxml2argv-pcie-root-port.xml                |   6 +-
 .../qemuxml2argv-pcie-switch-downstream-port.args  |  22 +-
 .../qemuxml2argv-pcie-switch-downstream-port.xml   |  18 +-
 .../qemuxml2argv-pcie-switch-upstream-port.args    |  10 +-
 .../qemuxml2argv-pcie-switch-upstream-port.xml     |  10 +-
 .../qemuxml2argv-pcihole64-q35.args                |   2 -
 .../qemuxml2argv-pcihole64-q35.xml                 |   2 -
 .../qemuxml2argv-q35-dmi-bad-address1.xml          |   5 +-
 .../qemuxml2argv-q35-dmi-bad-address2.xml          |  21 +-
 .../qemuxml2argv-q35-wrong-root.xml                |   7 -
 tests/qemuxml2argvdata/qemuxml2argv-q35.args       |   2 -
 tests/qemuxml2argvdata/qemuxml2argv-q35.xml        |   7 -
 .../qemuxml2argv-usb-controller-default-q35.args   |   2 -
 .../qemuxml2argv-usb-controller-default-q35.xml    |   5 -
 .../qemuxml2argv-usb-controller-explicit-q35.args  |   2 -
 .../qemuxml2argv-usb-controller-explicit-q35.xml   |   5 -
 .../qemuxml2argv-vcpu-placement-static.xml         |   6 -
 tests/qemuxml2argvtest.c                           |  83 ++------
 .../qemuxml2xmlout-intel-iommu.xml                 |  13 +-
 .../qemuxml2xmlout-pcie-expander-bus.xml           | 225 ++++++++++-----------
 .../qemuxml2xmlout-pcie-root-port.xml              |  15 +-
 .../qemuxml2xmlout-pcie-switch-downstream-port.xml |  61 +++---
 .../qemuxml2xmlout-pcie-switch-upstream-port.xml   |  25 +--
 .../qemuxml2xmlout-pcihole64-q35.xml               |   9 -
 tests/qemuxml2xmloutdata/qemuxml2xmlout-q35.xml    |   9 -
 .../qemuxml2xmlout-usb-controller-default-q35.xml  |   9 -
 .../qemuxml2xmlout-usb-controller-explicit-q35.xml |   9 -
 .../qemuxml2xmlout-vcpu-placement-static.xml       |   9 -
 tests/qemuxml2xmltest.c                            |  56 ++---
 46 files changed, 354 insertions(+), 680 deletions(-)

-- 
2.7.4

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list