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 <zhou...@gmail.com> > > Reported-at: http://openvswitch.org/pipermail/dev/2016-January/064413.html > > Signed-off-by: Russell Bryant <russ...@ovn.org> > > Tested-by: Kyle Mestery <mest...@mestery.com > > Acked-By: Kyle Mestery <mest...@mestery.com> > > --- > > > > > > v2->v3: > > - tweak localnet port input flow handling as suggested by ben > > Thank you. > > Acked-by: Ben Pfaff <b...@ovn.org>
Argh, I see that I reviewed v3 but v4 is already out. My fault. Let me look at v4 now. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev