[ovs-dev] [PATCH] ovn: Recirculate packets after a unSNAT.

[Ben Pfaff]

On Mon, Mar 26, 2018 at 02:18:28PM -0700, Gurucharan Shetty wrote:
> commit f6fabcc6245 (ofproto-dpif: Mark packets as "untracked"
> after call to ct().) changed the behavior after a call to ct().
> The +trk bit would automatically be unset if packet is sent to
> ct() and not forked.  This caused a bug in the OVN gateway
> pipeline when there is SNAT rule as well as load-balancing rule.
> 
> In the OVN gateway pipeline for the gateway router, we had an
> optimization where the packets sent to unSNAT need not go through
> a recirculation. But since doing this now means that the +trk bit
> gets unset, the DNAT rules for load-balancing a new packet in the next
> table won't get hit.
> 
> This commit removes the optimization for unSNAT packets so that
> there is always a recirculation.
> 
> Signed-off-by: Gurucharan Shetty <guru at ovn.org>

Acked-by: Ben Pfaff <blp at ovn.org>