stack->subfield.n_bits = stack->subfield.field->n_bits;
}
+static const struct sbrec_port_binding*
+get_localnet_port(struct hmap *local_datapaths, int64_t tunnel_key)
+{
+ struct local_datapath *ld;
+ ld = CONTAINER_OF(hmap_first_with_hash(local_datapaths, tunnel_key),
+ struct local_datapath, hmap_node);
+ return ld ? ld->localnet_port : NULL;
+}
+
void
physical_run(struct controller_ctx *ctx, enum mf_field_id mff_ovn_geneve,
const struct ovsrec_bridge *br_int, const char *this_chassis_id,
{
struct simap localvif_to_ofport = SIMAP_INITIALIZER(&localvif_to_ofport);
struct hmap tunnels = HMAP_INITIALIZER(&tunnels);
- struct simap localnet_to_ofport = SIMAP_INITIALIZER(&localnet_to_ofport);
for (int i = 0; i < br_int->n_ports; i++) {
const struct ovsrec_port *port_rec = br_int->ports[i];
* local logical port. */
bool is_patch = !strcmp(iface_rec->type, "patch");
if (is_patch && localnet) {
- simap_put(&localnet_to_ofport, localnet, ofport);
+ /* localnet patch ports can be handled just like VIFs. */
+ simap_put(&localvif_to_ofport, localnet, ofport);
break;
} else if (is_patch && logpatch) {
/* Logical patch ports can be handled just like VIFs. */
struct ofpbuf ofpacts;
ofpbuf_init(&ofpacts, 0);
- struct binding_elem {
- struct ovs_list list_elem;
- const struct sbrec_port_binding *binding;
- };
- /* The bindings for a given VLAN on a localnet port. */
- struct localnet_vlan {
- struct hmap_node node;
- int tag;
- struct ovs_list bindings;
- };
- /* A hash of localnet_vlans, hashed on VLAN ID, for a localnet port */
- struct localnet_bindings {
- ofp_port_t ofport;
- struct hmap vlans;
- };
- /* Maps from network name to "struct localnet_bindings". */
- struct shash localnet_inputs = SHASH_INITIALIZER(&localnet_inputs);
-
/* Set up flows in table 0 for physical-to-logical translation and in table
* 64 for logical-to-physical translation. */
const struct sbrec_port_binding *binding;
* - If the port is a VIF on the chassis we're managing, the
* OpenFlow port for the VIF. 'tun' will be NULL.
*
- * In this or the next case, for a container nested inside a VM
- * and accessible via a VLAN, 'tag' is the VLAN ID; otherwise
- * 'tag' is 0.
+ * The same logic handles logical patch ports, as well as
+ * localnet patch ports.
+ *
+ * For a container nested inside a VM and accessible via a VLAN,
+ * 'tag' is the VLAN ID; otherwise 'tag' is 0.
*
- * The same logic handles logical patch ports.
+ * For a localnet patch port, if a VLAN ID was configured, 'tag'
+ * is set to that VLAN ID; otherwise 'tag' is 0.
*
* - If the port is on a remote chassis, the OpenFlow port for a
* tunnel to the VIF's remote chassis. 'tun' identifies that
* tunnel.
- *
- * - If the port is a "localnet" port for a network that is
- * attached to the chassis we're managing, the OpenFlow port for
- * the localnet port (a patch port).
- *
- * The "localnet" port may be configured with a VLAN ID. If so,
- * 'tag' will be set to that VLAN ID; otherwise 'tag' is 0.
*/
int tag = 0;
ofp_port_t ofport;
- if (!strcmp(binding->type, "localnet")) {
- const char *network = smap_get(&binding->options, "network_name");
- if (!network) {
- continue;
- }
- ofport = u16_to_ofp(simap_get(&localnet_to_ofport, network));
- if (ofport && binding->tag) {
- tag = *binding->tag;
- }
- } else if (binding->parent_port && *binding->parent_port) {
+ bool is_remote = false;
+ if (binding->parent_port && *binding->parent_port) {
if (!binding->tag) {
continue;
}
} else {
ofport = u16_to_ofp(simap_get(&localvif_to_ofport,
binding->logical_port));
+ if (!strcmp(binding->type, "localnet") && ofport && binding->tag) {
+ tag = *binding->tag;
+ }
}
const struct chassis_tunnel *tun = NULL;
+ const struct sbrec_port_binding *localnet_port =
+ get_localnet_port(local_datapaths,
+ binding->datapath->tunnel_key);
if (!ofport) {
+ /* It is remote port, may be reached by tunnel or localnet port */
+ is_remote = true;
if (!binding->chassis) {
continue;
}
- tun = chassis_tunnel_find(&tunnels, binding->chassis->name);
- if (!tun) {
- continue;
+ if (localnet_port) {
+ ofport = u16_to_ofp(simap_get(&localvif_to_ofport,
+ localnet_port->logical_port));
+ if (!ofport) {
+ continue;
+ }
+ } else {
+ tun = chassis_tunnel_find(&tunnels, binding->chassis->name);
+ if (!tun) {
+ continue;
+ }
+ ofport = tun->ofport;
}
- ofport = tun->ofport;
}
struct match match;
- if (!tun) {
+ if (!is_remote) {
int zone_id = simap_get(ct_zones, binding->logical_port);
/* Packets that arrive from a vif can belong to a VM or
* to a container located inside that VM. Packets that
* input port, MFF_LOG_DATAPATH to the logical datapath, and
* resubmit into the logical ingress pipeline starting at table
* 16. */
- if (!strcmp(binding->type, "localnet")) {
- /* The same OpenFlow port may correspond to localnet ports
- * attached to more than one logical datapath, so keep track of
- * all associated bindings and add a flow at the end. */
-
- const char *network
- = smap_get(&binding->options, "network_name");
- struct localnet_bindings *ln_bindings;
- struct hmap_node *node;
- struct localnet_vlan *ln_vlan;
-
- ln_bindings = shash_find_data(&localnet_inputs, network);
- if (!ln_bindings) {
- ln_bindings = xmalloc(sizeof *ln_bindings);
- ln_bindings->ofport = ofport;
- hmap_init(&ln_bindings->vlans);
- shash_add(&localnet_inputs, network, ln_bindings);
- }
- node = hmap_first_with_hash(&ln_bindings->vlans, tag);
- if (node) {
- ASSIGN_CONTAINER(ln_vlan, node, node);
- } else {
- ln_vlan = xmalloc(sizeof *ln_vlan);
- ln_vlan->tag = tag;
- list_init(&ln_vlan->bindings);
- hmap_insert(&ln_bindings->vlans, &ln_vlan->node, tag);
- }
+ ofpbuf_clear(&ofpacts);
+ match_init_catchall(&match);
+ match_set_in_port(&match, ofport);
- struct binding_elem *b = xmalloc(sizeof *b);
- b->binding = binding;
- list_insert(&ln_vlan->bindings, &b->list_elem);
- } else {
- ofpbuf_clear(&ofpacts);
- match_init_catchall(&match);
- match_set_in_port(&match, ofport);
- if (tag) {
- match_set_dl_vlan(&match, htons(tag));
- }
+ /* Match a VLAN tag and strip it, including stripping priority tags
+ * (e.g. VLAN ID 0). In the latter case we'll add a second flow
+ * for frames that lack any 802.1Q header later. */
+ if (tag || !strcmp(binding->type, "localnet")) {
+ match_set_dl_vlan(&match, htons(tag));
+ ofpact_put_STRIP_VLAN(&ofpacts);
+ }
- if (zone_id) {
- put_load(zone_id, MFF_LOG_CT_ZONE, 0, 32, &ofpacts);
- }
+ /* Remember the size with just strip vlan added so far,
+ * as we're going to remove this with ofpbuf_pull() later. */
+ uint32_t ofpacts_orig_size = ofpacts.size;
- /* Set MFF_LOG_DATAPATH and MFF_LOG_INPORT. */
- put_load(binding->datapath->tunnel_key, MFF_LOG_DATAPATH, 0, 64,
- &ofpacts);
- put_load(binding->tunnel_key, MFF_LOG_INPORT, 0, 32,
- &ofpacts);
+ if (zone_id) {
+ put_load(zone_id, MFF_LOG_CT_ZONE, 0, 32, &ofpacts);
+ }
- /* Strip vlans. */
- if (tag) {
- ofpact_put_STRIP_VLAN(&ofpacts);
- }
+ /* Set MFF_LOG_DATAPATH and MFF_LOG_INPORT. */
+ put_load(binding->datapath->tunnel_key, MFF_LOG_DATAPATH, 0, 64,
+ &ofpacts);
+ put_load(binding->tunnel_key, MFF_LOG_INPORT, 0, 32,
+ &ofpacts);
- /* Resubmit to first logical ingress pipeline table. */
- put_resubmit(OFTABLE_LOG_INGRESS_PIPELINE, &ofpacts);
- ofctrl_add_flow(flow_table, OFTABLE_PHY_TO_LOG,
- tag ? 150 : 100, &match, &ofpacts);
+ /* Resubmit to first logical ingress pipeline table. */
+ put_resubmit(OFTABLE_LOG_INGRESS_PIPELINE, &ofpacts);
+ ofctrl_add_flow(flow_table, OFTABLE_PHY_TO_LOG,
+ tag ? 150 : 100, &match, &ofpacts);
+
+ if (!tag && !strcmp(binding->type, "localnet")) {
+ /* Add a second flow for frames that lack any 802.1Q
+ * header. For these, drop the OFPACT_STRIP_VLAN
+ * action. */
+ ofpbuf_pull(&ofpacts, ofpacts_orig_size);
+ match_set_dl_tci_masked(&match, 0, htons(VLAN_CFI));
+ ofctrl_add_flow(flow_table, 0, 100, &match, &ofpacts);
}
/* Table 33, priority 100.
}
ofctrl_add_flow(flow_table, OFTABLE_LOG_TO_PHY, 100,
&match, &ofpacts);
+ } else if (!tun) {
+ /* Remote port connected by localnet port */
+ /* Table 33, priority 100.
+ * =======================
+ *
+ * Implements switching to localnet port. Each flow matches a
+ * logical output port on remote hypervisor, switch the output port
+ * to connected localnet port and resubmits to same table.
+ */
+
+ match_init_catchall(&match);
+ ofpbuf_clear(&ofpacts);
+
+ /* Match MFF_LOG_DATAPATH, MFF_LOG_OUTPORT. */
+ match_set_metadata(&match, htonll(binding->datapath->tunnel_key));
+ match_set_reg(&match, MFF_LOG_OUTPORT - MFF_REG0,
+ binding->tunnel_key);
+
+ put_load(localnet_port->tunnel_key, MFF_LOG_OUTPORT, 0, 32, &ofpacts);
+
+ /* Resubmit to table 33. */
+ put_resubmit(OFTABLE_LOCAL_OUTPUT, &ofpacts);
+ ofctrl_add_flow(flow_table, OFTABLE_LOCAL_OUTPUT, 100, &match,
+ &ofpacts);
} else {
+ /* Remote port connected by tunnel */
/* Table 32, priority 100.
* =======================
*
? port->parent_port : port->logical_port)) {
put_load(port->tunnel_key, MFF_LOG_OUTPORT, 0, 32, &ofpacts);
put_resubmit(OFTABLE_DROP_LOOPBACK, &ofpacts);
- } else if (port->chassis) {
+ } else if (port->chassis && !get_localnet_port(local_datapaths,
+ mc->datapath->tunnel_key)) {
+ /* Add remote chassis only when localnet port not exist,
+ * otherwise multicast will reach remote ports through localnet
+ * port. */
sset_add(&remote_chassis, port->chassis->name);
- } else if (!strcmp(port->type, "localnet")) {
- const char *network = smap_get(&port->options, "network_name");
- if (!network) {
- continue;
- }
- if (!simap_contains(&localnet_to_ofport, network)) {
- continue;
- }
- put_load(port->tunnel_key, MFF_LOG_OUTPORT, 0, 32, &ofpacts);
- put_resubmit(OFTABLE_DROP_LOOPBACK, &ofpacts);
}
}
free(tun);
}
hmap_destroy(&tunnels);
-
- /* Table 0, Priority 100.
- * ======================
- *
- * We have now determined the full set of port bindings associated with
- * each "localnet" network. Only create flows for datapaths that have
- * another local binding. Otherwise, we know it would just be dropped.
- */
- struct shash_node *ln_bindings_node, *ln_bindings_node_next;
- SHASH_FOR_EACH_SAFE (ln_bindings_node, ln_bindings_node_next,
- &localnet_inputs) {
- struct localnet_bindings *ln_bindings = ln_bindings_node->data;
- struct localnet_vlan *ln_vlan, *ln_vlan_next;
- HMAP_FOR_EACH_SAFE (ln_vlan, ln_vlan_next, node, &ln_bindings->vlans) {
- struct match match;
- match_init_catchall(&match);
- match_set_in_port(&match, ln_bindings->ofport);
- if (ln_vlan->tag) {
- match_set_dl_vlan(&match, htons(ln_vlan->tag));
- } else {
- /* Match priority-tagged frames, e.g. VLAN ID 0.
- *
- * We'll add a second flow for frames that lack any 802.1Q
- * header later. */
- match_set_dl_tci_masked(&match, htons(VLAN_CFI),
- htons(VLAN_VID_MASK | VLAN_CFI));
- }
-
- struct ofpbuf ofpacts;
- ofpbuf_init(&ofpacts, 0);
-
- ofpact_put_STRIP_VLAN(&ofpacts);
- uint32_t ofpacts_orig_size = ofpacts.size;
-
- struct binding_elem *b;
- LIST_FOR_EACH_POP (b, list_elem, &ln_vlan->bindings) {
- struct hmap_node *ld;
- ld = hmap_first_with_hash(local_datapaths,
- b->binding->datapath->tunnel_key);
- if (ld) {
- /* Set MFF_LOG_DATAPATH and MFF_LOG_INPORT. */
- put_load(b->binding->datapath->tunnel_key, MFF_LOG_DATAPATH,
- 0, 64, &ofpacts);
- put_load(b->binding->tunnel_key, MFF_LOG_INPORT, 0, 32,
- &ofpacts);
- put_resubmit(OFTABLE_LOG_INGRESS_PIPELINE, &ofpacts);
- }
-
- free(b);
- }
-
- if (ofpacts.size > ofpacts_orig_size) {
- ofctrl_add_flow(flow_table, 0, 100, &match, &ofpacts);
-
- if (!ln_vlan->tag) {
- /* Add a second flow for frames that lack any 802.1Q
- * header. For these, drop the OFPACT_STRIP_VLAN
- * action. */
- ofpbuf_pull(&ofpacts, ofpacts_orig_size);
- match_set_dl_tci_masked(&match, 0, htons(VLAN_CFI));
- ofctrl_add_flow(flow_table, 0, 100, &match, &ofpacts);
- }
- }
-
- ofpbuf_uninit(&ofpacts);
-
- hmap_remove(&ln_bindings->vlans, &ln_vlan->node);
- free(ln_vlan);
- }
- shash_delete(&localnet_inputs, ln_bindings_node);
- hmap_destroy(&ln_bindings->vlans);
- free(ln_bindings);
- }
- shash_destroy(&localnet_inputs);
-
- simap_destroy(&localnet_to_ofport);
}
projects / cascardo / ovs.git / blobdiff