vlog: Introduce VLOG_DEFINE_THIS_MODULE for declaring vlog module in use.
[sliver-openvswitch.git] / vswitchd / bridge.c
index d4a08b9..be0e875 100644 (file)
@@ -46,6 +46,7 @@
 #include "ofpbuf.h"
 #include "ofproto/netflow.h"
 #include "ofproto/ofproto.h"
+#include "ovsdb-data.h"
 #include "packets.h"
 #include "poll-loop.h"
 #include "port-array.h"
 #include "vconn.h"
 #include "vswitchd/vswitch-idl.h"
 #include "xenserver.h"
+#include "vlog.h"
 #include "xtoxll.h"
 #include "sflow_api.h"
 
-#define THIS_MODULE VLM_bridge
-#include "vlog.h"
+VLOG_DEFINE_THIS_MODULE(bridge)
 
 struct dst {
     uint16_t vlan;
@@ -104,6 +105,7 @@ struct mirror {
     struct bridge *bridge;
     size_t idx;
     char *name;
+    struct uuid uuid;           /* UUID of this "mirror" record in database. */
 
     /* Selection criteria. */
     struct shash src_ports;     /* Name is port name; data is always NULL. */
@@ -239,7 +241,7 @@ static void port_update_bond_compat(struct port *);
 static void port_update_vlan_compat(struct port *);
 static void port_update_bonding(struct port *);
 
-static struct mirror *mirror_create(struct bridge *, const char *name);
+static void mirror_create(struct bridge *, struct ovsrec_mirror *);
 static void mirror_destroy(struct mirror *);
 static void mirror_reconfigure(struct bridge *);
 static void mirror_reconfigure_one(struct mirror *, struct ovsrec_mirror *);
@@ -645,9 +647,7 @@ bridge_reconfigure(const struct ovsrec_open_vswitch *ovs_cfg)
         shash_init(&cur_ifaces);
         for (i = 0; i < n_dpif_ports; i++) {
             const char *name = dpif_ports[i].devname;
-            if (!shash_find(&cur_ifaces, name)) {
-                shash_add(&cur_ifaces, name, NULL);
-            }
+            shash_add_once(&cur_ifaces, name, NULL);
         }
         free(dpif_ports);
 
@@ -845,25 +845,25 @@ bridge_reconfigure(const struct ovsrec_open_vswitch *ovs_cfg)
 }
 
 static const char *
-get_ovsrec_key_value(const char *key, char **keys, char **values, size_t n)
+get_ovsrec_key_value(const struct ovsdb_idl_row *row,
+                     const struct ovsdb_idl_column *column,
+                     const char *key)
 {
-    size_t i;
-
-    for (i = 0; i < n; i++) {
-        if (!strcmp(keys[i], key)) {
-            return values[i];
-        }
-    }
-    return NULL;
+    const struct ovsdb_datum *datum;
+    union ovsdb_atom atom;
+    unsigned int idx;
+
+    datum = ovsdb_idl_get(row, column, OVSDB_TYPE_STRING, OVSDB_TYPE_STRING);
+    atom.string = (char *) key;
+    idx = ovsdb_datum_find_key(datum, &atom, OVSDB_TYPE_STRING);
+    return idx == UINT_MAX ? NULL : datum->values[idx].string;
 }
 
 static const char *
 bridge_get_other_config(const struct ovsrec_bridge *br_cfg, const char *key)
 {
-    return get_ovsrec_key_value(key,
-                                br_cfg->key_other_config,
-                                br_cfg->value_other_config,
-                                br_cfg->n_other_config);
+    return get_ovsrec_key_value(&br_cfg->header_,
+                                &ovsrec_bridge_col_other_config, key);
 }
 
 static void
@@ -3239,10 +3239,10 @@ static const char *
 get_port_other_config(const struct ovsrec_port *port, const char *key,
                       const char *default_value)
 {
-    const char *value = get_ovsrec_key_value(key,
-                                             port->key_other_config,
-                                             port->value_other_config,
-                                             port->n_other_config);
+    const char *value;
+
+    value = get_ovsrec_key_value(&port->header_, &ovsrec_port_col_other_config,
+                                 key);
     return value ? value : default_value;
 }
 
@@ -3756,27 +3756,16 @@ shash_from_ovs_idl_map(char **keys, char **values, size_t n,
 
 struct iface_delete_queues_cbdata {
     struct netdev *netdev;
-    const int64_t *queue_ids;
-    size_t n_queue_ids;
+    const struct ovsdb_datum *queues;
 };
 
 static bool
-queue_ids_include(const int64_t *ids, size_t n, int64_t target)
+queue_ids_include(const struct ovsdb_datum *queues, int64_t target)
 {
-    size_t low = 0;
-    size_t high = n;
-
-    while (low < high) {
-        size_t mid = low + (high - low) / 2;
-        if (target > ids[mid]) {
-            high = mid;
-        } else if (target < ids[mid]) {
-            low = mid + 1;
-        } else {
-            return true;
-        }
-    }
-    return false;
+    union ovsdb_atom atom;
+
+    atom.integer = target;
+    return ovsdb_datum_find_key(queues, &atom, OVSDB_TYPE_INTEGER) != UINT_MAX;
 }
 
 static void
@@ -3785,7 +3774,7 @@ iface_delete_queues(unsigned int queue_id,
 {
     struct iface_delete_queues_cbdata *cbdata = cbdata_;
 
-    if (!queue_ids_include(cbdata->queue_ids, cbdata->n_queue_ids, queue_id)) {
+    if (!queue_ids_include(cbdata->queues, queue_id)) {
         netdev_delete_queue(cbdata->netdev, queue_id);
     }
 }
@@ -3808,8 +3797,8 @@ iface_update_qos(struct iface *iface, const struct ovsrec_qos *qos)
 
         /* Deconfigure queues that were deleted. */
         cbdata.netdev = iface->netdev;
-        cbdata.queue_ids = qos->key_queues;
-        cbdata.n_queue_ids = qos->n_queues;
+        cbdata.queues = ovsrec_qos_get_queues(qos, OVSDB_TYPE_INTEGER,
+                                              OVSDB_TYPE_UUID);
         netdev_dump_queues(iface->netdev, iface_delete_queues, &cbdata);
 
         /* Configure queues for 'iface'. */
@@ -3828,50 +3817,51 @@ iface_update_qos(struct iface *iface, const struct ovsrec_qos *qos)
 \f
 /* Port mirroring. */
 
+static struct mirror *
+mirror_find_by_uuid(struct bridge *br, const struct uuid *uuid)
+{
+    int i;
+
+    for (i = 0; i < MAX_MIRRORS; i++) {
+        struct mirror *m = br->mirrors[i];
+        if (m && uuid_equals(uuid, &m->uuid)) {
+            return m;
+        }
+    }
+    return NULL;
+}
+
 static void
 mirror_reconfigure(struct bridge *br)
 {
-    struct shash old_mirrors, new_mirrors;
-    struct shash_node *node;
     unsigned long *rspan_vlans;
     int i;
 
-    /* Collect old mirrors. */
-    shash_init(&old_mirrors);
+    /* Get rid of deleted mirrors. */
     for (i = 0; i < MAX_MIRRORS; i++) {
-        if (br->mirrors[i]) {
-            shash_add(&old_mirrors, br->mirrors[i]->name, br->mirrors[i]);
+        struct mirror *m = br->mirrors[i];
+        if (m) {
+            const struct ovsdb_datum *mc;
+            union ovsdb_atom atom;
+
+            mc = ovsrec_bridge_get_mirrors(br->cfg, OVSDB_TYPE_UUID);
+            atom.uuid = br->mirrors[i]->uuid;
+            if (ovsdb_datum_find_key(mc, &atom, OVSDB_TYPE_UUID) == UINT_MAX) {
+                mirror_destroy(m);
+            }
         }
     }
 
-    /* Collect new mirrors. */
-    shash_init(&new_mirrors);
+    /* Add new mirrors and reconfigure existing ones. */
     for (i = 0; i < br->cfg->n_mirrors; i++) {
         struct ovsrec_mirror *cfg = br->cfg->mirrors[i];
-        if (!shash_add_once(&new_mirrors, cfg->name, cfg)) {
-            VLOG_WARN("bridge %s: %s specified twice as mirror",
-                      br->name, cfg->name);
-        }
-    }
-
-    /* Get rid of deleted mirrors and add new mirrors. */
-    SHASH_FOR_EACH (node, &old_mirrors) {
-        if (!shash_find(&new_mirrors, node->name)) {
-            mirror_destroy(node->data);
-        }
-    }
-    SHASH_FOR_EACH (node, &new_mirrors) {
-        struct mirror *mirror = shash_find_data(&old_mirrors, node->name);
-        if (!mirror) {
-            mirror = mirror_create(br, node->name);
-            if (!mirror) {
-                break;
-            }
+        struct mirror *m = mirror_find_by_uuid(br, &cfg->header_.uuid);
+        if (m) {
+            mirror_reconfigure_one(m, cfg);
+        } else {
+            mirror_create(br, cfg);
         }
-        mirror_reconfigure_one(mirror, node->data);
     }
-    shash_destroy(&old_mirrors);
-    shash_destroy(&new_mirrors);
 
     /* Update port reserved status. */
     for (i = 0; i < br->n_ports; i++) {
@@ -3906,8 +3896,8 @@ mirror_reconfigure(struct bridge *br)
     }
 }
 
-static struct mirror *
-mirror_create(struct bridge *br, const char *name)
+static void
+mirror_create(struct bridge *br, struct ovsrec_mirror *cfg)
 {
     struct mirror *m;
     size_t i;
@@ -3915,21 +3905,21 @@ mirror_create(struct bridge *br, const char *name)
     for (i = 0; ; i++) {
         if (i >= MAX_MIRRORS) {
             VLOG_WARN("bridge %s: maximum of %d port mirrors reached, "
-                      "cannot create %s", br->name, MAX_MIRRORS, name);
-            return NULL;
+                      "cannot create %s", br->name, MAX_MIRRORS, cfg->name);
+            return;
         }
         if (!br->mirrors[i]) {
             break;
         }
     }
 
-    VLOG_INFO("created port mirror %s on bridge %s", name, br->name);
+    VLOG_INFO("created port mirror %s on bridge %s", cfg->name, br->name);
     bridge_flush(br);
 
     br->mirrors[i] = m = xzalloc(sizeof *m);
     m->bridge = br;
     m->idx = i;
-    m->name = xstrdup(name);
+    m->name = xstrdup(cfg->name);
     shash_init(&m->src_ports);
     shash_init(&m->dst_ports);
     m->vlans = NULL;
@@ -3937,7 +3927,7 @@ mirror_create(struct bridge *br, const char *name)
     m->out_vlan = -1;
     m->out_port = NULL;
 
-    return m;
+    mirror_reconfigure_one(m, cfg);
 }
 
 static void
@@ -3957,6 +3947,7 @@ mirror_destroy(struct mirror *m)
         free(m->vlans);
 
         m->bridge->mirrors[m->idx] = NULL;
+        free(m->name);
         free(m);
 
         bridge_flush(br);
@@ -4038,6 +4029,12 @@ mirror_reconfigure_one(struct mirror *m, struct ovsrec_mirror *cfg)
     int *vlans;
     size_t i;
 
+    /* Set name. */
+    if (strcmp(cfg->name, m->name)) {
+        free(m->name);
+        m->name = xstrdup(cfg->name);
+    }
+
     /* Get output port. */
     if (cfg->output_port) {
         out_port = port_lookup(m->bridge, cfg->output_port->name);