diff options
| author | Ben Pfaff <blp@nicira.com> | 2011-08-05 16:58:02 -0700 |
|---|---|---|
| committer | Ben Pfaff <blp@nicira.com> | 2011-08-08 10:17:54 -0700 |
| commit | 6c2d2a9f8318fea11250a7624144cb06436f91a4 (patch) | |
| tree | a1d62c11a9124b8300205d059a861d4d4d1ba48c /vswitchd | |
| parent | 92ada13254cbbc24ded4c9970d73dac1fe2f10d9 (diff) | |
| download | openvswitch-6c2d2a9f8318fea11250a7624144cb06436f91a4.tar.gz | |
Document and warn that mirroring to a VLAN is incompatible with SLB bonding.
vswitchd/INTERNALS explains the incompatibility:
2. When Open vSwitch forwards a multicast or broadcast packet to a
link in the SLB bond other than the active slave, the remote
switch will forward it to all of the other links in the SLB
bond, including the active slave. Without special handling,
this would mean that Open vSwitch would forward a second copy of
the packet to each switch port (other than the bond), including
the port that originated the packet.
Open vSwitch deals with this case by dropping packets received
on any SLB bonded link that have a source MAC+VLAN that has been
learned on any other port. (This means that SLB as implemented
in Open vSwitch relies critically on MAC learning. Notably, SLB
is incompatible with the "flood_vlans" feature.)
We could go farther than this and automatically change the bonding mode to
a safer one (e.g. active-backup) when flood_vlans are enabled. However,
that would still leave the SLB fallback for LACP modes in place; perhaps
active-backup would have to be the fallback for LACP modes when flood_vlans
are enabled.
Diffstat (limited to 'vswitchd')
| -rw-r--r-- | vswitchd/bridge.c | 5 | ||||
| -rw-r--r-- | vswitchd/vswitch.xml | 18 |
2 files changed, 18 insertions, 5 deletions
diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c index 6a4ebe58d..6491590a9 100644 --- a/vswitchd/bridge.c +++ b/vswitchd/bridge.c @@ -2256,6 +2256,11 @@ port_configure_bond(struct port *port, struct bond_settings *s, port->name, port->cfg->bond_mode, bond_mode_to_string(s->balance)); } + if (s->balance == BM_SLB && port->bridge->cfg->n_flood_vlans) { + VLOG_WARN("port %s: SLB bonds are incompatible with flood_vlans, " + "please use another bond type or disable flood_vlans", + port->name); + } miimon_interval = atoi(get_port_other_config(port->cfg, "bond-miimon-interval", "0")); diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml index bcabef567..b8aa813c9 100644 --- a/vswitchd/vswitch.xml +++ b/vswitchd/vswitch.xml @@ -334,11 +334,19 @@ </column> <column name="flood_vlans"> - VLAN IDs of VLANs on which MAC address learning should be disabled, so - that packets are flooded instead of being sent to specific ports that - are believed to contain packets' destination MACs. This should - ordinarily be used to disable MAC learning on VLANs used for mirroring - (RSPAN VLANs). It may also be useful for debugging. + <p> + VLAN IDs of VLANs on which MAC address learning should be disabled, + so that packets are flooded instead of being sent to specific ports + that are believed to contain packets' destination MACs. This should + ordinarily be used to disable MAC learning on VLANs used for + mirroring (RSPAN VLANs). It may also be useful for debugging. + </p> + <p> + SLB bonding (see the <ref table="Port" column="bond_mode"/> column in + the <ref table="Port"/> table) is incompatible with + <code>flood_vlans</code>. Consider using another bonding mode or + a different type of mirror instead. + </p> </column> </group> |