Re: [PATCH net-next v2] net: openvswitch: Set OvS recirc_id from tc chain index

2019-08-25 Thread Paul Blakey
On 8/22/2019 6:57 AM, David Miller wrote: > From: Paul Blakey > Date: Tue, 20 Aug 2019 15:30:51 +0300 > >> @@ -4050,6 +4060,9 @@ enum skb_ext_id { >> #ifdef CONFIG_XFRM >> SKB_EXT_SEC_PATH, >> #endif >> +#if IS_ENABLED(CONFIG_NET_TC_SKB_EXT) >> +TC_SKB_EXT, >> +#endif >> SKB_EXT

Re: [PATCH net-next v2] net: openvswitch: Set OvS recirc_id from tc chain index

2019-08-21 Thread David Miller
From: Paul Blakey Date: Tue, 20 Aug 2019 15:30:51 +0300 > @@ -4050,6 +4060,9 @@ enum skb_ext_id { > #ifdef CONFIG_XFRM > SKB_EXT_SEC_PATH, > #endif > +#if IS_ENABLED(CONFIG_NET_TC_SKB_EXT) > + TC_SKB_EXT, > +#endif > SKB_EXT_NUM, /* must be last */ > }; Sorry, no. The SKB ext

Re: [PATCH net-next v2] net: openvswitch: Set OvS recirc_id from tc chain index

2019-08-20 Thread Paul Blakey
Hey guys, sorry for spam, I used the --in-reply-to  this time so it gets to the original thread ("[PATCH net-next v2] net: openvswitch: Set OvS recirc_id from tc chain index") , Ignore this thread and respond there if needed. Thanks. On 8/20/2019 3:40 PM, Paul Blakey wrote: >

Re: [PATCH net-next v2] net: openvswitch: Set OvS recirc_id from tc chain index

2019-08-20 Thread Paul Blakey
Regarding the user_features change, I tested the above patch with this one in userspace that I'll send once this is accepted, togother with the rest of connection tracking offload patches. I also have a test for it, if anyone wants it. Patch is: lib/netdev-offloads-tc: Probe recirc tc sharing fea

Re: [PATCH net-next v2] net: openvswitch: Set OvS recirc_id from tc chain index

2019-08-20 Thread Paul Blakey
Regarding the user_features change, I tested the above patch with this one in userspace that I'll send once this is accepted, togother with the rest of connection tracking offload patches. I also have a test for it, if anyone wants it. Patch is: lib/netdev-offloads-tc: Probe recirc tc sharing fea

[PATCH net-next v2] net: openvswitch: Set OvS recirc_id from tc chain index

2019-08-20 Thread Paul Blakey
Offloaded OvS datapath rules are translated one to one to tc rules, for example the following simplified OvS rule: recirc_id(0),in_port(dev1),eth_type(0x0800),ct_state(-trk) actions:ct(),recirc(2) Will be translated to the following tc rule: $ tc filter add dev dev1 ingress \ prio 1