Hi Paul, On Sun, Feb 20, 2022 at 11:32:26AM +0200, Paul Blakey wrote: > After cited commit optimizted hw insertion, flow table entries are > populated with ifindex information which was intended to only be used > for HW offload. This tuple ifindex is hashed in the flow table key, so > it must be filled for lookup to be successful. But tuple ifindex is only > relevant for the netfilter flowtables (nft), so it's not filled in > act_ct flow table lookup, resulting in lookup failure, and no SW > offload and no offload teardown for TCP connection FIN/RST packets. > > To fix this, remove ifindex from hash, and allow lookup without > the ifindex. Act ct will lookup without the ifindex filled.
I think it is good to add FLOW_OFFLOAD_XMIT_TC (instead of relying on FLOW_OFFLOAD_XMIT_UNSPEC), this allows for more tc specific fields in the future. See attached patch. Thanks. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
