]> xenbits.xensource.com Git - libvirt.git/commitdiff
Revert "qemu: Pass migratable host model to virCPUUpdate"
authorJiri Denemark <jdenemar@redhat.com>
Fri, 7 Apr 2017 11:17:57 +0000 (13:17 +0200)
committerJiri Denemark <jdenemar@redhat.com>
Fri, 7 Apr 2017 11:19:55 +0000 (13:19 +0200)
This reverts commit 959e72d323d06d67a5c3a759869af6da49da0e0e which was
pushed accidentally.

src/qemu/qemu_capabilities.c
src/qemu/qemu_capabilities.h
src/qemu/qemu_command.c
src/qemu/qemu_process.c
tests/cputest.c

index d2eef432186598f216ee5183abc729f9ef4b6644..964299b01aca42cc8d4746b16950719fc7a87111 100644 (file)
@@ -2446,20 +2446,12 @@ virQEMUCapsGetCPUDefinitions(virQEMUCapsPtr qemuCaps,
 
 virCPUDefPtr
 virQEMUCapsGetHostModel(virQEMUCapsPtr qemuCaps,
-                        virDomainVirtType type,
-                        bool migratable)
+                        virDomainVirtType type)
 {
-    virQEMUCapsCPUModelPtr model;
-
     if (type == VIR_DOMAIN_VIRT_KVM)
-        model = &qemuCaps->hostCPU.kvm;
-    else
-        model = &qemuCaps->hostCPU.tcg;
-
-    if (migratable)
-        return model->migratable;
+        return qemuCaps->hostCPU.kvm.full;
     else
-        return model->full;
+        return qemuCaps->hostCPU.tcg.full;
 }
 
 
@@ -2477,7 +2469,7 @@ virQEMUCapsIsCPUModeSupported(virQEMUCapsPtr qemuCaps,
                virQEMUCapsGuestIsNative(caps->host.arch, qemuCaps->arch);
 
     case VIR_CPU_MODE_HOST_MODEL:
-        return !!virQEMUCapsGetHostModel(qemuCaps, type, false);
+        return !!virQEMUCapsGetHostModel(qemuCaps, type);
 
     case VIR_CPU_MODE_CUSTOM:
         if (type == VIR_DOMAIN_VIRT_KVM)
@@ -5542,8 +5534,7 @@ virQEMUCapsFillDomainCPUCaps(virCapsPtr caps,
 
     if (virQEMUCapsIsCPUModeSupported(qemuCaps, caps, domCaps->virttype,
                                       VIR_CPU_MODE_HOST_MODEL)) {
-        virCPUDefPtr cpu = virQEMUCapsGetHostModel(qemuCaps, domCaps->virttype,
-                                                   false);
+        virCPUDefPtr cpu = virQEMUCapsGetHostModel(qemuCaps, domCaps->virttype);
         domCaps->cpu.hostModel = virCPUDefCopy(cpu);
     }
 
index 9959c71a6f07845f56bee180f67e1c539f0975b8..cca9a12b5cd777f546ec7ec8dddff8bdaa19e78d 100644 (file)
@@ -450,8 +450,7 @@ int virQEMUCapsGetCPUDefinitions(virQEMUCapsPtr qemuCaps,
                                  char ***names,
                                  size_t *count);
 virCPUDefPtr virQEMUCapsGetHostModel(virQEMUCapsPtr qemuCaps,
-                                     virDomainVirtType type,
-                                     bool migratable);
+                                     virDomainVirtType type);
 bool virQEMUCapsIsCPUModeSupported(virQEMUCapsPtr qemuCaps,
                                    virCapsPtr caps,
                                    virDomainVirtType type,
index 7049be20a15564e0d8d5c410228be0ce4d71cfd7..445c696d6e7072250921b58b0c11a56278ebc98c 100644 (file)
@@ -6887,7 +6887,7 @@ qemuBuildCpuCommandLine(virCommandPtr cmd,
             if (def->cpu->mode == VIR_CPU_MODE_CUSTOM)
                 cpuDef = def->cpu;
             else if (def->cpu->mode == VIR_CPU_MODE_HOST_PASSTHROUGH)
-                cpuDef = virQEMUCapsGetHostModel(qemuCaps, def->virtType, false);
+                cpuDef = virQEMUCapsGetHostModel(qemuCaps, def->virtType);
 
             if (cpuDef) {
                 int svm = virCPUCheckFeature(def->os.arch, cpuDef, "svm");
index b4499beafe624eeea7fba8d32061c072e54bdff4..8323a1831667f3961ca2514421c26895672bcde0 100644 (file)
@@ -5304,12 +5304,12 @@ qemuProcessUpdateGuestCPU(virDomainDefPtr def,
 
     if (def->cpu->check == VIR_CPU_CHECK_PARTIAL &&
         virCPUCompare(caps->host.arch,
-                      virQEMUCapsGetHostModel(qemuCaps, def->virtType, false),
+                      virQEMUCapsGetHostModel(qemuCaps, def->virtType),
                       def->cpu, true) < 0)
         return -1;
 
     if (virCPUUpdate(def->os.arch, def->cpu,
-                     virQEMUCapsGetHostModel(qemuCaps, def->virtType, true)) < 0)
+                     virQEMUCapsGetHostModel(qemuCaps, def->virtType)) < 0)
         goto cleanup;
 
     if (virQEMUCapsGetCPUDefinitions(qemuCaps, def->virtType,
index d5e023c404dc04effa843903e131a09d34c5a74b..5280307545ee7c2605d6a9f2c6de2f3e720f09c3 100644 (file)
@@ -393,7 +393,6 @@ cpuTestUpdate(const void *arg)
     const struct data *data = arg;
     int ret = -1;
     virCPUDefPtr host = NULL;
-    virCPUDefPtr migHost = NULL;
     virCPUDefPtr cpu = NULL;
     char *result = NULL;
 
@@ -401,10 +400,7 @@ cpuTestUpdate(const void *arg)
         !(cpu = cpuTestLoadXML(data->arch, data->name)))
         goto cleanup;
 
-    if (!(migHost = virCPUCopyMigratable(data->arch, host)))
-        goto cleanup;
-
-    if (virCPUUpdate(host->arch, cpu, migHost) < 0)
+    if (virCPUUpdate(host->arch, cpu, host) < 0)
         goto cleanup;
 
     if (virAsprintf(&result, "%s+%s", data->host, data->name) < 0)
@@ -415,7 +411,6 @@ cpuTestUpdate(const void *arg)
  cleanup:
     virCPUDefFree(host);
     virCPUDefFree(cpu);
-    virCPUDefFree(migHost);
     VIR_FREE(result);
     return ret;
 }