[libvirt] [PATCH 1/5] Rename internal DumpXML functions to GetXMLDesc

Matthias Bolte matthias.bolte at googlemail.com
Sat May 7 12:28:36 UTC 2011


This matches the public API and helps to get rid of some speical case
code in the remote generator.

Rename driver API functions and XDR protocol structs.

No functional change included outside of the remote generator.
---
 daemon/remote_generator.pl           |    6 +---
 src/driver.h                         |   20 +++++++++---------
 src/esx/esx_device_monitor.c         |    2 +-
 src/esx/esx_driver.c                 |   10 ++++----
 src/esx/esx_network_driver.c         |    2 +-
 src/esx/esx_storage_driver.c         |    4 +-
 src/libvirt.c                        |   24 +++++++++++-----------
 src/libxl/libxl_driver.c             |    6 ++--
 src/lxc/lxc_driver.c                 |    8 +++---
 src/network/bridge_driver.c          |    4 +-
 src/node_device/node_device_driver.c |    6 ++--
 src/nwfilter/nwfilter_driver.c       |    6 ++--
 src/openvz/openvz_driver.c           |    6 ++--
 src/phyp/phyp_driver.c               |    6 ++--
 src/qemu/qemu_command.c              |    2 +-
 src/qemu/qemu_driver.c               |   12 +++++-----
 src/remote/remote_driver.c           |   12 +++++-----
 src/remote/remote_protocol.x         |   36 +++++++++++++++++-----------------
 src/remote_protocol-structs          |   24 +++++++++++-----------
 src/storage/storage_driver.c         |    6 ++--
 src/test/test_driver.c               |   22 ++++++++++----------
 src/uml/uml_conf.c                   |    2 +-
 src/uml/uml_driver.c                 |    8 +++---
 src/vbox/vbox_tmpl.c                 |   14 ++++++------
 src/vmware/vmware_driver.c           |    6 ++--
 src/xen/xen_driver.c                 |   12 +++++-----
 src/xen/xend_internal.c              |    4 +-
 src/xen/xend_internal.h              |    2 +-
 src/xen/xm_internal.c                |    2 +-
 src/xen/xm_internal.h                |    2 +-
 src/xenapi/xenapi_driver.c           |    8 +++---
 31 files changed, 141 insertions(+), 143 deletions(-)

diff --git a/daemon/remote_generator.pl b/daemon/remote_generator.pl
index 062ccc1..b06e51a 100755
--- a/daemon/remote_generator.pl
+++ b/daemon/remote_generator.pl
@@ -645,8 +645,6 @@ elsif ($opt_b) {
                 $prefix = "Connect"
             } elsif ($calls{$_}->{ProcName} eq "SupportsFeature") {
                 $prefix = "Drv"
-            } elsif ($calls{$_}->{ProcName} =~ m/^(\S+)DumpXML$/) {
-                $proc_name = "${1}GetXMLDesc"
             } elsif ($calls{$_}->{ProcName} eq "CPUBaseline") {
                 $proc_name = "ConnectBaselineCPU"
             } elsif ($calls{$_}->{ProcName} eq "CPUCompare") {
@@ -892,8 +890,8 @@ elsif ($opt_k) {
                         # fix bad decisions in the xdr protocol
                         if ($arg_name eq "flags" and
                             $call->{ProcName} ne "DomainCoreDump" and
-                            $call->{ProcName} ne "DomainDumpXML" and
-                            $call->{ProcName} ne "NetworkDumpXML") {
+                            $call->{ProcName} ne "DomainGetXMLDesc" and
+                            $call->{ProcName} ne "NetworkGetXMLDesc") {
                             $type_name = "unsigned int";
                         } elsif ($arg_name eq "nvcpus" and
                                  $call->{ProcName} eq "DomainSetVcpus") {
diff --git a/src/driver.h b/src/driver.h
index a8b79e6..6374ec7 100644
--- a/src/driver.h
+++ b/src/driver.h
@@ -176,7 +176,7 @@ typedef int
                                          const char *to,
                                          int flags);
 typedef char *
-        (*virDrvDomainDumpXML)		(virDomainPtr dom,
+        (*virDrvDomainGetXMLDesc)		(virDomainPtr dom,
                                          int flags);
 typedef char *
         (*virDrvConnectDomainXMLFromNative) (virConnectPtr conn,
@@ -474,8 +474,8 @@ typedef virDomainSnapshotPtr
                                      unsigned int flags);
 
 typedef char *
-    (*virDrvDomainSnapshotDumpXML)(virDomainSnapshotPtr snapshot,
-                                   unsigned int flags);
+    (*virDrvDomainSnapshotGetXMLDesc)(virDomainSnapshotPtr snapshot,
+                                      unsigned int flags);
 
 typedef int
     (*virDrvDomainSnapshotNum)(virDomainPtr domain, unsigned int flags);
@@ -574,7 +574,7 @@ struct _virDriver {
     virDrvDomainGetMaxVcpus		domainGetMaxVcpus;
     virDrvDomainGetSecurityLabel     domainGetSecurityLabel;
     virDrvNodeGetSecurityModel  nodeGetSecurityModel;
-    virDrvDomainDumpXML		domainDumpXML;
+    virDrvDomainGetXMLDesc		domainGetXMLDesc;
     virDrvConnectDomainXMLFromNative domainXMLFromNative;
     virDrvConnectDomainXMLToNative domainXMLToNative;
     virDrvListDefinedDomains	listDefinedDomains;
@@ -629,7 +629,7 @@ struct _virDriver {
     virDrvDomainHasManagedSaveImage domainHasManagedSaveImage;
     virDrvDomainManagedSaveRemove domainManagedSaveRemove;
     virDrvDomainSnapshotCreateXML domainSnapshotCreateXML;
-    virDrvDomainSnapshotDumpXML domainSnapshotDumpXML;
+    virDrvDomainSnapshotGetXMLDesc domainSnapshotGetXMLDesc;
     virDrvDomainSnapshotNum domainSnapshotNum;
     virDrvDomainSnapshotListNames domainSnapshotListNames;
     virDrvDomainSnapshotLookupByName domainSnapshotLookupByName;
@@ -672,7 +672,7 @@ typedef int
 typedef int
         (*virDrvNetworkDestroy)		(virNetworkPtr network);
 typedef char *
-        (*virDrvNetworkDumpXML)		(virNetworkPtr network,
+        (*virDrvNetworkGetXMLDesc)		(virNetworkPtr network,
                                          int flags);
 typedef char *
         (*virDrvNetworkGetBridgeName)	(virNetworkPtr network);
@@ -718,7 +718,7 @@ struct _virNetworkDriver {
         virDrvNetworkUndefine		networkUndefine;
         virDrvNetworkCreate		networkCreate;
         virDrvNetworkDestroy		networkDestroy;
-        virDrvNetworkDumpXML		networkDumpXML;
+        virDrvNetworkGetXMLDesc		networkGetXMLDesc;
         virDrvNetworkGetBridgeName	networkGetBridgeName;
         virDrvNetworkGetAutostart	networkGetAutostart;
         virDrvNetworkSetAutostart	networkSetAutostart;
@@ -1017,8 +1017,8 @@ typedef int (*virDevMonListDevices)(virConnectPtr conn,
 typedef virNodeDevicePtr (*virDevMonDeviceLookupByName)(virConnectPtr conn,
                                                         const char *name);
 
-typedef char * (*virDevMonDeviceDumpXML)(virNodeDevicePtr dev,
-                                         unsigned int flags);
+typedef char * (*virDevMonDeviceGetXMLDesc)(virNodeDevicePtr dev,
+                                            unsigned int flags);
 
 typedef char * (*virDevMonDeviceGetParent)(virNodeDevicePtr dev);
 
@@ -1047,7 +1047,7 @@ struct _virDeviceMonitor {
     virDevMonNumOfDevices numOfDevices;
     virDevMonListDevices listDevices;
     virDevMonDeviceLookupByName deviceLookupByName;
-    virDevMonDeviceDumpXML deviceDumpXML;
+    virDevMonDeviceGetXMLDesc deviceGetXMLDesc;
     virDevMonDeviceGetParent deviceGetParent;
     virDevMonDeviceNumOfCaps deviceNumOfCaps;
     virDevMonDeviceListCaps deviceListCaps;
diff --git a/src/esx/esx_device_monitor.c b/src/esx/esx_device_monitor.c
index d559f96..3fa63d5 100644
--- a/src/esx/esx_device_monitor.c
+++ b/src/esx/esx_device_monitor.c
@@ -72,7 +72,7 @@ static virDeviceMonitor esxDeviceMonitor = {
     NULL,                                  /* numOfDevices */
     NULL,                                  /* listDevices */
     NULL,                                  /* deviceLookupByName */
-    NULL,                                  /* deviceDumpXML */
+    NULL,                                  /* deviceGetXMLDesc */
     NULL,                                  /* deviceGetParent */
     NULL,                                  /* deviceNumOfCaps */
     NULL,                                  /* deviceListCaps */
diff --git a/src/esx/esx_driver.c b/src/esx/esx_driver.c
index 7933f11..bacacb8 100644
--- a/src/esx/esx_driver.c
+++ b/src/esx/esx_driver.c
@@ -2610,7 +2610,7 @@ esxDomainGetMaxVcpus(virDomainPtr domain)
 
 
 static char *
-esxDomainDumpXML(virDomainPtr domain, int flags)
+esxDomainGetXMLDesc(virDomainPtr domain, int flags)
 {
     esxPrivate *priv = domain->conn->privateData;
     esxVI_String *propertyNameList = NULL;
@@ -4137,8 +4137,8 @@ esxDomainSnapshotCreateXML(virDomainPtr domain, const char *xmlDesc,
 
 
 static char *
-esxDomainSnapshotDumpXML(virDomainSnapshotPtr snapshot,
-                         unsigned int flags)
+esxDomainSnapshotGetXMLDesc(virDomainSnapshotPtr snapshot,
+                            unsigned int flags)
 {
     esxPrivate *priv = snapshot->domain->conn->privateData;
     esxVI_VirtualMachineSnapshotTree *rootSnapshotList = NULL;
@@ -4634,7 +4634,7 @@ static virDriver esxDriver = {
     esxDomainGetMaxVcpus,            /* domainGetMaxVcpus */
     NULL,                            /* domainGetSecurityLabel */
     NULL,                            /* nodeGetSecurityModel */
-    esxDomainDumpXML,                /* domainDumpXML */
+    esxDomainGetXMLDesc,             /* domainGetXMLDesc */
     esxDomainXMLFromNative,          /* domainXMLFromNative */
     esxDomainXMLToNative,            /* domainXMLToNative */
     esxListDefinedDomains,           /* listDefinedDomains */
@@ -4689,7 +4689,7 @@ static virDriver esxDriver = {
     NULL,                            /* domainHasManagedSaveImage */
     NULL,                            /* domainManagedSaveRemove */
     esxDomainSnapshotCreateXML,      /* domainSnapshotCreateXML */
-    esxDomainSnapshotDumpXML,        /* domainSnapshotDumpXML */
+    esxDomainSnapshotGetXMLDesc,     /* domainSnapshotGetXMLDesc */
     esxDomainSnapshotNum,            /* domainSnapshotNum */
     esxDomainSnapshotListNames,      /* domainSnapshotListNames */
     esxDomainSnapshotLookupByName,   /* domainSnapshotLookupByName */
diff --git a/src/esx/esx_network_driver.c b/src/esx/esx_network_driver.c
index a64bb8e..bbc2541 100644
--- a/src/esx/esx_network_driver.c
+++ b/src/esx/esx_network_driver.c
@@ -80,7 +80,7 @@ static virNetworkDriver esxNetworkDriver = {
     NULL,                                  /* networkUndefine */
     NULL,                                  /* networkCreate */
     NULL,                                  /* networkDestroy */
-    NULL,                                  /* networkDumpXML */
+    NULL,                                  /* networkGetXMLDesc */
     NULL,                                  /* networkGetBridgeName */
     NULL,                                  /* networkGetAutostart */
     NULL,                                  /* networkSetAutostart */
diff --git a/src/esx/esx_storage_driver.c b/src/esx/esx_storage_driver.c
index 9e4dd9e..f37d064 100644
--- a/src/esx/esx_storage_driver.c
+++ b/src/esx/esx_storage_driver.c
@@ -1523,7 +1523,7 @@ esxStorageVolumeGetInfo(virStorageVolPtr volume, virStorageVolInfoPtr info)
 
 
 static char *
-esxStorageVolumeDumpXML(virStorageVolPtr volume, unsigned int flags)
+esxStorageVolumeGetXMLDesc(virStorageVolPtr volume, unsigned int flags)
 {
     esxPrivate *priv = volume->conn->storagePrivateData;
     virStoragePoolDef pool;
@@ -1676,7 +1676,7 @@ static virStorageDriver esxStorageDriver = {
     esxStorageVolumeDelete,                /* volDelete */
     esxStorageVolumeWipe,                  /* volWipe */
     esxStorageVolumeGetInfo,               /* volGetInfo */
-    esxStorageVolumeDumpXML,               /* volGetXMLDesc */
+    esxStorageVolumeGetXMLDesc,            /* volGetXMLDesc */
     esxStorageVolumeGetPath,               /* volGetPath */
     esxStoragePoolIsActive,                /* poolIsActive */
     esxStoragePoolIsPersistent,            /* poolIsPersistent */
diff --git a/src/libvirt.c b/src/libvirt.c
index e74e977..c93ab57 100644
--- a/src/libvirt.c
+++ b/src/libvirt.c
@@ -3208,9 +3208,9 @@ virDomainGetXMLDesc(virDomainPtr domain, int flags)
 
     flags &= VIR_DOMAIN_XML_FLAGS_MASK;
 
-    if (conn->driver->domainDumpXML) {
+    if (conn->driver->domainGetXMLDesc) {
         char *ret;
-        ret = conn->driver->domainDumpXML (domain, flags);
+        ret = conn->driver->domainGetXMLDesc(domain, flags);
         if (!ret)
             goto error;
         return ret;
@@ -3433,14 +3433,14 @@ virDomainMigrateVersion2 (virDomainPtr domain,
      * different.  We fetch the domain XML of the source domain
      * and pass it to Prepare2.
      */
-    if (!domain->conn->driver->domainDumpXML) {
+    if (!domain->conn->driver->domainGetXMLDesc) {
         virLibConnError(VIR_ERR_INTERNAL_ERROR, __FUNCTION__);
         virDispatchError(domain->conn);
         return NULL;
     }
-    dom_xml = domain->conn->driver->domainDumpXML (domain,
-                                                   VIR_DOMAIN_XML_SECURE |
-                                                   VIR_DOMAIN_XML_UPDATE_CPU);
+    dom_xml = domain->conn->driver->domainGetXMLDesc(domain,
+                                                     VIR_DOMAIN_XML_SECURE |
+                                                     VIR_DOMAIN_XML_UPDATE_CPU);
     if (!dom_xml)
         return NULL;
 
@@ -6719,9 +6719,9 @@ virNetworkGetXMLDesc(virNetworkPtr network, int flags)
 
     conn = network->conn;
 
-    if (conn->networkDriver && conn->networkDriver->networkDumpXML) {
+    if (conn->networkDriver && conn->networkDriver->networkGetXMLDesc) {
         char *ret;
-        ret = conn->networkDriver->networkDumpXML (network, flags);
+        ret = conn->networkDriver->networkGetXMLDesc(network, flags);
         if (!ret)
             goto error;
         return ret;
@@ -9636,9 +9636,9 @@ char *virNodeDeviceGetXMLDesc(virNodeDevicePtr dev, unsigned int flags)
         return NULL;
     }
 
-    if (dev->conn->deviceMonitor && dev->conn->deviceMonitor->deviceDumpXML) {
+    if (dev->conn->deviceMonitor && dev->conn->deviceMonitor->deviceGetXMLDesc) {
         char *ret;
-        ret = dev->conn->deviceMonitor->deviceDumpXML (dev, flags);
+        ret = dev->conn->deviceMonitor->deviceGetXMLDesc(dev, flags);
         if (!ret)
             goto error;
         return ret;
@@ -13057,9 +13057,9 @@ virDomainSnapshotGetXMLDesc(virDomainSnapshotPtr snapshot,
         goto error;
     }
 
-    if (conn->driver->domainSnapshotDumpXML) {
+    if (conn->driver->domainSnapshotGetXMLDesc) {
         char *ret;
-        ret = conn->driver->domainSnapshotDumpXML(snapshot, flags);
+        ret = conn->driver->domainSnapshotGetXMLDesc(snapshot, flags);
         if (!ret)
             goto error;
         return ret;
diff --git a/src/libxl/libxl_driver.c b/src/libxl/libxl_driver.c
index a2c8467..fd4984f 100644
--- a/src/libxl/libxl_driver.c
+++ b/src/libxl/libxl_driver.c
@@ -1916,7 +1916,7 @@ cleanup:
 }
 
 static char *
-libxlDomainDumpXML(virDomainPtr dom, int flags)
+libxlDomainGetXMLDesc(virDomainPtr dom, int flags)
 {
     libxlDriverPrivatePtr driver = dom->conn->privateData;
     virDomainObjPtr vm;
@@ -2725,7 +2725,7 @@ static virDriver libxlDriver = {
     NULL,                       /* domainGetMaxVcpus */
     NULL,                       /* domainGetSecurityLabel */
     NULL,                       /* nodeGetSecurityModel */
-    libxlDomainDumpXML,         /* domainDumpXML */
+    libxlDomainGetXMLDesc,      /* domainGetXMLDesc */
     libxlDomainXMLFromNative,   /* domainXmlFromNative */
     libxlDomainXMLToNative,     /* domainXmlToNative */
     libxlListDefinedDomains,    /* listDefinedDomains */
@@ -2780,7 +2780,7 @@ static virDriver libxlDriver = {
     NULL,                       /* domainHasManagedSaveImage */
     NULL,                       /* domainManagedSaveRemove */
     NULL,                       /* domainSnapshotCreateXML */
-    NULL,                       /* domainSnapshotDumpXML */
+    NULL,                       /* domainSnapshotGetXMLDesc */
     NULL,                       /* domainSnapshotNum */
     NULL,                       /* domainSnapshotListNames */
     NULL,                       /* domainSnapshotLookupByName */
diff --git a/src/lxc/lxc_driver.c b/src/lxc/lxc_driver.c
index b94941d..a081c6d 100644
--- a/src/lxc/lxc_driver.c
+++ b/src/lxc/lxc_driver.c
@@ -916,8 +916,8 @@ cleanup:
     return ret;
 }
 
-static char *lxcDomainDumpXML(virDomainPtr dom,
-                              int flags)
+static char *lxcDomainGetXMLDesc(virDomainPtr dom,
+                                 int flags)
 {
     lxc_driver_t *driver = dom->conn->privateData;
     virDomainObjPtr vm;
@@ -2840,7 +2840,7 @@ static virDriver lxcDriver = {
     NULL, /* domainGetMaxVcpus */
     NULL, /* domainGetSecurityLabel */
     NULL, /* nodeGetSecurityModel */
-    lxcDomainDumpXML, /* domainDumpXML */
+    lxcDomainGetXMLDesc, /* domainGetXMLDesc */
     NULL, /* domainXMLFromNative */
     NULL, /* domainXMLToNative */
     lxcListDefinedDomains, /* listDefinedDomains */
@@ -2895,7 +2895,7 @@ static virDriver lxcDriver = {
     NULL, /* domainHasManagedSaveImage */
     NULL, /* domainManagedSaveRemove */
     NULL, /* domainSnapshotCreateXML */
-    NULL, /* domainSnapshotDumpXML */
+    NULL, /* domainSnapshotGetXMLDesc */
     NULL, /* domainSnapshotNum */
     NULL, /* domainSnapshotListNames */
     NULL, /* domainSnapshotLookupByName */
diff --git a/src/network/bridge_driver.c b/src/network/bridge_driver.c
index 12050b3..7ca2f23 100644
--- a/src/network/bridge_driver.c
+++ b/src/network/bridge_driver.c
@@ -2356,7 +2356,7 @@ cleanup:
     return ret;
 }
 
-static char *networkDumpXML(virNetworkPtr net, int flags ATTRIBUTE_UNUSED) {
+static char *networkGetXMLDesc(virNetworkPtr net, int flags ATTRIBUTE_UNUSED) {
     struct network_driver *driver = net->conn->networkPrivateData;
     virNetworkObjPtr network;
     char *ret = NULL;
@@ -2517,7 +2517,7 @@ static virNetworkDriver networkDriver = {
     networkUndefine, /* networkUndefine */
     networkStart, /* networkCreate */
     networkDestroy, /* networkDestroy */
-    networkDumpXML, /* networkDumpXML */
+    networkGetXMLDesc, /* networkGetXMLDesc */
     networkGetBridgeName, /* networkGetBridgeName */
     networkGetAutostart, /* networkGetAutostart */
     networkSetAutostart, /* networkSetAutostart */
diff --git a/src/node_device/node_device_driver.c b/src/node_device/node_device_driver.c
index a6c6042..8ec3e44 100644
--- a/src/node_device/node_device_driver.c
+++ b/src/node_device/node_device_driver.c
@@ -251,8 +251,8 @@ out:
 }
 
 
-static char *nodeDeviceDumpXML(virNodeDevicePtr dev,
-                               unsigned int flags ATTRIBUTE_UNUSED)
+static char *nodeDeviceGetXMLDesc(virNodeDevicePtr dev,
+                                  unsigned int flags ATTRIBUTE_UNUSED)
 {
     virDeviceMonitorStatePtr driver = dev->conn->devMonPrivateData;
     virNodeDeviceObjPtr obj;
@@ -657,7 +657,7 @@ void registerCommonNodeFuncs(virDeviceMonitorPtr driver)
     driver->numOfDevices = nodeNumOfDevices;
     driver->listDevices = nodeListDevices;
     driver->deviceLookupByName = nodeDeviceLookupByName;
-    driver->deviceDumpXML = nodeDeviceDumpXML;
+    driver->deviceGetXMLDesc = nodeDeviceGetXMLDesc;
     driver->deviceGetParent = nodeDeviceGetParent;
     driver->deviceNumOfCaps = nodeDeviceNumOfCaps;
     driver->deviceListCaps = nodeDeviceListCaps;
diff --git a/src/nwfilter/nwfilter_driver.c b/src/nwfilter/nwfilter_driver.c
index 8af3f8a..17af7a6 100644
--- a/src/nwfilter/nwfilter_driver.c
+++ b/src/nwfilter/nwfilter_driver.c
@@ -406,8 +406,8 @@ cleanup:
 
 
 static char *
-nwfilterDumpXML(virNWFilterPtr obj,
-                unsigned int flags) {
+nwfilterGetXMLDesc(virNWFilterPtr obj,
+                   unsigned int flags) {
     virNWFilterDriverStatePtr driver = obj->conn->nwfilterPrivateData;
     virNWFilterObjPtr nwfilter;
     char *ret = NULL;
@@ -457,7 +457,7 @@ static virNWFilterDriver nwfilterDriver = {
     .nwfilterLookupByUUID = nwfilterLookupByUUID,
     .defineXML = nwfilterDefine,
     .undefine = nwfilterUndefine,
-    .getXMLDesc = nwfilterDumpXML,
+    .getXMLDesc = nwfilterGetXMLDesc,
 };
 
 
diff --git a/src/openvz/openvz_driver.c b/src/openvz/openvz_driver.c
index 0bd007a..539c850 100644
--- a/src/openvz/openvz_driver.c
+++ b/src/openvz/openvz_driver.c
@@ -425,7 +425,7 @@ static int openvzDomainIsUpdated(virDomainPtr dom ATTRIBUTE_UNUSED)
     return 0;
 }
 
-static char *openvzDomainDumpXML(virDomainPtr dom, int flags) {
+static char *openvzDomainGetXMLDesc(virDomainPtr dom, int flags) {
     struct openvz_driver *driver = dom->conn->privateData;
     virDomainObjPtr vm;
     char *ret = NULL;
@@ -1602,7 +1602,7 @@ static virDriver openvzDriver = {
     openvzDomainGetMaxVcpus, /* domainGetMaxVcpus */
     NULL, /* domainGetSecurityLabel */
     NULL, /* nodeGetSecurityModel */
-    openvzDomainDumpXML, /* domainDumpXML */
+    openvzDomainGetXMLDesc, /* domainGetXMLDesc */
     NULL, /* domainXmlFromNative */
     NULL, /* domainXmlToNative */
     openvzListDefinedDomains, /* listDefinedDomains */
@@ -1657,7 +1657,7 @@ static virDriver openvzDriver = {
     NULL, /* domainHasManagedSaveImage */
     NULL, /* domainManagedSaveRemove */
     NULL, /* domainSnapshotCreateXML */
-    NULL, /* domainSnapshotDumpXML */
+    NULL, /* domainSnapshotGetXMLDesc */
     NULL, /* domainSnapshotNum */
     NULL, /* domainSnapshotListNames */
     NULL, /* domainSnapshotLookupByName */
diff --git a/src/phyp/phyp_driver.c b/src/phyp/phyp_driver.c
index 30d4adf..b58b206 100644
--- a/src/phyp/phyp_driver.c
+++ b/src/phyp/phyp_driver.c
@@ -3302,7 +3302,7 @@ cleanup:
 }
 
 static char *
-phypDomainDumpXML(virDomainPtr dom, int flags)
+phypDomainGetXMLDesc(virDomainPtr dom, int flags)
 {
     ConnectionData *connection_data = dom->conn->networkPrivateData;
     phyp_driverPtr phyp_driver = dom->conn->privateData;
@@ -3763,7 +3763,7 @@ static virDriver phypDriver = {
     phypGetLparCPUMAX,          /* domainGetMaxVcpus */
     NULL,                       /* domainGetSecurityLabel */
     NULL,                       /* nodeGetSecurityModel */
-    phypDomainDumpXML,          /* domainDumpXML */
+    phypDomainGetXMLDesc,       /* domainGetXMLDesc */
     NULL,                       /* domainXMLFromNative */
     NULL,                       /* domainXMLToNative */
     phypListDefinedDomains,     /* listDefinedDomains */
@@ -3818,7 +3818,7 @@ static virDriver phypDriver = {
     NULL,                       /* domainHasManagedSaveImage */
     NULL,                       /* domainManagedSaveRemove */
     NULL,                       /* domainSnapshotCreateXML */
-    NULL,                       /* domainSnapshotDumpXML */
+    NULL,                       /* domainSnapshotGetXMLDesc */
     NULL,                       /* domainSnapshotNum */
     NULL,                       /* domainSnapshotListNames */
     NULL,                       /* domainSnapshotLookupByName */
diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
index 22b2634..2828823 100644
--- a/src/qemu/qemu_command.c
+++ b/src/qemu/qemu_command.c
@@ -241,7 +241,7 @@ qemuNetworkIfaceConnect(virDomainDefPtr def,
             virReportOOMError();
             goto cleanup;
         }
-        /* avoid exposing vnet%d in dumpxml or error outputs */
+        /* avoid exposing vnet%d in getXMLDesc or error outputs */
         template_ifname = 1;
     }
 
diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 0fd0f10..6275826 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -3287,8 +3287,8 @@ cleanup:
 }
 
 
-static char *qemudDomainDumpXML(virDomainPtr dom,
-                                int flags) {
+static char *qemuDomainGetXMLDesc(virDomainPtr dom,
+                                  int flags) {
     struct qemud_driver *driver = dom->conn->privateData;
     virDomainObjPtr vm;
     char *ret = NULL;
@@ -6580,8 +6580,8 @@ cleanup:
     return snapshot;
 }
 
-static char *qemuDomainSnapshotDumpXML(virDomainSnapshotPtr snapshot,
-                                       unsigned int flags)
+static char *qemuDomainSnapshotGetXMLDesc(virDomainSnapshotPtr snapshot,
+                                          unsigned int flags)
 {
     struct qemud_driver *driver = snapshot->domain->conn->privateData;
     virDomainObjPtr vm = NULL;
@@ -7127,7 +7127,7 @@ static virDriver qemuDriver = {
     qemudDomainGetMaxVcpus, /* domainGetMaxVcpus */
     qemudDomainGetSecurityLabel, /* domainGetSecurityLabel */
     qemudNodeGetSecurityModel, /* nodeGetSecurityModel */
-    qemudDomainDumpXML, /* domainDumpXML */
+    qemuDomainGetXMLDesc, /* domainGetXMLDesc */
     qemuDomainXMLFromNative, /* domainXmlFromNative */
     qemuDomainXMLToNative, /* domainXMLToNative */
     qemudListDefinedDomains, /* listDefinedDomains */
@@ -7182,7 +7182,7 @@ static virDriver qemuDriver = {
     qemuDomainHasManagedSaveImage, /* domainHasManagedSaveImage */
     qemuDomainManagedSaveRemove, /* domainManagedSaveRemove */
     qemuDomainSnapshotCreateXML, /* domainSnapshotCreateXML */
-    qemuDomainSnapshotDumpXML, /* domainSnapshotDumpXML */
+    qemuDomainSnapshotGetXMLDesc, /* domainSnapshotGetXMLDesc */
     qemuDomainSnapshotNum, /* domainSnapshotNum */
     qemuDomainSnapshotListNames, /* domainSnapshotListNames */
     qemuDomainSnapshotLookupByName, /* domainSnapshotLookupByName */
diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c
index d076a90..322173c 100644
--- a/src/remote/remote_driver.c
+++ b/src/remote/remote_driver.c
@@ -6428,7 +6428,7 @@ static virDriver remote_driver = {
     remoteDomainGetMaxVcpus, /* domainGetMaxVcpus */
     remoteDomainGetSecurityLabel, /* domainGetSecurityLabel */
     remoteNodeGetSecurityModel, /* nodeGetSecurityModel */
-    remoteDomainDumpXML, /* domainDumpXML */
+    remoteDomainGetXMLDesc, /* domainGetXMLDesc */
     remoteDomainXMLFromNative, /* domainXMLFromNative */
     remoteDomainXMLToNative, /* domainXMLToNative */
     remoteListDefinedDomains, /* listDefinedDomains */
@@ -6483,7 +6483,7 @@ static virDriver remote_driver = {
     remoteDomainHasManagedSaveImage, /* domainHasManagedSaveImage */
     remoteDomainManagedSaveRemove, /* domainManagedSaveRemove */
     remoteDomainSnapshotCreateXML, /* domainSnapshotCreateXML */
-    remoteDomainSnapshotDumpXML, /* domainSnapshotDumpXML */
+    remoteDomainSnapshotGetXMLDesc, /* domainSnapshotGetXMLDesc */
     remoteDomainSnapshotNum, /* domainSnapshotNum */
     remoteDomainSnapshotListNames, /* domainSnapshotListNames */
     remoteDomainSnapshotLookupByName, /* domainSnapshotLookupByName */
@@ -6510,7 +6510,7 @@ static virNetworkDriver network_driver = {
     .networkUndefine = remoteNetworkUndefine,
     .networkCreate = remoteNetworkCreate,
     .networkDestroy = remoteNetworkDestroy,
-    .networkDumpXML = remoteNetworkDumpXML,
+    .networkGetXMLDesc = remoteNetworkGetXMLDesc,
     .networkGetBridgeName = remoteNetworkGetBridgeName,
     .networkGetAutostart = remoteNetworkGetAutostart,
     .networkSetAutostart = remoteNetworkSetAutostart,
@@ -6557,7 +6557,7 @@ static virStorageDriver storage_driver = {
     .poolDelete = remoteStoragePoolDelete,
     .poolRefresh = remoteStoragePoolRefresh,
     .poolGetInfo = remoteStoragePoolGetInfo,
-    .poolGetXMLDesc = remoteStoragePoolDumpXML,
+    .poolGetXMLDesc = remoteStoragePoolGetXMLDesc,
     .poolGetAutostart = remoteStoragePoolGetAutostart,
     .poolSetAutostart = remoteStoragePoolSetAutostart,
     .poolNumOfVolumes = remoteStoragePoolNumOfVolumes,
@@ -6573,7 +6573,7 @@ static virStorageDriver storage_driver = {
     .volDelete = remoteStorageVolDelete,
     .volWipe = remoteStorageVolWipe,
     .volGetInfo = remoteStorageVolGetInfo,
-    .volGetXMLDesc = remoteStorageVolDumpXML,
+    .volGetXMLDesc = remoteStorageVolGetXMLDesc,
     .volGetPath = remoteStorageVolGetPath,
     .poolIsActive = remoteStoragePoolIsActive,
     .poolIsPersistent = remoteStoragePoolIsPersistent,
@@ -6601,7 +6601,7 @@ static virDeviceMonitor dev_monitor = {
     .numOfDevices = remoteNodeNumOfDevices,
     .listDevices = remoteNodeListDevices,
     .deviceLookupByName = remoteNodeDeviceLookupByName,
-    .deviceDumpXML = remoteNodeDeviceDumpXML,
+    .deviceGetXMLDesc = remoteNodeDeviceGetXMLDesc,
     .deviceGetParent = remoteNodeDeviceGetParent,
     .deviceNumOfCaps = remoteNodeDeviceNumOfCaps,
     .deviceListCaps = remoteNodeDeviceListCaps,
diff --git a/src/remote/remote_protocol.x b/src/remote/remote_protocol.x
index c706c36..7f9a30c 100644
--- a/src/remote/remote_protocol.x
+++ b/src/remote/remote_protocol.x
@@ -714,12 +714,12 @@ struct remote_domain_core_dump_args {
     int flags;
 };
 
-struct remote_domain_dump_xml_args {
+struct remote_domain_get_xml_desc_args {
     remote_nonnull_domain dom;
     int flags;
 };
 
-struct remote_domain_dump_xml_ret {
+struct remote_domain_get_xml_desc_ret {
     remote_nonnull_string xml;
 };
 
@@ -987,12 +987,12 @@ struct remote_network_destroy_args {
     remote_nonnull_network net;
 };
 
-struct remote_network_dump_xml_args {
+struct remote_network_get_xml_desc_args {
     remote_nonnull_network net;
     int flags;
 };
 
-struct remote_network_dump_xml_ret {
+struct remote_network_get_xml_desc_ret {
     remote_nonnull_string xml;
 };
 
@@ -1289,12 +1289,12 @@ struct remote_storage_pool_refresh_args {
     unsigned flags;
 };
 
-struct remote_storage_pool_dump_xml_args {
+struct remote_storage_pool_get_xml_desc_args {
     remote_nonnull_storage_pool pool;
     unsigned flags;
 };
 
-struct remote_storage_pool_dump_xml_ret {
+struct remote_storage_pool_get_xml_desc_ret {
     remote_nonnull_string xml;
 };
 
@@ -1399,12 +1399,12 @@ struct remote_storage_vol_wipe_args {
     unsigned flags;
 };
 
-struct remote_storage_vol_dump_xml_args {
+struct remote_storage_vol_get_xml_desc_args {
     remote_nonnull_storage_vol vol;
     unsigned flags;
 };
 
-struct remote_storage_vol_dump_xml_ret {
+struct remote_storage_vol_get_xml_desc_ret {
     remote_nonnull_string xml;
 };
 
@@ -1455,12 +1455,12 @@ struct remote_node_device_lookup_by_name_ret {
     remote_nonnull_node_device dev;
 };
 
-struct remote_node_device_dump_xml_args {
+struct remote_node_device_get_xml_desc_args {
     remote_nonnull_string name;
     unsigned flags;
 };
 
-struct remote_node_device_dump_xml_ret {
+struct remote_node_device_get_xml_desc_ret {
     remote_nonnull_string xml;
 };
 
@@ -1854,12 +1854,12 @@ struct remote_domain_snapshot_create_xml_ret {
     remote_nonnull_domain_snapshot snap;
 };
 
-struct remote_domain_snapshot_dump_xml_args {
+struct remote_domain_snapshot_get_xml_desc_args {
     remote_nonnull_domain_snapshot snap;
     int flags;
 };
 
-struct remote_domain_snapshot_dump_xml_ret {
+struct remote_domain_snapshot_get_xml_desc_ret {
     remote_nonnull_string xml;
 };
 
@@ -1962,7 +1962,7 @@ enum remote_procedure {
     REMOTE_PROC_DOMAIN_DEFINE_XML = 11,
     REMOTE_PROC_DOMAIN_DESTROY = 12,
     REMOTE_PROC_DOMAIN_DETACH_DEVICE = 13,
-    REMOTE_PROC_DOMAIN_DUMP_XML = 14,
+    REMOTE_PROC_DOMAIN_GET_XML_DESC = 14,
     REMOTE_PROC_DOMAIN_GET_AUTOSTART = 15,
     REMOTE_PROC_DOMAIN_GET_INFO = 16,
     REMOTE_PROC_DOMAIN_GET_MAX_MEMORY = 17,
@@ -1994,7 +1994,7 @@ enum remote_procedure {
 
     REMOTE_PROC_NETWORK_DEFINE_XML = 41,
     REMOTE_PROC_NETWORK_DESTROY = 42,
-    REMOTE_PROC_NETWORK_DUMP_XML = 43,
+    REMOTE_PROC_NETWORK_GET_XML_DESC = 43,
     REMOTE_PROC_NETWORK_GET_AUTOSTART = 44,
     REMOTE_PROC_NETWORK_GET_BRIDGE_NAME = 45,
     REMOTE_PROC_NETWORK_LOOKUP_BY_NAME = 46,
@@ -2043,7 +2043,7 @@ enum remote_procedure {
     REMOTE_PROC_STORAGE_POOL_LOOKUP_BY_UUID = 85,
     REMOTE_PROC_STORAGE_POOL_LOOKUP_BY_VOLUME = 86,
     REMOTE_PROC_STORAGE_POOL_GET_INFO = 87,
-    REMOTE_PROC_STORAGE_POOL_DUMP_XML = 88,
+    REMOTE_PROC_STORAGE_POOL_GET_XML_DESC = 88,
     REMOTE_PROC_STORAGE_POOL_GET_AUTOSTART = 89,
     REMOTE_PROC_STORAGE_POOL_SET_AUTOSTART = 90,
 
@@ -2055,7 +2055,7 @@ enum remote_procedure {
     REMOTE_PROC_STORAGE_VOL_LOOKUP_BY_KEY = 96,
     REMOTE_PROC_STORAGE_VOL_LOOKUP_BY_PATH = 97,
     REMOTE_PROC_STORAGE_VOL_GET_INFO = 98,
-    REMOTE_PROC_STORAGE_VOL_DUMP_XML = 99,
+    REMOTE_PROC_STORAGE_VOL_GET_XML_DESC = 99,
     REMOTE_PROC_STORAGE_VOL_GET_PATH = 100,
 
     REMOTE_PROC_NODE_GET_CELLS_FREE_MEMORY = 101,
@@ -2072,7 +2072,7 @@ enum remote_procedure {
     REMOTE_PROC_NODE_NUM_OF_DEVICES = 111,
     REMOTE_PROC_NODE_LIST_DEVICES = 112,
     REMOTE_PROC_NODE_DEVICE_LOOKUP_BY_NAME = 113,
-    REMOTE_PROC_NODE_DEVICE_DUMP_XML = 114,
+    REMOTE_PROC_NODE_DEVICE_GET_XML_DESC = 114,
     REMOTE_PROC_NODE_DEVICE_GET_PARENT = 115,
     REMOTE_PROC_NODE_DEVICE_NUM_OF_CAPS = 116,
     REMOTE_PROC_NODE_DEVICE_LIST_CAPS = 117,
@@ -2151,7 +2151,7 @@ enum remote_procedure {
     REMOTE_PROC_DOMAIN_HAS_MANAGED_SAVE_IMAGE = 183,
     REMOTE_PROC_DOMAIN_MANAGED_SAVE_REMOVE = 184,
     REMOTE_PROC_DOMAIN_SNAPSHOT_CREATE_XML = 185,
-    REMOTE_PROC_DOMAIN_SNAPSHOT_DUMP_XML = 186,
+    REMOTE_PROC_DOMAIN_SNAPSHOT_GET_XML_DESC = 186,
     REMOTE_PROC_DOMAIN_SNAPSHOT_NUM = 187,
     REMOTE_PROC_DOMAIN_SNAPSHOT_LIST_NAMES = 188,
     REMOTE_PROC_DOMAIN_SNAPSHOT_LOOKUP_BY_NAME = 189,
diff --git a/src/remote_protocol-structs b/src/remote_protocol-structs
index f904c4d..eed63ca 100644
--- a/src/remote_protocol-structs
+++ b/src/remote_protocol-structs
@@ -403,11 +403,11 @@ struct remote_domain_core_dump_args {
         remote_nonnull_string      to;
         int                        flags;
 };
-struct remote_domain_dump_xml_args {
+struct remote_domain_get_xml_desc_args {
         remote_nonnull_domain      dom;
         int                        flags;
 };
-struct remote_domain_dump_xml_ret {
+struct remote_domain_get_xml_desc_ret {
         remote_nonnull_string      xml;
 };
 struct remote_domain_migrate_prepare_args {
@@ -659,11 +659,11 @@ struct remote_network_create_args {
 struct remote_network_destroy_args {
         remote_nonnull_network     net;
 };
-struct remote_network_dump_xml_args {
+struct remote_network_get_xml_desc_args {
         remote_nonnull_network     net;
         int                        flags;
 };
-struct remote_network_dump_xml_ret {
+struct remote_network_get_xml_desc_ret {
         remote_nonnull_string      xml;
 };
 struct remote_network_get_bridge_name_args {
@@ -912,11 +912,11 @@ struct remote_storage_pool_refresh_args {
         remote_nonnull_storage_pool pool;
         u_int                      flags;
 };
-struct remote_storage_pool_dump_xml_args {
+struct remote_storage_pool_get_xml_desc_args {
         remote_nonnull_storage_pool pool;
         u_int                      flags;
 };
-struct remote_storage_pool_dump_xml_ret {
+struct remote_storage_pool_get_xml_desc_ret {
         remote_nonnull_string      xml;
 };
 struct remote_storage_pool_get_info_args {
@@ -998,11 +998,11 @@ struct remote_storage_vol_wipe_args {
         remote_nonnull_storage_vol vol;
         u_int                      flags;
 };
-struct remote_storage_vol_dump_xml_args {
+struct remote_storage_vol_get_xml_desc_args {
         remote_nonnull_storage_vol vol;
         u_int                      flags;
 };
-struct remote_storage_vol_dump_xml_ret {
+struct remote_storage_vol_get_xml_desc_ret {
         remote_nonnull_string      xml;
 };
 struct remote_storage_vol_get_info_args {
@@ -1043,11 +1043,11 @@ struct remote_node_device_lookup_by_name_args {
 struct remote_node_device_lookup_by_name_ret {
         remote_nonnull_node_device dev;
 };
-struct remote_node_device_dump_xml_args {
+struct remote_node_device_get_xml_desc_args {
         remote_nonnull_string      name;
         u_int                      flags;
 };
-struct remote_node_device_dump_xml_ret {
+struct remote_node_device_get_xml_desc_ret {
         remote_nonnull_string      xml;
 };
 struct remote_node_device_get_parent_args {
@@ -1355,11 +1355,11 @@ struct remote_domain_snapshot_create_xml_args {
 struct remote_domain_snapshot_create_xml_ret {
         remote_nonnull_domain_snapshot snap;
 };
-struct remote_domain_snapshot_dump_xml_args {
+struct remote_domain_snapshot_get_xml_desc_args {
         remote_nonnull_domain_snapshot snap;
         int                        flags;
 };
-struct remote_domain_snapshot_dump_xml_ret {
+struct remote_domain_snapshot_get_xml_desc_ret {
         remote_nonnull_string      xml;
 };
 struct remote_domain_snapshot_num_args {
diff --git a/src/storage/storage_driver.c b/src/storage/storage_driver.c
index 5118ffb..9328ca4 100644
--- a/src/storage/storage_driver.c
+++ b/src/storage/storage_driver.c
@@ -939,8 +939,8 @@ cleanup:
 }
 
 static char *
-storagePoolDumpXML(virStoragePoolPtr obj,
-                   unsigned int flags ATTRIBUTE_UNUSED) {
+storagePoolGetXMLDesc(virStoragePoolPtr obj,
+                      unsigned int flags ATTRIBUTE_UNUSED) {
     virStorageDriverStatePtr driver = obj->conn->storagePrivateData;
     virStoragePoolObjPtr pool;
     char *ret = NULL;
@@ -2122,7 +2122,7 @@ static virStorageDriver storageDriver = {
     .poolDelete = storagePoolDelete,
     .poolRefresh = storagePoolRefresh,
     .poolGetInfo = storagePoolGetInfo,
-    .poolGetXMLDesc = storagePoolDumpXML,
+    .poolGetXMLDesc = storagePoolGetXMLDesc,
     .poolGetAutostart = storagePoolGetAutostart,
     .poolSetAutostart = storagePoolSetAutostart,
     .poolNumOfVolumes = storagePoolNumVolumes,
diff --git a/src/test/test_driver.c b/src/test/test_driver.c
index 0978214..8524a19 100644
--- a/src/test/test_driver.c
+++ b/src/test/test_driver.c
@@ -2336,7 +2336,7 @@ cleanup:
     return ret;
 }
 
-static char *testDomainDumpXML(virDomainPtr domain, int flags)
+static char *testDomainGetXMLDesc(virDomainPtr domain, int flags)
 {
     testConnPtr privconn = domain->conn->privateData;
     virDomainDefPtr def;
@@ -3137,7 +3137,7 @@ cleanup:
     return ret;
 }
 
-static char *testNetworkDumpXML(virNetworkPtr network, int flags ATTRIBUTE_UNUSED) {
+static char *testNetworkGetXMLDesc(virNetworkPtr network, int flags ATTRIBUTE_UNUSED) {
     testConnPtr privconn = network->conn->privateData;
     virNetworkObjPtr privnet;
     char *ret = NULL;
@@ -4162,8 +4162,8 @@ cleanup:
 }
 
 static char *
-testStoragePoolDumpXML(virStoragePoolPtr pool,
-                       unsigned int flags ATTRIBUTE_UNUSED) {
+testStoragePoolGetXMLDesc(virStoragePoolPtr pool,
+                          unsigned int flags ATTRIBUTE_UNUSED) {
     testConnPtr privconn = pool->conn->privateData;
     virStoragePoolObjPtr privpool;
     char *ret = NULL;
@@ -4914,8 +4914,8 @@ cleanup:
 }
 
 static char *
-testNodeDeviceDumpXML(virNodeDevicePtr dev,
-                      unsigned int flags ATTRIBUTE_UNUSED)
+testNodeDeviceGetXMLDesc(virNodeDevicePtr dev,
+                         unsigned int flags ATTRIBUTE_UNUSED)
 {
     testConnPtr driver = dev->conn->privateData;
     virNodeDeviceObjPtr obj;
@@ -5382,7 +5382,7 @@ static virDriver testDriver = {
     testDomainGetMaxVcpus, /* domainGetMaxVcpus */
     NULL, /* domainGetSecurityLabel */
     NULL, /* nodeGetSecurityModel */
-    testDomainDumpXML, /* domainDumpXML */
+    testDomainGetXMLDesc, /* domainGetXMLDesc */
     NULL, /* domainXMLFromNative */
     NULL, /* domainXMLToNative */
     testListDefinedDomains, /* listDefinedDomains */
@@ -5437,7 +5437,7 @@ static virDriver testDriver = {
     NULL, /* domainHasManagedSaveImage */
     NULL, /* domainManagedSaveRemove */
     NULL, /* domainSnapshotCreateXML */
-    NULL, /* domainSnapshotDumpXML */
+    NULL, /* domainSnapshotGetXMLDesc */
     NULL, /* domainSnapshotNum */
     NULL, /* domainSnapshotListNames */
     NULL, /* domainSnapshotLookupByName */
@@ -5464,7 +5464,7 @@ static virNetworkDriver testNetworkDriver = {
     testNetworkUndefine, /* networkUndefine */
     testNetworkStart, /* networkCreate */
     testNetworkDestroy, /* networkDestroy */
-    testNetworkDumpXML, /* networkDumpXML */
+    testNetworkGetXMLDesc, /* networkGetXMLDesc */
     testNetworkGetBridgeName, /* networkGetBridgeName */
     testNetworkGetAutostart, /* networkGetAutostart */
     testNetworkSetAutostart, /* networkSetAutostart */
@@ -5513,7 +5513,7 @@ static virStorageDriver testStorageDriver = {
     .poolDelete = testStoragePoolDelete,
     .poolRefresh = testStoragePoolRefresh,
     .poolGetInfo = testStoragePoolGetInfo,
-    .poolGetXMLDesc = testStoragePoolDumpXML,
+    .poolGetXMLDesc = testStoragePoolGetXMLDesc,
     .poolGetAutostart = testStoragePoolGetAutostart,
     .poolSetAutostart = testStoragePoolSetAutostart,
     .poolNumOfVolumes = testStoragePoolNumVolumes,
@@ -5540,7 +5540,7 @@ static virDeviceMonitor testDevMonitor = {
     .numOfDevices = testNodeNumOfDevices,
     .listDevices = testNodeListDevices,
     .deviceLookupByName = testNodeDeviceLookupByName,
-    .deviceDumpXML = testNodeDeviceDumpXML,
+    .deviceGetXMLDesc = testNodeDeviceGetXMLDesc,
     .deviceGetParent = testNodeDeviceGetParent,
     .deviceNumOfCaps = testNodeDeviceNumOfCaps,
     .deviceListCaps = testNodeDeviceListCaps,
diff --git a/src/uml/uml_conf.c b/src/uml/uml_conf.c
index 84b74f3..b741edb 100644
--- a/src/uml/uml_conf.c
+++ b/src/uml/uml_conf.c
@@ -132,7 +132,7 @@ umlConnectTapDevice(virConnectPtr conn,
         VIR_FREE(net->ifname);
         if (!(net->ifname = strdup("vnet%d")))
             goto no_memory;
-        /* avoid exposing vnet%d in dumpxml or error outputs */
+        /* avoid exposing vnet%d in getXMLDesc or error outputs */
         template_ifname = 1;
     }
 
diff --git a/src/uml/uml_driver.c b/src/uml/uml_driver.c
index 33849a0..42590c0 100644
--- a/src/uml/uml_driver.c
+++ b/src/uml/uml_driver.c
@@ -1522,8 +1522,8 @@ cleanup:
 }
 
 
-static char *umlDomainDumpXML(virDomainPtr dom,
-                                int flags ATTRIBUTE_UNUSED) {
+static char *umlDomainGetXMLDesc(virDomainPtr dom,
+                                 int flags ATTRIBUTE_UNUSED) {
     struct uml_driver *driver = dom->conn->privateData;
     virDomainObjPtr vm;
     char *ret = NULL;
@@ -2188,7 +2188,7 @@ static virDriver umlDriver = {
     NULL, /* domainGetMaxVcpus */
     NULL, /* domainGetSecurityLabel */
     NULL, /* nodeGetSecurityModel */
-    umlDomainDumpXML, /* domainDumpXML */
+    umlDomainGetXMLDesc, /* domainGetXMLDesc */
     NULL, /* domainXMLFromNative */
     NULL, /* domainXMLToNative */
     umlListDefinedDomains, /* listDefinedDomains */
@@ -2243,7 +2243,7 @@ static virDriver umlDriver = {
     NULL, /* domainHasManagedSaveImage */
     NULL, /* domainManagedSaveRemove */
     NULL, /* domainSnapshotCreateXML */
-    NULL, /* domainSnapshotDumpXML */
+    NULL, /* domainSnapshotGetXMLDesc */
     NULL, /* domainSnapshotNum */
     NULL, /* domainSnapshotListNames */
     NULL, /* domainSnapshotLookupByName */
diff --git a/src/vbox/vbox_tmpl.c b/src/vbox/vbox_tmpl.c
index 8241d34..c38472f 100644
--- a/src/vbox/vbox_tmpl.c
+++ b/src/vbox/vbox_tmpl.c
@@ -2066,7 +2066,7 @@ vboxDomainGetMaxVcpus(virDomainPtr dom)
                                          VIR_DOMAIN_VCPU_MAXIMUM));
 }
 
-static char *vboxDomainDumpXML(virDomainPtr dom, int flags) {
+static char *vboxDomainGetXMLDesc(virDomainPtr dom, int flags) {
     VBOX_OBJECT_CHECK(dom->conn, char *, NULL);
     virDomainDefPtr def  = NULL;
     IMachine *machine    = NULL;
@@ -5638,8 +5638,8 @@ cleanup:
 }
 
 static char *
-vboxDomainSnapshotDumpXML(virDomainSnapshotPtr snapshot,
-                          unsigned int flags)
+vboxDomainSnapshotGetXMLDesc(virDomainSnapshotPtr snapshot,
+                             unsigned int flags)
 {
     virDomainPtr dom = snapshot->domain;
     VBOX_OBJECT_CHECK(dom->conn, char *, NULL);
@@ -7510,7 +7510,7 @@ static int vboxNetworkDestroy(virNetworkPtr network) {
     return vboxNetworkUndefineDestroy(network, false);
 }
 
-static char *vboxNetworkDumpXML(virNetworkPtr network, int flags ATTRIBUTE_UNUSED) {
+static char *vboxNetworkGetXMLDesc(virNetworkPtr network, int flags ATTRIBUTE_UNUSED) {
     VBOX_OBJECT_HOST_CHECK(network->conn, char *, NULL);
     virNetworkDefPtr def  = NULL;
     virNetworkIpDefPtr ipdef = NULL;
@@ -8577,7 +8577,7 @@ virDriver NAME(Driver) = {
     vboxDomainGetMaxVcpus, /* domainGetMaxVcpus */
     NULL, /* domainGetSecurityLabel */
     NULL, /* nodeGetSecurityModel */
-    vboxDomainDumpXML, /* domainDumpXML */
+    vboxDomainGetXMLDesc, /* domainGetXMLDesc */
     NULL, /* domainXMLFromNative */
     NULL, /* domainXMLToNative */
     vboxListDefinedDomains, /* listDefinedDomains */
@@ -8642,7 +8642,7 @@ virDriver NAME(Driver) = {
     NULL, /* domainHasManagedSaveImage */
     NULL, /* domainManagedSaveRemove */
     vboxDomainSnapshotCreateXML, /* domainSnapshotCreateXML */
-    vboxDomainSnapshotDumpXML, /* domainSnapshotDumpXML */
+    vboxDomainSnapshotGetXMLDesc, /* domainSnapshotGetXMLDesc */
     vboxDomainSnapshotNum, /* domainSnapshotNum */
     vboxDomainSnapshotListNames, /* domainSnapshotListNames */
     vboxDomainSnapshotLookupByName, /* domainSnapshotLookupByName */
@@ -8669,7 +8669,7 @@ virNetworkDriver NAME(NetworkDriver) = {
     .networkUndefine        = vboxNetworkUndefine,
     .networkCreate          = vboxNetworkCreate,
     .networkDestroy         = vboxNetworkDestroy,
-    .networkDumpXML         = vboxNetworkDumpXML,
+    .networkGetXMLDesc      = vboxNetworkGetXMLDesc,
     .networkGetBridgeName   = NULL,
     .networkGetAutostart    = NULL,
     .networkSetAutostart    = NULL
diff --git a/src/vmware/vmware_driver.c b/src/vmware/vmware_driver.c
index bbfb1a4..27e8cef 100644
--- a/src/vmware/vmware_driver.c
+++ b/src/vmware/vmware_driver.c
@@ -789,7 +789,7 @@ vmwareDomainIsPersistent(virDomainPtr dom)
 
 
 static char *
-vmwareDomainDumpXML(virDomainPtr dom, int flags)
+vmwareDomainGetXMLDesc(virDomainPtr dom, int flags)
 {
     struct vmware_driver *driver = dom->conn->privateData;
     virDomainObjPtr vm;
@@ -942,7 +942,7 @@ static virDriver vmwareDriver = {
     NULL,                       /* domainGetMaxVcpus */
     NULL,                       /* domainGetSecurityLabel */
     NULL,                       /* nodeGetSecurityModel */
-    vmwareDomainDumpXML,        /* domainDumpXML */
+    vmwareDomainGetXMLDesc,     /* domainGetXMLDesc */
     NULL,                       /* domainXmlFromNative */
     NULL,                       /* domainXmlToNative */
     vmwareListDefinedDomains,   /* listDefinedDomains */
@@ -997,7 +997,7 @@ static virDriver vmwareDriver = {
     NULL,                       /* domainHasManagedSaveImage */
     NULL,                       /* domainManagedSaveRemove */
     NULL,                       /* domainSnapshotCreateXML */
-    NULL,                       /* domainSnapshotDumpXML */
+    NULL,                       /* domainSnapshotGetXMLDesc */
     NULL,                       /* domainSnapshotNum */
     NULL,                       /* domainSnapshotListNames */
     NULL,                       /* domainSnapshotLookupByName */
diff --git a/src/xen/xen_driver.c b/src/xen/xen_driver.c
index dd94fbc..96b6809 100644
--- a/src/xen/xen_driver.c
+++ b/src/xen/xen_driver.c
@@ -1192,20 +1192,20 @@ xenUnifiedDomainGetMaxVcpus (virDomainPtr dom)
 }
 
 static char *
-xenUnifiedDomainDumpXML (virDomainPtr dom, int flags)
+xenUnifiedDomainGetXMLDesc(virDomainPtr dom, int flags)
 {
     GET_PRIVATE(dom->conn);
 
     if (dom->id == -1 && priv->xendConfigVersion < 3 ) {
         if (priv->opened[XEN_UNIFIED_XM_OFFSET])
-            return xenXMDomainDumpXML(dom, flags);
+            return xenXMDomainGetXMLDesc(dom, flags);
     } else {
         if (priv->opened[XEN_UNIFIED_XEND_OFFSET]) {
             char *cpus, *res;
             xenUnifiedLock(priv);
             cpus = xenDomainUsedCpus(dom);
             xenUnifiedUnlock(priv);
-            res = xenDaemonDomainDumpXML(dom, flags, cpus);
+            res = xenDaemonDomainGetXMLDesc(dom, flags, cpus);
             VIR_FREE(cpus);
             return(res);
         }
@@ -1374,7 +1374,7 @@ xenUnifiedDomainMigrateFinish (virConnectPtr dconn,
     }
 
     if (flags & VIR_MIGRATE_PERSIST_DEST) {
-        domain_xml = xenDaemonDomainDumpXML (dom, 0, NULL);
+        domain_xml = xenDaemonDomainGetXMLDesc(dom, 0, NULL);
         if (! domain_xml) {
             xenUnifiedError(VIR_ERR_MIGRATE_PERSIST_FAILED,
                             "%s", _("failed to get XML representation of migrated domain"));
@@ -2143,7 +2143,7 @@ static virDriver xenUnifiedDriver = {
     xenUnifiedDomainGetMaxVcpus, /* domainGetMaxVcpus */
     NULL, /* domainGetSecurityLabel */
     NULL, /* nodeGetSecurityModel */
-    xenUnifiedDomainDumpXML, /* domainDumpXML */
+    xenUnifiedDomainGetXMLDesc, /* domainGetXMLDesc */
     xenUnifiedDomainXMLFromNative, /* domainXMLFromNative */
     xenUnifiedDomainXMLToNative, /* domainXMLToNative */
     xenUnifiedListDefinedDomains, /* listDefinedDomains */
@@ -2198,7 +2198,7 @@ static virDriver xenUnifiedDriver = {
     NULL, /* domainHasManagedSaveImage */
     NULL, /* domainManagedSaveRemove */
     NULL, /* domainSnapshotCreateXML */
-    NULL, /* domainSnapshotDumpXML */
+    NULL, /* domainSnapshotGetXMLDesc */
     NULL, /* domainSnapshotNum */
     NULL, /* domainSnapshotListNames */
     NULL, /* domainSnapshotLookupByName */
diff --git a/src/xen/xend_internal.c b/src/xen/xend_internal.c
index a4420d8..38a2586 100644
--- a/src/xen/xend_internal.c
+++ b/src/xen/xend_internal.c
@@ -1813,7 +1813,7 @@ cleanup:
 
 
 /**
- * xenDaemonDomainDumpXML:
+ * xenDaemonDomainGetXMLDesc:
  * @domain: a domain object
  * @flags: potential dump flags
  * @cpus: list of cpu the domain is pinned to.
@@ -1824,7 +1824,7 @@ cleanup:
  *         the caller must free() the returned value.
  */
 char *
-xenDaemonDomainDumpXML(virDomainPtr domain, int flags, const char *cpus)
+xenDaemonDomainGetXMLDesc(virDomainPtr domain, int flags, const char *cpus)
 {
     xenUnifiedPrivatePtr priv;
     virDomainDefPtr def;
diff --git a/src/xen/xend_internal.h b/src/xen/xend_internal.h
index 805cf91..22393b0 100644
--- a/src/xen/xend_internal.h
+++ b/src/xen/xend_internal.h
@@ -110,7 +110,7 @@ int xenDaemonDomainRestore(virConnectPtr conn, const char *filename);
 int xenDaemonDomainSetMemory(virDomainPtr domain, unsigned long memory);
 int xenDaemonDomainSetMaxMemory(virDomainPtr domain, unsigned long memory);
 int xenDaemonDomainGetInfo(virDomainPtr domain, virDomainInfoPtr info);
-char *xenDaemonDomainDumpXML(virDomainPtr domain, int flags, const char *cpus);
+char *xenDaemonDomainGetXMLDesc(virDomainPtr domain, int flags, const char *cpus);
 unsigned long xenDaemonDomainGetMaxMemory(virDomainPtr domain);
 char **xenDaemonListDomainsOld(virConnectPtr xend);
 
diff --git a/src/xen/xm_internal.c b/src/xen/xm_internal.c
index 07a0c0f..69e14c3 100644
--- a/src/xen/xm_internal.c
+++ b/src/xen/xm_internal.c
@@ -513,7 +513,7 @@ error:
  * Turn a config record into a lump of XML describing the
  * domain, suitable for later feeding for virDomainCreateXML
  */
-char *xenXMDomainDumpXML(virDomainPtr domain, int flags) {
+char *xenXMDomainGetXMLDesc(virDomainPtr domain, int flags) {
     xenUnifiedPrivatePtr priv;
     const char *filename;
     xenXMConfCachePtr entry;
diff --git a/src/xen/xm_internal.h b/src/xen/xm_internal.h
index 695bb3e..5a070b1 100644
--- a/src/xen/xm_internal.h
+++ b/src/xen/xm_internal.h
@@ -40,7 +40,7 @@ virDrvOpenStatus xenXMOpen(virConnectPtr conn, virConnectAuthPtr auth, int flags
 int xenXMClose(virConnectPtr conn);
 const char *xenXMGetType(virConnectPtr conn);
 int xenXMDomainGetInfo(virDomainPtr domain, virDomainInfoPtr info);
-char *xenXMDomainDumpXML(virDomainPtr domain, int flags);
+char *xenXMDomainGetXMLDesc(virDomainPtr domain, int flags);
 int xenXMDomainSetMemory(virDomainPtr domain, unsigned long memory);
 int xenXMDomainSetMaxMemory(virDomainPtr domain, unsigned long memory);
 unsigned long xenXMDomainGetMaxMemory(virDomainPtr domain);
diff --git a/src/xenapi/xenapi_driver.c b/src/xenapi/xenapi_driver.c
index 3fbdcc6..64d53b8 100644
--- a/src/xenapi/xenapi_driver.c
+++ b/src/xenapi/xenapi_driver.c
@@ -1224,13 +1224,13 @@ xenapiDomainGetMaxVcpus (virDomainPtr dom)
 }
 
 /*
- * xenapiDomainDumpXML
+ * xenapiDomainGetXMLDesc
  *
  *
  * Returns XML string of the domain configuration on success or -1 in case of error
  */
 static char *
-xenapiDomainDumpXML (virDomainPtr dom, int flags ATTRIBUTE_UNUSED)
+xenapiDomainGetXMLDesc(virDomainPtr dom, int flags ATTRIBUTE_UNUSED)
 {
     xen_vm vm=NULL;
     xen_vm_set *vms;
@@ -1824,7 +1824,7 @@ static virDriver xenapiDriver = {
     xenapiDomainGetMaxVcpus, /* domainGetMaxVcpus */
     NULL, /* domainGetSecurityLabel */
     NULL, /* nodeGetSecurityModel */
-    xenapiDomainDumpXML, /* domainDumpXML */
+    xenapiDomainGetXMLDesc, /* domainGetXMLDesc */
     NULL, /* domainXmlFromNative */
     NULL, /* domainXmlToNative */
     xenapiListDefinedDomains, /* listDefinedDomains */
@@ -1879,7 +1879,7 @@ static virDriver xenapiDriver = {
     NULL, /* domainHasManagedSaveImage */
     NULL, /* domainManagedSaveRemove */
     NULL, /* domainSnapshotCreateXML */
-    NULL, /* domainSnapshotDumpXML */
+    NULL, /* domainSnapshotGetXMLDesc */
     NULL, /* domainSnapshotNum */
     NULL, /* domainSnapshotListNames */
     NULL, /* domainSnapshotLookupByName */
-- 
1.7.0.4




More information about the libvir-list mailing list