]> xenbits.xensource.com Git - libvirt.git/commit
qemu: update tests to not assume dmi-to-pci-bridge is always added
authorLaine Stump <laine@laine.org>
Fri, 11 Nov 2016 17:14:02 +0000 (12:14 -0500)
committerLaine Stump <laine@laine.org>
Mon, 14 Nov 2016 19:21:15 +0000 (14:21 -0500)
commit815b51d97ab6e9512b2cf1819d57e3cc88c413e1
treecced11b6c58862f47da0fb7a9860418651645f77
parent0702f48ef485d6b893a1633886a1b92851a2101f
qemu: update tests to not assume dmi-to-pci-bridge is always added

A few of the qemu test cases assume that a dmi-to-pci-bridge will
always be added at index 1, and so they omit it from the input data
even though a pci-bridge is present at index 2, e.g.:

   <controller type='pci' index='0' model='pcie-root'/>
   <controller type='pci' index='2' model='pci-bridge'/>

Support for this odd practice was discussed on libvir-list and we
decided that the complex code required to make this continue was not
worth the headache of maintaining. So instead, this patch modifies the
test cases to manually add a dmi-to-pci-bridge at index 1 (since an
upcoming patch is going to eliminate the unconditional adding of
dmi-to-pci-bridge).

Because the auto-add was placing the dmi-to-pci-bridge later in the
list (even though it has a lower index) the test output is also
updated to take account for the new order (which puts the pci
controllers in index-order)
tests/qemuxml2argvdata/qemuxml2argv-usb-controller-default-q35.args
tests/qemuxml2argvdata/qemuxml2argv-usb-controller-default-q35.xml
tests/qemuxml2argvdata/qemuxml2argv-usb-controller-explicit-q35.args
tests/qemuxml2argvdata/qemuxml2argv-usb-controller-explicit-q35.xml
tests/qemuxml2xmloutdata/qemuxml2xmlout-usb-controller-default-q35.xml
tests/qemuxml2xmloutdata/qemuxml2xmlout-usb-controller-explicit-q35.xml