[ovs-dev] [suse 6/7] xenserver: Add defaults for ENABLE_BRCOMPAT, ENABLE_FAKE_PROC_NET.

Patrick Mullaney pm.mullaney at gmail.com
Thu Dec 30 21:57:35 UTC 2010


On Tue, 2010-12-28 at 11:20 -0800, Justin Pettit wrote:
> Do we expect that the general case is that we want to enable bridge compatibility?
My thought was yes, at least until ovs and the standard bridge can run
side by side( I think I heard that was planned?). For the stable
distros with older kernels, this may be a needed for a while though.

> 
> --Justin
> 
> 
> On Dec 15, 2010, at 11:19 AM, Ben Pfaff wrote:
> 
> > From: Patrick Mullaney <pm.mullaney at gmail.com>
> > 
> > On non-XenServer systems there is no $PRODUCT_VERSION to test for default
> > behavior, so use reasonable defaults in that case.
> > 
> > Signed-off-by: Patrick Mullaney <pm.mullaney at gmail.com>
> > Signed-off-by: Ben Pfaff <blp at nicira.com>
> > ---
> > xenserver/etc_init.d_openvswitch |   17 +++++++++++------
> > 1 files changed, 11 insertions(+), 6 deletions(-)
> > 
> > diff --git a/xenserver/etc_init.d_openvswitch b/xenserver/etc_init.d_openvswitch
> > index 72d7d57..74ada6f 100755
> > --- a/xenserver/etc_init.d_openvswitch
> > +++ b/xenserver/etc_init.d_openvswitch
> > @@ -76,14 +76,19 @@ case ${NETWORK_MODE:=openvswitch} in
> > esac
> > 
> > # General config variables in /etc/sysconfig/openvswitch
> > -if test "$PRODUCT_VERSION" = "5.5.0"; then
> > -    # XenServer 5.5.0 needs ovs-brcompatd and /proc/net simulation.
> > +if [ -f /etc/xensource-inventory ]; then
> > +    if test "$PRODUCT_VERSION" = "5.5.0"; then
> > +        # XenServer 5.5.0 needs ovs-brcompatd and /proc/net simulation.
> > +        : ${ENABLE_BRCOMPAT:=y}
> > +        : ${ENABLE_FAKE_PROC_NET:=y}
> > +    else
> > +        # Later versions don't need them.
> > +        : ${ENABLE_BRCOMPAT:=n}
> > +        : ${ENABLE_FAKE_PROC_NET:=n}
> > +    fi
> > +else
> >     : ${ENABLE_BRCOMPAT:=y}
> >     : ${ENABLE_FAKE_PROC_NET:=y}
> > -else
> > -    # Later versions don't need them.
> > -    : ${ENABLE_BRCOMPAT:=n}
> > -    : ${ENABLE_FAKE_PROC_NET:=n}
> > fi
> > : ${ENABLE_MONITOR:=y}
> > : ${FORCE_COREFILES:=y}
> > -- 
> > 1.7.1
> > 
> > 
> > _______________________________________________
> > dev mailing list
> > dev at openvswitch.org
> > http://openvswitch.org/mailman/listinfo/dev_openvswitch.org
> 






More information about the dev mailing list