[libvirt] [PATCH] qemu: process: comment on min_guarantee validation

Cole Robinson crobinso at redhat.com
Mon Apr 18 23:13:08 UTC 2016


Explain why we check it at process startup time, and not parse time
where most other XML validation checks are performed
---
 src/qemu/qemu_process.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index c087300..628b4b6 100644
--- a/src/qemu/qemu_process.c
+++ b/src/qemu/qemu_process.c
@@ -4550,6 +4550,8 @@ qemuProcessStartValidate(virQEMUDriverPtr driver,
         virDomainDefCheckDuplicateDiskInfo(vm->def) < 0)
         return -1;
 
+    /* Previously we silently accepted this parameter; we can't reject
+       it at parse time without breaking those configs, so check it here */
     if (vm->def->mem.min_guarantee) {
         virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
                        _("Parameter 'min_guarantee' "
-- 
2.7.3




More information about the libvir-list mailing list