From 0177d1107426587991e8698328c14cf72c157e3e Mon Sep 17 00:00:00 2001 From: "Daniel P. Berrange" Date: Tue, 30 Apr 2013 15:01:22 +0100 Subject: [PATCH] Simplify the Xen domain create driver method Unconditionally call xenDaemonCreateXML in the xenUnifiedDomainCreateXML driver, since the XenD driver is always present. Signed-off-by: Daniel P. Berrange --- src/xen/xen_driver.c | 6 ++---- src/xen/xend_internal.c | 4 +--- src/xen/xend_internal.h | 3 +-- 3 files changed, 4 insertions(+), 9 deletions(-) diff --git a/src/xen/xen_driver.c b/src/xen/xen_driver.c index 63295357b6..ec5ef0703e 100644 --- a/src/xen/xen_driver.c +++ b/src/xen/xen_driver.c @@ -596,11 +596,9 @@ static virDomainPtr xenUnifiedDomainCreateXML(virConnectPtr conn, const char *xmlDesc, unsigned int flags) { - xenUnifiedPrivatePtr priv = conn->privateData; + virCheckFlags(0, NULL); - if (priv->opened[XEN_UNIFIED_XEND_OFFSET]) - return xenDaemonCreateXML(conn, xmlDesc, flags); - return NULL; + return xenDaemonCreateXML(conn, xmlDesc); } /* Assumption made in underlying drivers: diff --git a/src/xen/xend_internal.c b/src/xen/xend_internal.c index 6cd5469ffc..8981065c96 100644 --- a/src/xen/xend_internal.c +++ b/src/xen/xend_internal.c @@ -2288,7 +2288,7 @@ xenDaemonLookupByUUID(virConnectPtr conn, const unsigned char *uuid) * Returns a new domain object or NULL in case of failure */ virDomainPtr -xenDaemonCreateXML(virConnectPtr conn, const char *xmlDesc, unsigned int flags) +xenDaemonCreateXML(virConnectPtr conn, const char *xmlDesc) { int ret; char *sexpr; @@ -2296,8 +2296,6 @@ xenDaemonCreateXML(virConnectPtr conn, const char *xmlDesc, unsigned int flags) xenUnifiedPrivatePtr priv = conn->privateData; virDomainDefPtr def; - virCheckFlags(0, NULL); - if (!(def = virDomainDefParseString(xmlDesc, priv->caps, priv->xmlopt, 1 << VIR_DOMAIN_VIRT_XEN, VIR_DOMAIN_XML_INACTIVE))) diff --git a/src/xen/xend_internal.h b/src/xen/xend_internal.h index f6760a2f1c..5f82f045eb 100644 --- a/src/xen/xend_internal.h +++ b/src/xen/xend_internal.h @@ -152,8 +152,7 @@ int xenDaemonDomainSetAutostart (virDomainPtr domain, extern struct xenUnifiedDriver xenDaemonDriver; int xenDaemonInit (void); -virDomainPtr xenDaemonCreateXML(virConnectPtr conn, const char *xmlDesc, - unsigned int flags); +virDomainPtr xenDaemonCreateXML(virConnectPtr conn, const char *xmlDesc); virDomainPtr xenDaemonLookupByID(virConnectPtr conn, int id); virDomainPtr xenDaemonLookupByUUID(virConnectPtr conn, const unsigned char *uuid); virDomainPtr xenDaemonLookupByName(virConnectPtr conn, const char *domname); -- GitLab