[ovs-dev] [PATCH v3 2/2] ovn: Fix localnet ports on the same chassis.

[Ben Pfaff]

On Tue, Feb 02, 2016 at 02:15:29PM -0800, Ben Pfaff wrote:
> On Thu, Jan 21, 2016 at 04:11:50PM -0500, Russell Bryant wrote:
> > Multiple logical ports on the same chassis that were connected to the
> > same physical network via localnet ports were not able to send packets
> > to each other.  This was because ovn-controller created a single patch
> > port between br-int and the physical network access bridge and used it
> > for all localnet ports.
> > 
> > The fix implemented here is to create a separate patch port for every
> > logical port of type=localnet.  An optimization is included where these
> > ports are only created if the localnet port is on a logical switch with
> > another logical port with an associated local VIF.
> > 
> > A nice side effect of this fix is that the code in physical.c got a lot
> > simpler, as localnet ports are now handled mostly like local VIFs.
> > 
> > Fixes: c02819293d52 ("ovn: Add "localnet" logical port type.")
> > Reported-by: Han Zhou <zhouhan at gmail.com>
> > Reported-at: http://openvswitch.org/pipermail/dev/2016-January/064413.html
> > Signed-off-by: Russell Bryant <russell at ovn.org>
> > Tested-by: Kyle Mestery <mestery at mestery.com
> > Acked-By: Kyle Mestery <mestery at mestery.com>
> > ---
> > 
> > 
> > v2->v3:
> >  - tweak localnet port input flow handling as suggested by ben
> 
> Thank you.
> 
> Acked-by: Ben Pfaff <blp at ovn.org>

Argh, I see that I reviewed v3 but v4 is already out.  My fault.  Let me
look at v4 now.