Thanks, I pushed this.
To expand on Jesse's explanation a little, originally OVS had to steal
hooks used by the Linux bridge. There was no way for OVS to cooperate
with the bridge, so we arranged so that only one or the other could be
loaded. Somewhere in the 2.6.3x series, the Linux network stack
changed the hook arrangement so that OVS and the bridge could coexist,
so after that both can be loaded. But it never makes any sense to
have both the bridge and brcompat loaded.
On Mon, Jan 16, 2012 at 02:47:23PM -0800, Ethan Jackson wrote:
> Looks good,
>
> Are we allowed to load openvswitch_mod and bridge at the same time?
>
> Ethan
>
> On Mon, Jan 16, 2012 at 14:46, Ben Pfaff <b...@nicira.com> wrote:
> > brcompat_mod and the Linux bridge module are incompatible, so it doesn't
> > make sense to try to load both at the same time. ?This commit prevents
> > ovs-ctl from trying.
> >
> > Signed-off-by: Ben Pfaff <b...@nicira.com>
> > Reported-by: Rog?rio Vinhal Nunes
> > Launchpad bug #917309.
> > ---
> > ?AUTHORS ? ? ? ? ? ? ?| ? ?1 +
> > ?utilities/ovs-ctl.in | ? ?4 ++++
> > ?2 files changed, 5 insertions(+), 0 deletions(-)
> >
> > diff --git a/AUTHORS b/AUTHORS
> > index 87b3ccd..328218d 100644
> > --- a/AUTHORS
> > +++ b/AUTHORS
> > @@ -114,6 +114,7 @@ Peter Phaal ? ? ? ? ? ? peter.ph...@inmon.com
> > ?Ram Jothikumar ? ? ? ? ?rjothiku...@nicira.com
> > ?Ramana Reddy ? ? ? ? ? ?gtvrre...@gmail.com
> > ?Roger Leigh ? ? ? ? ? ? rle...@codelibre.net
> > +Rog?rio Vinhal Nunes
> > ?Sean Brady ? ? ? ? ? ? ?sbr...@gtfservices.com
> > ?Sebastian Andrzej Siewior ?sebast...@breakpoint.cc
> > ?S?bastien RICCIO ? ? ? ?s...@swisscenter.com
> > diff --git a/utilities/ovs-ctl.in b/utilities/ovs-ctl.in
> > index 5640ee8..d1df6a8 100755
> > --- a/utilities/ovs-ctl.in
> > +++ b/utilities/ovs-ctl.in
> > @@ -52,6 +52,10 @@ insert_openvswitch_mod_if_required () {
> > ?}
> >
> > ?insert_brcompat_mod_if_required () {
> > + ? ?if test -e /sys/module/bridge; then
> > + ? ? ? ?log_warning_msg "bridge module is loaded, not loading brcompat_mod"
> > + ? ? ? ?return 1
> > + ? ?fi
> > ? ? test -e /sys/module/brcompat_mod && return 0
> > ? ? action "Inserting brcompat module" modprobe brcompat_mod
> > ?}
> > --
> > 1.7.2.5
> >
> > _______________________________________________
> > dev mailing list
> > dev@openvswitch.org
> > http://openvswitch.org/mailman/listinfo/dev
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
