[libvirt] [PATCH 4/6] qemu: Move check that validates 'min_guarantee' to qemuDomainDefValidate

Peter Krempa pkrempa at redhat.com
Tue May 17 14:25:40 UTC 2016


Now that we have a good place for this check let's move it there.
---
 src/qemu/qemu_domain.c  | 6 ++++++
 src/qemu/qemu_process.c | 6 ------
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index 5ccb483..f172d6f 100644
--- a/src/qemu/qemu_domain.c
+++ b/src/qemu/qemu_domain.c
@@ -5362,5 +5362,11 @@ qemuDomainDefValidate(const virDomainDef *def)
     if (virDomainDefValidate(def) < 0)
         return -1;

+    if (def->mem.min_guarantee) {
+        virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
+                       _("Parameter 'min_guarantee' not supported by QEMU."));
+        return -1;
+    }
+
     return 0;
 }
diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index d079b18..ae435e4 100644
--- a/src/qemu/qemu_process.c
+++ b/src/qemu/qemu_process.c
@@ -4620,12 +4620,6 @@ qemuProcessStartValidateXML(virDomainObjPtr vm,
      * If back compat isn't a concern, XML validation should probably
      * be done at parse time.
      */
-    if (vm->def->mem.min_guarantee) {
-        virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
-                       _("Parameter 'min_guarantee' "
-                         "not supported by QEMU."));
-        return -1;
-    }

     /* checks below should not be executed when starting a qemu process for a
      * VM that was running before (migration, snapshots, save). It's more
-- 
2.8.2




More information about the libvir-list mailing list