Re: [ovs-dev] [PATCH RFC v2 0/8] Introduce connection tracking tc offload

2019-07-30 Thread Marcelo Ricardo Leitner
On Mon, Jul 29, 2019 at 01:05:50PM +, Paul Blakey wrote: > Hi, > > I reproduced both traces. > > The first rcu trace is because we deference the qdisc block without rtnl > locked (flower allows it unlocked) > > and the second trace is because res->goto_tp can be null if the last > prio of

Re: [ovs-dev] [PATCH RFC v2 0/8] Introduce connection tracking tc offload

2019-07-30 Thread Paul Blakey
On 7/29/2019 4:16 PM, Ilya Maximets wrote: > Hi. > > This patch-set has some checkpatch issues. It'll be good if you > could fix them before submitting the next version. > > Best regards, Ilya Maximets. Sure thanks. ___ dev mailing list

Re: [ovs-dev] [PATCH RFC v2 0/8] Introduce connection tracking tc offload

2019-07-29 Thread Ilya Maximets
Hi. This patch-set has some checkpatch issues. It'll be good if you could fix them before submitting the next version. Best regards, Ilya Maximets. On 04.07.2019 17:28, Paul Blakey wrote: > Hi, > > The following patches add connection tracking offload to tc. > > We plan on offloading the

Re: [ovs-dev] [PATCH RFC v2 0/8] Introduce connection tracking tc offload

2019-07-29 Thread Paul Blakey
On 7/23/2019 6:42 PM, Marcelo Ricardo Leitner wrote: > On Sat, Jul 20, 2019 at 08:26:59AM +, Paul Blakey wrote: >> Hi Marcelo, thanks for reporting this, can you dump the datapath rules via >> ovs-appctl dpctl/dump-flows -m --names >> Also running "tc filter show dev ns2-veth-ab ingress"

Re: [ovs-dev] [PATCH RFC v2 0/8] Introduce connection tracking tc offload

2019-07-24 Thread Paul Blakey
On 7/19/2019 12:59 AM, Marcelo Ricardo Leitner wrote: > > Hi Paul, > > Sometimes I'm seeing ghosts^Wbogus values with nat: > > [root@localhost ~]# ovs-ofctl dump-flows br0 > cookie=0x0, duration=788.595s, table=0, n_packets=3, n_bytes=180, > priority=50,ct_state=-trk,tcp,in_port="ns2-veth-ab"

Re: [ovs-dev] [PATCH RFC v2 0/8] Introduce connection tracking tc offload

2019-07-23 Thread Paul Blakey
On 7/23/2019 6:42 PM, Marcelo Ricardo Leitner wrote: > On Sat, Jul 20, 2019 at 08:26:59AM +, Paul Blakey wrote: >> Hi Marcelo, thanks for reporting this, can you dump the datapath rules via >> ovs-appctl dpctl/dump-flows -m --names >> Also running "tc filter show dev ns2-veth-ab ingress"

Re: [ovs-dev] [PATCH RFC v2 0/8] Introduce connection tracking tc offload

2019-07-23 Thread Marcelo Ricardo Leitner
On Sat, Jul 20, 2019 at 08:26:59AM +, Paul Blakey wrote: > Hi Marcelo, thanks for reporting this, can you dump the datapath rules via > ovs-appctl dpctl/dump-flows -m --names > Also running "tc filter show dev ns2-veth-ab ingress" while it happened can > show us more details. > > I'll try

Re: [ovs-dev] [PATCH RFC v2 0/8] Introduce connection tracking tc offload

2019-07-20 Thread Paul Blakey
Hi Marcelo, thanks for reporting this, can you dump the datapath rules via ovs-appctl dpctl/dump-flows -m --names Also running "tc filter show dev ns2-veth-ab ingress" while it happened can show us more details. I'll try and reproduce it on my end. How often does this happen? and what is the

Re: [ovs-dev] [PATCH RFC v2 0/8] Introduce connection tracking tc offload

2019-07-18 Thread Marcelo Ricardo Leitner
On Thu, Jul 04, 2019 at 05:28:19PM +0300, Paul Blakey wrote: > Hi, > > The following patches add connection tracking offload to tc. > > We plan on offloading the datapath rules to netdev one to one to tc rules. > We'll be using upcoming act_ct tc module which is currently under review in >