[libvirt] [PATCHv2 2/5] Revert "Error out on missing machine type in machine configs"

Ján Tomko jtomko at redhat.com
Thu Feb 25 15:40:54 UTC 2016


Revert commit 55e6d8cd9eac7eb2aaa4d221585e9402cf7269d5.

This fix for https://bugzilla.redhat.com/show_bug.cgi?id=1267256
unconditionally required a machine type for all machine types
even though qemu is the only emulator using them.

Revert it to fix persistent configs for drivers with no machine type:
https://www.redhat.com/archives/libvir-list/2016-February/msg01228.html
---
 src/conf/domain_conf.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 3b15cb4..79758d4 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -14851,12 +14851,6 @@ virDomainDefParseXML(xmlDocPtr xml,
             goto error;
         }
         VIR_FREE(capsdata);
-    } else {
-        if (!def->os.machine) {
-            virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
-                           _("Missing machine type"));
-            goto error;
-        }
     }
 
     /* Extract domain name */
-- 
2.4.10




More information about the libvir-list mailing list