Re: [ovs-dev] [PATCH] conntrack: Fix conntrack multiple new state

2022-07-25 Thread Ilya Maximets
On 7/25/22 11:23, Paolo Valerio wrote: > Hello Eli, > > Eli Britstein via dev writes: > >> A connection is established if we see packets from both directions. >> The cited commit [1] fixed the issue of sending twice in one direction, >> but still an issue if more than that. >> Fix it. >> > > Th

Re: [ovs-dev] [PATCH] conntrack: Fix conntrack multiple new state

2022-07-25 Thread Paolo Valerio
Hello Eli, Eli Britstein via dev writes: > A connection is established if we see packets from both directions. > The cited commit [1] fixed the issue of sending twice in one direction, > but still an issue if more than that. > Fix it. > The patch LGTM. Just a very minor nit: I guess "[1]" could

[ovs-dev] [PATCH] conntrack: Fix conntrack multiple new state

2022-07-17 Thread Eli Britstein via dev
A connection is established if we see packets from both directions. The cited commit [1] fixed the issue of sending twice in one direction, but still an issue if more than that. Fix it. Fixes: a867c010ee91 ("conntrack: Fix conntrack new state") Signed-off-by: Eli Britstein --- lib/conntrack-othe