[libvirt] [PATCH 4/6] qemu: Move check that validates 'min_guarantee' to qemuDomainDefValidate
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