[ovs-dev] [PATCH 4/6] connmgr: Make treatment of active and passive connections more uniform.

Ben Pfaff blp at ovn.org
Tue Feb 5 17:19:50 UTC 2019


On Mon, Feb 04, 2019 at 05:51:11PM -0800, Justin Pettit wrote:
> 
> > On Oct 29, 2018, at 3:57 PM, Ben Pfaff <blp at ovn.org> wrote:
> > 
> > Until now, connmgr has handled active and passive OpenFlow connections in
> > quite different ways.  Any active connection, whether it was currently
> > connected or not, was always maintained as an ofconn.  Whenever such a
> > connection (re)connected, its settings were cleared.  On the other hand,
> > passive connections had a separate listener which created an ofconn when
> > a new connection came in, and these ofconns would be deleted when such a
> > connection was closed.  This approach is inelegant and has occasionally
> > led to bugs when reconnection didn't clear all of the state that it
> > should have.
> > 
> > There's another motivation here.  Currently, active connections are
> > always primary controllers and passive connections are always service
> > controllers (as documented in ovs-vswitchd.conf.db(5)).  Sometimes it would
> > be useful to have passive primary controllers (maybe active service
> > controllers too but I haven't personally run into that use case).  As is,
> > this is difficult to implement because there is so much different code in
> > use between active and passive connections.  This commit will make it
> > easier.
> > 
> > Signed-off-by: Ben Pfaff <blp at ovn.org>
> 
> Acked-by: Justin Pettit <jpettit at ovn.org>

Thanks, applied to master.


More information about the dev mailing list