Re: [ovs-dev] [PATCH v2 2/3] conntrack: Support conntrack flush by ct 5-tuple

2017-12-11 Thread Darrell Ball
A few suggestions Yi-hung Thanks Darrell On 11/21/17, 5:04 PM, "ovs-dev-boun...@openvswitch.org on behalf of Yi-Hung Wei" wrote: This patch adds support of flushing a conntrack entry specified by the conntrack 5-tuple

Re: [ovs-dev] [PATCH v2 2/3] conntrack: Support conntrack flush by ct 5-tuple

2017-12-08 Thread Justin Pettit
> On Dec 8, 2017, at 6:23 PM, Darrell Ball wrote: > > >I believe the kernel clears out expectations. As we discussed off-line, I > think it would make sense to follow similar behavior, since presumably if the > control channel is being flushed, the related flows

Re: [ovs-dev] [PATCH v2 2/3] conntrack: Support conntrack flush by ct 5-tuple

2017-12-08 Thread Darrell Ball
On 12/6/17, 2:57 PM, "ovs-dev-boun...@openvswitch.org on behalf of Justin Pettit" wrote: > On Nov 21, 2017, at 5:00 PM, Yi-Hung Wei wrote: > > diff --git a/lib/conntrack.c

Re: [ovs-dev] [PATCH v2 2/3] conntrack: Support conntrack flush by ct 5-tuple

2017-12-06 Thread Justin Pettit
> On Nov 21, 2017, at 5:00 PM, Yi-Hung Wei wrote: > > diff --git a/lib/conntrack.c b/lib/conntrack.c > index f5a3aa9fab34..c9077c07042c 100644 > --- a/lib/conntrack.c > +++ b/lib/conntrack.c > @@ -2335,6 +2335,18 @@ ct_endpoint_to_ct_dpif_inet_addr(const struct ct_addr >

[ovs-dev] [PATCH v2 2/3] conntrack: Support conntrack flush by ct 5-tuple

2017-11-21 Thread Yi-Hung Wei
This patch adds support of flushing a conntrack entry specified by the conntrack 5-tuple in dpif-netdev. Signed-off-by: Yi-Hung Wei --- lib/conntrack.c | 64 +++ lib/conntrack.h | 3 +++ lib/dpif-netdev.c | 2 +- 3