[libvirt] [PATCH 5/7] remote: Implement virDomainInterfacesAddresses

Michal Privoznik mprivozn at redhat.com
Fri Jun 8 08:04:36 UTC 2012


---
 daemon/remote.c              |  122 ++++++++++++++++++++++++++++++++++++++++++
 src/remote/remote_driver.c   |   95 ++++++++++++++++++++++++++++++++
 src/remote/remote_protocol.x |   25 ++++++++-
 3 files changed, 241 insertions(+), 1 deletions(-)

diff --git a/daemon/remote.c b/daemon/remote.c
index a02c09b..029b695 100644
--- a/daemon/remote.c
+++ b/daemon/remote.c
@@ -3907,3 +3907,125 @@ no_memory:
     virReportOOMError();
     return -1;
 }
+
+static int
+remoteSerializeDomainInterfacePtr(virDomainInterfacePtr ifaces,
+                                  unsigned int ifaces_count,
+                                  remote_domain_interfaces_addresses_ret *ret)
+{
+    int i, j;
+
+    if (VIR_ALLOC_N(ret->ifaces.ifaces_val, ifaces_count) < 0)
+        goto no_memory;
+
+    ret->ifaces.ifaces_len = ifaces_count;
+
+    for (i = 0; i < ifaces_count; i++) {
+        virDomainInterfacePtr iface = &(ifaces[i]);
+        remote_domain_interface *iface_ret = &(ret->ifaces.ifaces_val[i]);
+
+        if (!(iface_ret->name = strdup(iface->name)))
+            goto no_memory;
+
+        if (iface->hwaddr) {
+            char **hwaddr_p = NULL;
+            if (VIR_ALLOC(hwaddr_p) < 0)
+                goto no_memory;
+            *hwaddr_p = strdup(iface->hwaddr);
+            if (!hwaddr_p)
+                goto no_memory;
+
+            iface_ret->hwaddr = hwaddr_p;
+        }
+
+        if (VIR_ALLOC_N(iface_ret->ip_addrs.ip_addrs_val,
+                        iface->ip_addrs_count) < 0)
+            goto no_memory;
+
+        iface_ret->ip_addrs.ip_addrs_len = iface->ip_addrs_count;
+
+        for (j = 0; j < iface->ip_addrs_count; j++) {
+            virDomainIPAddressPtr ip_addr = &(iface->ip_addrs[j]);
+            remote_domain_ip_addr *ip_addr_ret =
+                &(iface_ret->ip_addrs.ip_addrs_val[j]);
+
+            if (!(ip_addr_ret->addr = strdup(ip_addr->addr)))
+                goto no_memory;
+
+            ip_addr_ret->prefix = ip_addr->prefix;
+            ip_addr_ret->type = ip_addr->type;
+        }
+    }
+
+    return 0;
+
+no_memory:
+    if (ret->ifaces.ifaces_val) {
+        for (i = 0; i < ifaces_count; i++) {
+            remote_domain_interface *iface_ret = &(ret->ifaces.ifaces_val[i]);
+            VIR_FREE(iface_ret->name);
+            VIR_FREE(iface_ret->hwaddr);
+            for (j = 0; j < iface_ret->ip_addrs.ip_addrs_len; j++) {
+                remote_domain_ip_addr *ip_addr =
+                    &(iface_ret->ip_addrs.ip_addrs_val[j]);
+                VIR_FREE(ip_addr->addr);
+            }
+        }
+        VIR_FREE(ret->ifaces.ifaces_val);
+    }
+    virReportOOMError();
+    return -1;
+}
+
+static int
+remoteDispatchDomainInterfacesAddresses(
+    virNetServerPtr server ATTRIBUTE_UNUSED,
+    virNetServerClientPtr client,
+    virNetMessagePtr msg ATTRIBUTE_UNUSED,
+    virNetMessageErrorPtr rerr,
+    remote_domain_interfaces_addresses_args *args,
+    remote_domain_interfaces_addresses_ret *ret)
+{
+    int rv = -1;
+    virDomainPtr dom = NULL;
+    virDomainInterfacePtr ifaces = NULL;
+    unsigned int ifaces_count = 0;
+    struct daemonClientPrivate *priv =
+        virNetServerClientGetPrivateData(client);
+
+    if (!priv->conn) {
+        virNetError(VIR_ERR_INTERNAL_ERROR, "%s", _("connection not open"));
+        goto cleanup;
+    }
+
+    if (!(dom = get_nonnull_domain(priv->conn, args->dom)))
+        goto cleanup;
+
+    if (virDomainInterfacesAddresses(dom, &ifaces,
+                                     &ifaces_count, args->flags) < 0)
+        goto cleanup;
+
+    if (remoteSerializeDomainInterfacePtr(ifaces, ifaces_count, ret) < 0)
+        goto cleanup;
+
+    rv = 0;
+
+cleanup:
+    if (rv < 0)
+        virNetMessageSaveError(rerr);
+    if (dom)
+        virDomainFree(dom);
+    if (ifaces) {
+        unsigned int i, j;
+
+        for (i = 0; i < ifaces_count; i++) {
+            VIR_FREE(ifaces[i].name);
+            VIR_FREE(ifaces[i].hwaddr);
+            for (j = 0; j < ifaces[i].ip_addrs_count; j++)
+                VIR_FREE(ifaces[i].ip_addrs[j].addr);
+            VIR_FREE(ifaces[i].ip_addrs);
+        }
+        VIR_FREE(ifaces);
+    }
+    return rv;
+}
diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c
index 299cd69..43d7049 100644
--- a/src/remote/remote_driver.c
+++ b/src/remote/remote_driver.c
@@ -4810,6 +4810,100 @@ done:
     return rv;
 }
 
+static int
+remoteDomainInterfacesAddresses(virDomainPtr dom,
+                                virDomainInterfacePtr *ifaces,
+                                unsigned int *ifaces_count,
+                                unsigned int flags)
+{
+    int rv = -1;
+    remote_domain_interfaces_addresses_args args;
+    remote_domain_interfaces_addresses_ret ret;
+    struct private_data *priv = dom->conn->privateData;
+    int i, j;
+
+    memset(&ret, 0, sizeof(ret));
+    args.flags = flags;
+    make_nonnull_domain(&args.dom, dom);
+
+    remoteDriverLock(priv);
+
+    if (call(dom->conn, priv, 0, REMOTE_PROC_DOMAIN_INTERFACES_ADDRESSES,
+             (xdrproc_t)xdr_remote_domain_interfaces_addresses_args,
+             (char *)&args,
+             (xdrproc_t)xdr_remote_domain_interfaces_addresses_ret,
+             (char *)&ret) == -1) {
+        goto done;
+    }
+
+    if (ret.ifaces.ifaces_len &&
+        VIR_ALLOC_N(*ifaces, ret.ifaces.ifaces_len) < 0) {
+        virReportOOMError();
+        goto cleanup;
+    }
+
+    *ifaces_count = ret.ifaces.ifaces_len;
+
+    for (i = 0; i < *ifaces_count; i++) {
+        virDomainInterfacePtr iface = &((*ifaces)[i]);
+        remote_domain_interface *iface_ret = &(ret.ifaces.ifaces_val[i]);
+
+        if (!(iface->name = strdup(iface_ret->name))) {
+            virReportOOMError();
+            goto cleanup;
+        }
+
+        if (iface_ret->hwaddr &&
+            !(iface->hwaddr = strdup(*iface_ret->hwaddr))) {
+            virReportOOMError();
+            goto cleanup;
+        }
+
+        iface->ip_addrs_count = iface_ret->ip_addrs.ip_addrs_len;
+
+        if (iface->ip_addrs_count) {
+            if (VIR_ALLOC_N(iface->ip_addrs, iface->ip_addrs_count) < 0) {
+                virReportOOMError();
+                goto cleanup;
+            }
+
+            for (j = 0; j < iface->ip_addrs_count; j++) {
+                virDomainIPAddressPtr ip_addr = &(iface->ip_addrs[j]);
+                remote_domain_ip_addr *ip_addr_ret = &(iface_ret->ip_addrs.ip_addrs_val[j]);
+
+                if (!(ip_addr->addr = strdup(ip_addr_ret->addr))) {
+                    virReportOOMError();
+                    goto cleanup;
+                }
+                ip_addr->prefix = ip_addr_ret->prefix;
+                ip_addr->type = ip_addr_ret->type;
+            }
+        }
+    }
+
+    rv = 0;
+
+cleanup:
+    if (rv < 0) {
+        for (i = 0; i < *ifaces_count; i++) {
+            VIR_FREE((*ifaces)[i].name);
+            VIR_FREE((*ifaces)[i].hwaddr);
+            for (j = 0; j < (*ifaces)[i].ip_addrs_count; j++)
+                VIR_FREE((*ifaces)[i].ip_addrs[j].addr);
+            VIR_FREE((*ifaces)[i].ip_addrs);
+        }
+        VIR_FREE(*ifaces);
+    }
+    xdr_free((xdrproc_t)xdr_remote_domain_interfaces_addresses_ret,
+             (char *) &ret);
+done:
+    remoteDriverUnlock(priv);
+    return rv;
+}
+
+
+
+
 static void
 remoteDomainEventQueue(struct private_data *priv, virDomainEventPtr event)
 {
@@ -5111,6 +5205,7 @@ static virDriver remote_driver = {
     .domainGetDiskErrors = remoteDomainGetDiskErrors, /* 0.9.10 */
     .domainSetMetadata = remoteDomainSetMetadata, /* 0.9.10 */
     .domainGetMetadata = remoteDomainGetMetadata, /* 0.9.10 */
+    .domainInterfacesAddresses = remoteDomainInterfacesAddresses, /* 0.9.13 */
 };
 
 static virNetworkDriver network_driver = {
diff --git a/src/remote/remote_protocol.x b/src/remote/remote_protocol.x
index d8a6576..6f1949d 100644
--- a/src/remote/remote_protocol.x
+++ b/src/remote/remote_protocol.x
@@ -653,6 +653,27 @@ struct remote_domain_get_interface_parameters_ret {
     int nparams;
 };
 
+struct remote_domain_ip_addr {
+    int type;
+    remote_nonnull_string addr;
+    int prefix;
+};
+
+struct remote_domain_interface {
+    remote_nonnull_string name;
+    remote_string hwaddr;
+    remote_domain_ip_addr ip_addrs<>;
+};
+
+struct remote_domain_interfaces_addresses_args {
+    remote_nonnull_domain dom;
+    unsigned int flags;
+};
+
+struct remote_domain_interfaces_addresses_ret {
+    remote_domain_interface ifaces<>;
+};
+
 struct remote_domain_memory_stats_args {
     remote_nonnull_domain dom;
     unsigned int maxStats;
@@ -2782,7 +2803,9 @@ enum remote_procedure {
     REMOTE_PROC_DOMAIN_PM_WAKEUP = 267, /* autogen autogen */
     REMOTE_PROC_DOMAIN_EVENT_TRAY_CHANGE = 268, /* autogen autogen */
     REMOTE_PROC_DOMAIN_EVENT_PMWAKEUP = 269, /* autogen autogen */
-    REMOTE_PROC_DOMAIN_EVENT_PMSUSPEND = 270 /* autogen autogen */
+    REMOTE_PROC_DOMAIN_EVENT_PMSUSPEND = 270, /* autogen autogen */
+
+    REMOTE_PROC_DOMAIN_INTERFACES_ADDRESSES = 271 /* skipgen skipgen */
 
     /*
      * Notice how the entries are grouped in sets of 10 ?
-- 
1.7.8.5




More information about the libvir-list mailing list