On 2/2/21 1:47 PM, Paul Blakey wrote:
> Add offload support for ct_state rpl and inv flags.

Hi.

Since you're working on this, could you, please, review the following
patch:
  
http://patchwork.ozlabs.org/project/openvswitch/patch/[email protected]/

It seems like current OVS just ignores all the flags that can not
be passed to TC and this doesn't look right.

> 
> For example:
> ovs-ofctl del-flows br-ovs
> ovs-ofctl add-flow br-ovs arp,actions=normal
> ovs-ofctl add-flow br-ovs "table=0, ip,ct_state=-trk 
> actions=ct(table=1,zone=5)"
> ovs-ofctl add-flow br-ovs "table=1, ip,ct_state=+trk+new actions=ct(zone=5, 
> commit),normal"
> ovs-ofctl add-flow br-ovs "table=1, ip,ct_zone=5,ct_state=+trk+est+rpl 
> actions=normal"
> ovs-ofctl add-flow br-ovs "table=1, ip,ct_zone=5,ct_state=+trk+est-rpl 
> actions=normal"
> 
> Paul Blakey (2):
>   compat: Add TCA_FLOWER_KEY_CT_FLAGS_REPLY/INVALID definitions
>   netdev-offload-tc: Add support for ct_state rpl and inv flags
> 
>  acinclude.m4            |  6 +++---
>  include/linux/pkt_cls.h |  4 +++-
>  lib/netdev-offload-tc.c | 28 ++++++++++++++++++++++++++++
>  3 files changed, 34 insertions(+), 4 deletions(-)
> 

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to