On Tue, Sep 15, 2015 at 8:48 PM, Jesse Gross <je...@nicira.com> wrote: > If a packet is output to a tunnel port when userspace tunneling is > enabled, it will cause an ARP packet to be generated if the destination > is unknown. This ARP packet is injected into the physical bridge as > a new packet, where it is flooded. > > If there is a loop (such as if the tunnel destination is the same bridge), > the result will be infinite recursion. Even though we currently track > recursion limits, they are not effective here since each ARP packet is > considered to be a new translation. This changes the behavior so that > each ARP flow translation is initialized with the recursion counter of > the previous flow. Note that the problem only applies to ARP - data > packets in a loop will hit an existing recursion counter in the datapath. > > An additional side effect of this change is that ARP packets are no > longer unconditionally flooded in the new bridge. They will now follow any > flow rules in the new bridge that might apply to them, the same as with > the kernel datapath. > > Reported-by: David Evans <davidjoshuaev...@gmail.com> > Signed-off-by: Jesse Gross <je...@nicira.com>
LGTM Acked-by: Pravin B Shelar <pshe...@nicira.com> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev