mirror of
https://github.com/openvswitch/ovs
synced 2025-10-11 13:57:52 +00:00
bridge: Enable support for access and native VLAN ports on bonds.
Since Open vSwitch's inception we've disabled the use of bonds as access ports, for no particularly good reason. This also unintentionally prevented bonds from being used as native VLAN ports. This commit removes the code that prevented using bonds these ways Reported-and-tested-by: "Michael A. Collins" <mike.a.collins@ark-net.org> Signed-off-by: Ben Pfaff <blp@nicira.com>
This commit is contained in:
1
AUTHORS
1
AUTHORS
@@ -93,6 +93,7 @@ John Galgay john@galgay.net
|
||||
Koichi Yagishita yagishita.koichi@jrc.co.jp
|
||||
Krishna Miriyala krishna@nicira.com
|
||||
Luiz Henrique Ozaki luiz.ozaki@gmail.com
|
||||
Michael A. Collins mike.a.collins@ark-net.org
|
||||
Michael Hu mhu@nicira.com
|
||||
Michael Mao mmao@nicira.com
|
||||
Mike Bursell mike.bursell@citrix.com
|
||||
|
@@ -539,17 +539,8 @@ port_configure(struct port *port)
|
||||
|
||||
/* Get VLAN tag. */
|
||||
s.vlan = -1;
|
||||
if (cfg->tag) {
|
||||
if (list_is_short(&port->ifaces)) {
|
||||
if (*cfg->tag >= 0 && *cfg->tag <= 4095) {
|
||||
s.vlan = *cfg->tag;
|
||||
}
|
||||
} else {
|
||||
/* It's possible that bonded, VLAN-tagged ports make sense. Maybe
|
||||
* they even work as-is. But they have not been tested. */
|
||||
VLOG_WARN("port %s: VLAN tags not supported on bonded ports",
|
||||
port->name);
|
||||
}
|
||||
if (cfg->tag && *cfg->tag >= 0 && *cfg->tag <= 4095) {
|
||||
s.vlan = *cfg->tag;
|
||||
}
|
||||
|
||||
/* Get VLAN trunks. */
|
||||
|
Reference in New Issue
Block a user