vswitchd: Factor code to configure netdevs out of iface_create().
authorBen Pfaff <blp@nicira.com>
Tue, 24 Apr 2012 23:28:18 +0000 (16:28 -0700)
committerBen Pfaff <blp@nicira.com>
Wed, 25 Apr 2012 01:08:41 +0000 (18:08 -0700)
An upcoming patch will need the same code in another function.

Signed-off-by: Ben Pfaff <blp@nicira.com>
vswitchd/bridge.c

index 208396b3218476e1634b95a42249549966662438..7495d728bca3a2dc9db474ac05d115daede6920c 100644 (file)
@@ -1144,6 +1144,29 @@ iface_set_ofp_port(struct iface *iface, int ofp_port)
     iface_set_ofport(iface->cfg, ofp_port);
 }
 
+/* Configures 'netdev' based on the "options" column in 'iface_cfg'.
+ * Returns 0 if successful, otherwise a positive errno value. */
+static int
+iface_set_netdev_config(const struct ovsrec_interface *iface_cfg,
+                        struct netdev *netdev)
+{
+    struct shash args;
+    int error;
+
+    shash_init(&args);
+    shash_from_ovs_idl_map(iface_cfg->key_options,
+                           iface_cfg->value_options,
+                           iface_cfg->n_options, &args);
+    error = netdev_set_config(netdev, &args);
+    shash_destroy(&args);
+
+    if (error) {
+        VLOG_WARN("could not configure network device %s (%s)",
+                  iface_cfg->name, strerror(error));
+    }
+    return error;
+}
+
 static void
 bridge_ofproto_port_del(struct bridge *br, struct ofproto_port ofproto_port)
 {
@@ -1279,18 +1302,8 @@ iface_create(struct bridge *br, struct if_cfg *if_cfg, int ofp_port)
 
     /* Configure the netdev. */
     if (iface->netdev) {
-        struct shash args;
-
-        shash_init(&args);
-        shash_from_ovs_idl_map(iface->cfg->key_options,
-                               iface->cfg->value_options,
-                               iface->cfg->n_options, &args);
-        error = netdev_set_config(iface->netdev, &args);
-        shash_destroy(&args);
-
+        int error = iface_set_netdev_config(iface->cfg, iface->netdev);
         if (error) {
-            VLOG_WARN("could not configure network device %s (%s)",
-                      iface->name, strerror(error));
             netdev_close(iface->netdev);
             iface->netdev = NULL;
         }