diff options
author | Cole Robinson <crobinso@redhat.com> | 2016-06-17 17:17:49 -0400 |
---|---|---|
committer | Cole Robinson <crobinso@redhat.com> | 2016-06-17 17:17:49 -0400 |
commit | 40e035463193e676e875261d9d85cb7885fc8efc (patch) | |
tree | 82d92c8a746e8d1ae4b10cb6af7be757a27bdb4b | |
parent | 118bf85cbaa8a376707873d8385bb86d315ea22d (diff) | |
download | virt-manager-40e035463193e676e875261d9d85cb7885fc8efc.tar.gz |
guest: Fix multiple address.type=pci auto allocate requests
And test it for aarch64, which is the only place it's really
interesting at the moment
-rw-r--r-- | tests/cli-test-xml/compare/virt-install-aarch64-kvm-gic.xml | 13 | ||||
-rw-r--r-- | tests/clitest.py | 2 | ||||
-rw-r--r-- | virtinst/guest.py | 2 |
3 files changed, 14 insertions, 3 deletions
diff --git a/tests/cli-test-xml/compare/virt-install-aarch64-kvm-gic.xml b/tests/cli-test-xml/compare/virt-install-aarch64-kvm-gic.xml index 189373db..5bf41449 100644 --- a/tests/cli-test-xml/compare/virt-install-aarch64-kvm-gic.xml +++ b/tests/cli-test-xml/compare/virt-install-aarch64-kvm-gic.xml @@ -19,10 +19,19 @@ <on_crash>restart</on_crash> <devices> <emulator>/usr/bin/qemu-system-aarch64</emulator> - <interface type="bridge"> - <source bridge="eth0"/> + <disk type="file" device="disk"> + <driver name="qemu" type="qcow2"/> + <source file="/var/lib/libvirt/images/foobar.qcow2"/> + <target dev="sda" bus="scsi"/> + </disk> + <controller type="scsi" index="0" model="virtio-scsi"> + <address type="pci"/> + </controller> + <interface type="network"> + <source network="default"/> <mac address="00:11:22:33:44:55"/> <model type="virtio"/> + <address type="pci"/> </interface> <console type="pty"/> <channel type="unix"> diff --git a/tests/clitest.py b/tests/clitest.py index 2625142c..b63a92a7 100644 --- a/tests/clitest.py +++ b/tests/clitest.py @@ -714,7 +714,7 @@ c.add_compare("--arch aarch64 --machine virt --boot kernel=/f19-arm.kernel,initr c.add_compare("--arch aarch64 --boot kernel=/f19-arm.kernel,initrd=/f19-arm.initrd,kernel_args=\"console=ttyAMA0,1234 rw root=/dev/vda3\" --disk %(EXISTIMG1)s", "aarch64-machdefault") c.add_compare("--arch aarch64 --cdrom %(EXISTIMG2)s --boot loader=CODE.fd,nvram_template=VARS.fd --disk %(EXISTIMG1)s --cpu none --events on_crash=preserve,on_reboot=destroy,on_poweroff=restart", "aarch64-cdrom") c.add_compare("--connect %(URI-KVM-AARCH64)s --disk %(EXISTIMG1)s --import --os-variant fedora21", "aarch64-kvm-import") -c.add_compare("--connect %(URI-KVM-AARCH64)s --disk none --os-variant fedora22 --features gic_version=host", "aarch64-kvm-gic") +c.add_compare("--connect %(URI-KVM-AARCH64)s --disk size=1 --os-variant fedora22 --features gic_version=host --network network=default,address.type=pci --controller type=scsi,model=virtio-scsi,address.type=pci", "aarch64-kvm-gic") # ppc64 tests c.add_compare("--arch ppc64 --machine pseries --boot network --disk %(EXISTIMG1)s --disk device=cdrom --os-variant fedora20 --network none", "ppc64-pseries-f20") diff --git a/virtinst/guest.py b/virtinst/guest.py index 8e7aef7e..6a42536e 100644 --- a/virtinst/guest.py +++ b/virtinst/guest.py @@ -914,6 +914,8 @@ class Guest(XMLBuilder): for d in self.get_all_devices(): if d.address.type != d.address.ADDRESS_TYPE_PCI: continue + if None in [d.address.domain, d.address.bus, d.address.slot]: + continue addr = d.address addrstr = "%d%d%d" % (d.address.domain, |