On Mon, Dec 10, 2018 at 5:42 PM Ben Pfaff <[email protected]> wrote:

> On Mon, Nov 26, 2018 at 08:48:39AM -0800, Darrell Ball wrote:
> > The user should only reference a conntrack entry by the forward
> > direction context, as per 'conntrack_flush()', enforce this by
> > checking for 'default' conn_type.  The likelihood of a user
> > not using the original tuple is low, but it should be guarded
> > against, logged and documented.
> >
> > Signed-off-by: Darrell Ball <[email protected]>
> > ---
> >
> > Backport to 2.9.
> >
> > v3: Move backport hint out of commit message.
> >     Remove warning log conditional for now.
>
> Would it be more user-friendly to translate these into the forward
> equivalent and flush that one?
>

If there were a practical application to try to flush a tuple using the
dynamic/random
NAT tuple assignment, yes; but there is not and it is also not worth the
added complexity
to handle the race b/w buckets for this purpose.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to