[libvirt] [PATCH 4/6] conf: Don't validate vcpu count in XML parser

Peter Krempa pkrempa at redhat.com
Thu Aug 25 22:42:48 UTC 2016


Validating the vcpu count is more intricate and doing it in the XML
parser will make previously valid configs (with older qemus) vanish.

Now that we have the validation callbacks we can do it in a more
appropriate place.

This basically reverts commit b54de0830a.

Partially resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1370066
---
 src/conf/domain_conf.c | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 17c2f53..ae3eb14 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -16376,15 +16376,6 @@ virDomainDefParseXML(xmlDocPtr xml,

         if (def->cpu == NULL)
             goto error;
-
-        if (def->cpu->sockets &&
-            virDomainDefGetVcpusMax(def) >
-            def->cpu->sockets * def->cpu->cores * def->cpu->threads) {
-            virReportError(VIR_ERR_XML_DETAIL, "%s",
-                           _("Maximum CPUs greater than topology limit"));
-            goto error;
-        }
-
     }

     if (virDomainNumaDefCPUParseXML(def->numa, ctxt) < 0)
-- 
2.8.2




More information about the libvir-list mailing list