1
0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2024-12-23 21:34:54 +03:00

qemu: Don't duplicate binary name in capabilities

virCapabilitiesAddGuestDomain() takes an optional binary
name: this is intended for cases where a certain domain
type can't use the default one registered for the guest
architecture, but has to use a special binary instead.

The current code, however, will pass 'binary' again when
'kvmbin' is not defined, which is unnecessary as 'binary'
has been registered as default earlier, and will result
in capabilities output such as

  <emulator>/usr/bin/qemu-system-x86_64</emulator>
  <domain type='qemu'/>
  <domain type='kvm'>
    <emulator>/usr/bin/qemu-system-x86_64</emulator>
  </domain>

with the second <emulator> element providing no additional
information.

Change it so that, when 'kvmbin' is not defined, NULL is
passed and so the default emulator will be used instead.

Signed-off-by: Andrea Bolognani <abologna@redhat.com>
This commit is contained in:
Andrea Bolognani 2018-09-14 13:10:27 +02:00
parent 99d5a516d0
commit b37b41f868
29 changed files with 29 additions and 85 deletions

View File

@ -885,7 +885,7 @@ virQEMUCapsInitGuestFromBinary(virCapsPtr caps,
if ((dom = virCapabilitiesAddGuestDomain(guest,
VIR_DOMAIN_VIRT_KVM,
kvmbin ? kvmbin : binary,
kvmbin ? kvmbin : NULL,
NULL,
nmachines,
machines)) == NULL) {

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-aarch64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-aarch64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-ppc64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-ppc64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-s390x</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-s390x</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-s390x</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-s390x</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-aarch64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-aarch64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-ppc64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-ppc64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-s390x</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-s390x</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-aarch64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-aarch64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-ppc64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-ppc64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-s390x</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-s390x</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-s390x</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-s390x</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-ppc64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-ppc64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-s390x</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-s390x</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-ppc64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-ppc64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>

View File

@ -14,9 +14,7 @@
<wordsize>64</wordsize>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
<domain type='qemu'/>
<domain type='kvm'>
<emulator>/usr/bin/qemu-system-x86_64</emulator>
</domain>
<domain type='kvm'/>
</arch>
<features>
<cpuselection/>