mir/ovs
2
0
Fork 0
mirror of https://github.com/openvswitch/ovs synced 2025-08-29 13:27:59 +00:00
Code Issues Releases Wiki Activity

bridge: Inline iterate_and_prune_ifaces() and remove it.

Browse Source 
The main reason that iterate_and_prune_ifaces() existed was because it was
somewhat inconvenient to iterate across all of the interfaces, especially
if anything needed to be deleted.  Now that we've switched from arrays to
lists and hmaps, it's a bit easier, and certainly it's easier to read code
when there aren't any callbacks involved, so inline what this was doing.

This was the only remaining caller of iterate_and_prune_ifaces() so this
removes that function as well as the callback.
This commit is contained in:
Ben Pfaff 2011-03-29 11:32:25 -07:00
parent 7ae792355f
commit c3effea5f6
1 changed files with 30 additions and 51 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

81
vswitchd/bridge.c
View File

@ -419,54 +419,6 @@ bridge_configure_once(const struct ovsrec_open_vswitch *cfg)
sset_destroy(&dpif_types); sset_destroy(&dpif_types);
} }
/* Callback for iterate_and_prune_ifaces(). */
static bool
check_iface(struct bridge *br, struct iface *iface, void *aux OVS_UNUSED)
{
if (!iface->netdev) {
/* We already reported a related error, don't bother duplicating it. */
return false;
}
if (iface->dp_ifidx < 0) {
VLOG_ERR("%s interface not in %s, dropping",
iface->name, dpif_name(br->dpif));
return false;
}
VLOG_DBG("%s has interface %s on port %d", dpif_name(br->dpif),
iface->name, iface->dp_ifidx);
return true;
}
/* Calls 'cb' for each interfaces in 'br', passing along the 'aux' argument.
* Deletes from 'br' all the interfaces for which 'cb' returns false, and then
* deletes from 'br' any ports that no longer have any interfaces. */
static void
iterate_and_prune_ifaces(struct bridge *br,
bool (*cb)(struct bridge *, struct iface *,
void *aux),
void *aux)
{
struct port *port, *next_port;
HMAP_FOR_EACH_SAFE (port, next_port, hmap_node, &br->ports) {
struct iface *iface, *next_iface;
LIST_FOR_EACH_SAFE (iface, next_iface, port_elem, &port->ifaces) {
if (!cb(br, iface, aux)) {
iface_set_ofport(iface->cfg, -1);
iface_destroy(iface);
}
}
if (list_is_empty(&port->ifaces)) {
VLOG_WARN("%s port has no interfaces, dropping", port->name);
port_destroy(port);
}
}
}
/* Looks at the list of managers in 'ovs_cfg' and extracts their remote IP /* Looks at the list of managers in 'ovs_cfg' and extracts their remote IP
* addresses and ports into '*managersp' and '*n_managersp'. The caller is * addresses and ports into '*managersp' and '*n_managersp'. The caller is
* responsible for freeing '*managersp' (with free()). * responsible for freeing '*managersp' (with free()).
@ -735,6 +687,7 @@ bridge_reconfigure(const struct ovsrec_open_vswitch *ovs_cfg)
uint8_t ea[ETH_ADDR_LEN]; uint8_t ea[ETH_ADDR_LEN];
uint64_t dpid; uint64_t dpid;
struct iface *local_iface; struct iface *local_iface;
struct port *port, *next_port;
struct iface *hw_addr_iface; struct iface *hw_addr_iface;
char *dpid_string; char *dpid_string;
@ -742,9 +695,35 @@ bridge_reconfigure(const struct ovsrec_open_vswitch *ovs_cfg)
/* Delete interfaces that cannot be opened. /* Delete interfaces that cannot be opened.
* *
* From this point forward we are guaranteed that every "struct iface" * Following this loop, every remaining "struct iface" has nonnull
* has nonnull 'netdev' and correct 'dp_ifidx'. */ * 'netdev' and correct 'dp_ifidx'. */
iterate_and_prune_ifaces(br, check_iface, NULL); HMAP_FOR_EACH_SAFE (port, next_port, hmap_node, &br->ports) {
struct iface *iface, *next_iface;
LIST_FOR_EACH_SAFE (iface, next_iface, port_elem, &port->ifaces) {
if (iface->netdev && iface->dp_ifidx >= 0) {
VLOG_DBG("%s has interface %s on port %d",
dpif_name(br->dpif), iface->name,
iface->dp_ifidx);
} else {
if (iface->netdev) {
VLOG_ERR("%s interface not in %s, dropping",
iface->name, dpif_name(br->dpif));
} else {
/* We already reported a related error, don't bother
* duplicating it. */
}
iface_set_ofport(iface->cfg, -1);
iface_destroy(iface);
}
}
if (list_is_empty(&port->ifaces)) {
VLOG_WARN("%s port has no interfaces, dropping", port->name);
port_destroy(port);
}
}
/* Pick local port hardware address, datapath ID. */ /* Pick local port hardware address, datapath ID. */
bridge_pick_local_hw_addr(br, ea, &hw_addr_iface); bridge_pick_local_hw_addr(br, ea, &hw_addr_iface);