[PATCH 04/23] Refactor LXC NIC creation to allow reuse by hotplug code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: "Daniel P. Berrange" <berrange@xxxxxxxxxx>

The code for creating veth/macvlan devices is part of the
LXC process startup code. Refactor this a little and export
the methods to the rest of the LXC driver. This allows them
to be reused for NIC hotplug code

Signed-off-by: Daniel P. Berrange <berrange@xxxxxxxxxx>
---
 src/lxc/lxc_process.c | 102 +++++++++++++++++++++-----------------------------
 src/lxc/lxc_process.h |   8 ++++
 2 files changed, 50 insertions(+), 60 deletions(-)

diff --git a/src/lxc/lxc_process.c b/src/lxc/lxc_process.c
index 8b9d02f..72e1be3 100644
--- a/src/lxc/lxc_process.c
+++ b/src/lxc/lxc_process.c
@@ -293,14 +293,12 @@ static void virLXCProcessCleanup(virLXCDriverPtr driver,
 }
 
 
-static int virLXCProcessSetupInterfaceBridged(virConnectPtr conn,
-                                              virDomainDefPtr vm,
-                                              virDomainNetDefPtr net,
-                                              const char *brname,
-                                              unsigned int *nveths,
-                                              char ***veths)
+char *virLXCProcessSetupInterfaceBridged(virConnectPtr conn,
+                                         virDomainDefPtr vm,
+                                         virDomainNetDefPtr net,
+                                         const char *brname)
 {
-    int ret = -1;
+    char *ret = NULL;
     char *parentVeth;
     char *containerVeth = NULL;
     const virNetDevVPortProfilePtr vport = virDomainNetGetActualVirtPortProfile(net);
@@ -314,24 +312,17 @@ static int virLXCProcessSetupInterfaceBridged(virConnectPtr conn,
     if (net->ifname == NULL)
         net->ifname = parentVeth;
 
-    if (VIR_REALLOC_N(*veths, (*nveths)+1) < 0) {
-        virReportOOMError();
-        VIR_FREE(containerVeth);
-        goto cleanup;
-    }
-    (*veths)[(*nveths)] = containerVeth;
-    (*nveths)++;
-
     if (virNetDevSetMAC(containerVeth, &net->mac) < 0)
         goto cleanup;
 
-    if (vport && vport->virtPortType == VIR_NETDEV_VPORT_PROFILE_OPENVSWITCH)
-        ret = virNetDevOpenvswitchAddPort(brname, parentVeth, &net->mac,
-                                          vm->uuid, vport, virDomainNetGetActualVlan(net));
-    else
-        ret = virNetDevBridgeAddPort(brname, parentVeth);
-    if (ret < 0)
-        goto cleanup;
+    if (vport && vport->virtPortType == VIR_NETDEV_VPORT_PROFILE_OPENVSWITCH) {
+        if (virNetDevOpenvswitchAddPort(brname, parentVeth, &net->mac,
+                                        vm->uuid, vport, virDomainNetGetActualVlan(net)) < 0)
+            goto cleanup;
+    } else {
+        if (virNetDevBridgeAddPort(brname, parentVeth) < 0)
+            goto cleanup;
+    }
 
     if (virNetDevSetOnline(parentVeth, true) < 0)
         goto cleanup;
@@ -348,20 +339,18 @@ static int virLXCProcessSetupInterfaceBridged(virConnectPtr conn,
         virDomainConfNWFilterInstantiate(conn, vm->uuid, net) < 0)
         goto cleanup;
 
-    ret = 0;
+    ret = containerVeth;
 
 cleanup:
     return ret;
 }
 
 
-static int virLXCProcessSetupInterfaceDirect(virConnectPtr conn,
-                                             virDomainDefPtr def,
-                                             virDomainNetDefPtr net,
-                                             unsigned int *nveths,
-                                             char ***veths)
+char *virLXCProcessSetupInterfaceDirect(virConnectPtr conn,
+                                        virDomainDefPtr def,
+                                        virDomainNetDefPtr net)
 {
-    int ret = -1;
+    char *ret = NULL;
     char *res_ifname = NULL;
     virLXCDriverPtr driver = conn->privateData;
     virNetDevBandwidthPtr bw;
@@ -376,7 +365,7 @@ static int virLXCProcessSetupInterfaceDirect(virConnectPtr conn,
     if (bw) {
         virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
                        _("Unable to set network bandwidth on direct interfaces"));
-        return -1;
+        return NULL;
     }
 
     /* XXX how todo port profiles ?
@@ -390,15 +379,9 @@ static int virLXCProcessSetupInterfaceDirect(virConnectPtr conn,
     if (prof) {
         virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
                        _("Unable to set port profile on direct interfaces"));
-        return -1;
+        return NULL;
     }
 
-    if (VIR_REALLOC_N(*veths, (*nveths)+1) < 0) {
-        virReportOOMError();
-        return -1;
-    }
-    (*veths)[(*nveths)] = NULL;
-
     if (virNetDevMacVLanCreateWithVPortProfile(
             net->ifname, &net->mac,
             virDomainNetGetActualDirectDev(net),
@@ -411,10 +394,7 @@ static int virLXCProcessSetupInterfaceDirect(virConnectPtr conn,
             virDomainNetGetActualBandwidth(net)) < 0)
         goto cleanup;
 
-    (*veths)[(*nveths)] = res_ifname;
-    (*nveths)++;
-
-    ret = 0;
+    ret = res_ifname;
 
 cleanup:
     return ret;
@@ -436,13 +416,14 @@ cleanup:
  */
 static int virLXCProcessSetupInterfaces(virConnectPtr conn,
                                         virDomainDefPtr def,
-                                        unsigned int *nveths,
+                                        size_t *nveths,
                                         char ***veths)
 {
     int ret = -1;
     size_t i;
 
     for (i = 0 ; i < def->nnets ; i++) {
+        char *veth = NULL;
         /* If appropriate, grab a physical device from the configured
          * network's pool of devices, or resolve bridge device name
          * to the one defined in the network definition.
@@ -450,6 +431,11 @@ static int virLXCProcessSetupInterfaces(virConnectPtr conn,
         if (networkAllocateActualDevice(def->nets[i]) < 0)
             goto cleanup;
 
+        if (VIR_EXPAND_N(*veths, *nveths, 1) < 0) {
+            virReportOOMError();
+            goto cleanup;
+        }
+
         switch (virDomainNetGetActualType(def->nets[i])) {
         case VIR_DOMAIN_NET_TYPE_NETWORK: {
             virNetworkPtr network;
@@ -486,12 +472,10 @@ static int virLXCProcessSetupInterfaces(virConnectPtr conn,
             if (fail)
                 goto cleanup;
 
-            if (virLXCProcessSetupInterfaceBridged(conn,
-                                                   def,
-                                                   def->nets[i],
-                                                   brname,
-                                                   nveths,
-                                                   veths) < 0) {
+            if (!(veth = virLXCProcessSetupInterfaceBridged(conn,
+                                                            def,
+                                                            def->nets[i],
+                                                            brname))) {
                 VIR_FREE(brname);
                 goto cleanup;
             }
@@ -505,21 +489,17 @@ static int virLXCProcessSetupInterfaces(virConnectPtr conn,
                                _("No bridge name specified"));
                 goto cleanup;
             }
-            if (virLXCProcessSetupInterfaceBridged(conn,
-                                                   def,
-                                                   def->nets[i],
-                                                   brname,
-                                                   nveths,
-                                                   veths) < 0)
+            if (!(veth = virLXCProcessSetupInterfaceBridged(conn,
+                                                            def,
+                                                            def->nets[i],
+                                                            brname)))
                 goto cleanup;
         }   break;
 
         case VIR_DOMAIN_NET_TYPE_DIRECT:
-            if (virLXCProcessSetupInterfaceDirect(conn,
-                                                  def,
-                                                  def->nets[i],
-                                                  nveths,
-                                                  veths) < 0)
+            if (!(veth = virLXCProcessSetupInterfaceDirect(conn,
+                                                           def,
+                                                           def->nets[i])))
                 goto cleanup;
             break;
 
@@ -537,6 +517,8 @@ static int virLXCProcessSetupInterfaces(virConnectPtr conn,
                                ));
             goto cleanup;
         }
+
+        (*veths)[(*nveths)-1] = veth;
     }
 
     ret = 0;
@@ -918,7 +900,7 @@ int virLXCProcessStart(virConnectPtr conn,
     size_t i;
     char *logfile = NULL;
     int logfd = -1;
-    unsigned int nveths = 0;
+    size_t nveths = 0;
     char **veths = NULL;
     int handshakefds[2] = { -1, -1 };
     off_t pos = -1;
diff --git a/src/lxc/lxc_process.h b/src/lxc/lxc_process.h
index bac2c6c..779cc5f 100644
--- a/src/lxc/lxc_process.h
+++ b/src/lxc/lxc_process.h
@@ -47,4 +47,12 @@ void virLXCProcessAutostartAll(virLXCDriverPtr driver);
 int virLXCProcessReconnectAll(virLXCDriverPtr driver,
                               virDomainObjListPtr doms);
 
+char *virLXCProcessSetupInterfaceBridged(virConnectPtr conn,
+                                         virDomainDefPtr vm,
+                                         virDomainNetDefPtr net,
+                                         const char *brname);
+char *virLXCProcessSetupInterfaceDirect(virConnectPtr conn,
+                                        virDomainDefPtr def,
+                                        virDomainNetDefPtr net);
+
 #endif /* __LXC_PROCESS_H__ */
-- 
1.8.0.1

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]