[ovs-dev] [PATCH] bridge: Enable support for access and native VLAN ports on bonds.

Ben Pfaff blp at nicira.com
Mon Dec 12 22:46:12 UTC 2011


And "branch-1.3" now that I remember that native VLANs are supported
there too.

On Mon, Dec 12, 2011 at 02:45:18PM -0800, Ben Pfaff wrote:
> Thanks, I pushed this to "master" and "branch-1.4".
> 
> On Mon, Dec 12, 2011 at 01:09:02PM -0800, Justin Pettit wrote:
> > Looks good.
> > 
> > --Justin
> > 
> > 
> > On Dec 12, 2011, at 11:10 AM, Ben Pfaff wrote:
> > 
> > > 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-by: "Michael A. Collins" <mike.a.collins at ark-net.org>
> > > Signed-off-by: Ben Pfaff <blp at nicira.com>
> > > ---
> > > AUTHORS           |    1 +
> > > vswitchd/bridge.c |   13 ++-----------
> > > 2 files changed, 3 insertions(+), 11 deletions(-)
> > > 
> > > diff --git a/AUTHORS b/AUTHORS
> > > index 9e0ac22..8af74e4 100644
> > > --- a/AUTHORS
> > > +++ b/AUTHORS
> > > @@ -93,6 +93,7 @@ John Galgay             john at galgay.net
> > > Koichi Yagishita        yagishita.koichi at jrc.co.jp
> > > Krishna Miriyala        krishna at nicira.com
> > > Luiz Henrique Ozaki     luiz.ozaki at gmail.com
> > > +Michael A. Collins      mike.a.collins at ark-net.org
> > > Michael Hu              mhu at nicira.com
> > > Michael Mao             mmao at nicira.com
> > > Mike Bursell            mike.bursell at citrix.com
> > > diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c
> > > index adf01f7..de773f6 100644
> > > --- a/vswitchd/bridge.c
> > > +++ b/vswitchd/bridge.c
> > > @@ -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. */
> > > -- 
> > > 1.7.4.4
> > > 
> > > _______________________________________________
> > > dev mailing list
> > > dev at openvswitch.org
> > > http://openvswitch.org/mailman/listinfo/dev
> > 



More information about the dev mailing list