qemu: Expect a single binary in virQEMUCapsInitGuest()

We're only ever passing a single binary when calling this
function, so we can remove all code dealing with the
possibility of a second binary being specified.

Signed-off-by: Andrea Bolognani <abologna@redhat.com>

diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
index a911ac41c27d57a6bf29fb76b925bd5c2ba77513..161eede4161ee7fa398473cb9ed15bab5a5a429f 100644 (file)
--- a/src/qemu/qemu_capabilities.c
+++ b/src/qemu/qemu_capabilities.c
@@ -789,7 +789,6 @@ virQEMUCapsInitGuest(virCapsPtr caps,
 
     ret = virQEMUCapsInitGuestFromBinary(caps,
                                          binary, qemubinCaps,
-                                         NULL, NULL,
                                          guestarch);
 
     VIR_FREE(binary);
@@ -802,8 +801,6 @@ int
 virQEMUCapsInitGuestFromBinary(virCapsPtr caps,
                                const char *binary,
                                virQEMUCapsPtr qemubinCaps,
-                               const char *kvmbin,
-                               virQEMUCapsPtr kvmbinCaps,
                                virArch guestarch)
 {
     virCapsGuestPtr guest;
@@ -816,8 +813,7 @@ virQEMUCapsInitGuestFromBinary(virCapsPtr caps,
     if (!binary)
         return 0;
 
-    if (virQEMUCapsGet(qemubinCaps, QEMU_CAPS_KVM) ||
-        kvmbin)
+    if (virQEMUCapsGet(qemubinCaps, QEMU_CAPS_KVM))
         haskvm = true;
 
     if (virQEMUCapsGetMachineTypesCaps(qemubinCaps, &nmachines, &machines) < 0)
@@ -863,21 +859,14 @@ virQEMUCapsInitGuestFromBinary(virCapsPtr caps,
     if (haskvm) {
         virCapsGuestDomainPtr dom;
 
-        if (kvmbin &&
-            virQEMUCapsGetMachineTypesCaps(kvmbinCaps, &nmachines, &machines) < 0)
-            goto cleanup;
-
         if ((dom = virCapabilitiesAddGuestDomain(guest,
                                                  VIR_DOMAIN_VIRT_KVM,
-                                                 kvmbin ? kvmbin : NULL,
                                                  NULL,
-                                                 nmachines,
-                                                 machines)) == NULL) {
+                                                 NULL,
+                                                 0,
+                                                 NULL)) == NULL) {
             goto cleanup;
         }
-
-        machines = NULL;
-        nmachines = 0;
     }
 
     if ((ARCH_IS_X86(guestarch) || guestarch == VIR_ARCH_AARCH64) &&

diff --git a/src/qemu/qemu_capabilities.h b/src/qemu/qemu_capabilities.h
index e671f74ebb170152ec79721b4c6512b76926e395..94b3bbb2f6557fb37ba8d91562e78407945369ae 100644 (file)
--- a/src/qemu/qemu_capabilities.h
+++ b/src/qemu/qemu_capabilities.h
@@ -612,8 +612,6 @@ const char *virQEMUCapsGetPreferredMachine(virQEMUCapsPtr qemuCaps);
 int virQEMUCapsInitGuestFromBinary(virCapsPtr caps,
                                    const char *binary,
                                    virQEMUCapsPtr qemubinCaps,
-                                   const char *kvmbin,
-                                   virQEMUCapsPtr kvmbinCaps,
                                    virArch guestarch);
 
 int virQEMUCapsFillDomainCaps(virCapsPtr caps,

diff --git a/tests/qemucaps2xmltest.c b/tests/qemucaps2xmltest.c
index e765a03b73ec41524aa1f9f82f68b69988481afe..883909a9731c74e41e0c8052c40896099237c9b1 100644 (file)
--- a/tests/qemucaps2xmltest.c
+++ b/tests/qemucaps2xmltest.c
@@ -106,8 +106,6 @@ testGetCaps(char *capsData, const testQemuData *data)
     if (virQEMUCapsInitGuestFromBinary(caps,
                                        binary,
                                        qemuCaps,
-                                       NULL,
-                                       NULL,
                                        arch) < 0) {
         fprintf(stderr, "failed to create the capabilities from qemu");
         goto error;