mir/ovs
2
0
Fork 0
mirror of https://github.com/openvswitch/ovs synced 2025-08-31 14:25:26 +00:00
Code Issues Releases Wiki Activity

netdev-offload-tc: Fix tc conntrack force commit support.

Browse Source 
tc was not setting the OVS_CT_ATTR_FORCE_COMMIT flag when a forced
commit was requested. This patch will fix this.

Fixes: 576126a931 ("netdev-offload-tc: Add conntrack support")
Signed-off-by: Eelco Chaudron <echaudro@redhat.com>
Acked-by: Roi Dayan <roid@nvidia.com>
Reviewed-by: Simon Horman <simon.horman@corigine.com>
Tested-by: Simon Horman <simon.horman@corigine.com>
Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
This commit is contained in:
Eelco Chaudron
2023-02-07 15:04:42 +01:00
committed by Ilya Maximets
parent 3209287103
commit 564d09ef53
1 changed files with 11 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
lib/netdev-offload-tc.c
View File

@@ -888,7 +888,11 @@ parse_tc_flower_to_actions__(struct tc_flower *flower, struct ofpbuf *buf,
ct_offset = nl_msg_start_nested(buf, OVS_ACTION_ATTR_CT);
if (action->ct.commit) {
nl_msg_put_flag(buf, OVS_CT_ATTR_COMMIT);
if (action->ct.force) {
nl_msg_put_flag(buf, OVS_CT_ATTR_FORCE_COMMIT);
} else {
nl_msg_put_flag(buf, OVS_CT_ATTR_COMMIT);
}
}
if (action->ct.zone) {
@@ -1377,7 +1381,12 @@ parse_put_flow_ct_action(struct tc_flower *flower,
NL_ATTR_FOR_EACH_UNSAFE (ct_attr, ct_left, ct, ct_len) {
switch (nl_attr_type(ct_attr)) {
case OVS_CT_ATTR_COMMIT: {
action->ct.commit = true;
action->ct.commit = true;
}
break;
case OVS_CT_ATTR_FORCE_COMMIT: {
action->ct.commit = true;
action->ct.force = true;
}
break;
case OVS_CT_ATTR_ZONE: {