]> xenbits.xensource.com Git - libvirt.git/commitdiff
vmx: Relax virtualHW.version check
authorMatthias Bolte <matthias.bolte@googlemail.com>
Sat, 10 May 2014 14:36:56 +0000 (16:36 +0200)
committerMatthias Bolte <matthias.bolte@googlemail.com>
Sat, 7 Jun 2014 19:37:51 +0000 (21:37 +0200)
The original implementation of the VMX config parser assumed that the
virtualHW.version would have more influence on the content of the VMX
file than it actually seems to have. It started with accepting only
version 4. Additonal versions were added later without any additional
changes in the parser itself. This suggests that the influence of the
virtualHW.version on the content and format of the VMX file is small
or non-existent.

The parser worked without any changes across several virtualHW and
vSphere versions. So instead of adding new virtualHW.version values to
the parser as they come along, or adding an extra flag to allow unknown
virtualHW.version values just relax the check to require version 4 or
later.

src/vmx/vmx.c

index 9b576f7e99facee81aa2c6380a99f6dec1b03307..30ee384f42b7c6a750f0d584fb44004737d7903c 100644 (file)
@@ -1316,12 +1316,10 @@ virVMXParseConfig(virVMXContext *ctx,
         goto cleanup;
     }
 
-    if (virtualHW_version != 4 && virtualHW_version != 7 &&
-        virtualHW_version != 8 && virtualHW_version != 9 &&
-        virtualHW_version != 10) {
+    if (virtualHW_version < 4) {
         virReportError(VIR_ERR_INTERNAL_ERROR,
                        _("Expecting VMX entry 'virtualHW.version' to be "
-                         "4, 7, 8, 9 or 10 but found %lld"),
+                         "4 or higher but found %lld"),
                        virtualHW_version);
         goto cleanup;
     }