diff options
author | Xin Long <lucien.xin@gmail.com> | 2024-08-12 13:17:53 -0400 |
---|---|---|
committer | Jakub Kicinski <kuba@kernel.org> | 2024-08-15 19:05:19 -0700 |
commit | fcb1aa5163b1ae4cf2864b688b08927aac51f51e (patch) | |
tree | bd1a7a743d27cdb15909aced333994dd2bbf0ca9 /usr/include | |
parent | e5efc2311cc437e2b565d164a3de884fa33f13e9 (diff) | |
download | linux-fcb1aa5163b1ae4cf2864b688b08927aac51f51e.tar.gz linux-fcb1aa5163b1ae4cf2864b688b08927aac51f51e.tar.bz2 linux-fcb1aa5163b1ae4cf2864b688b08927aac51f51e.zip |
openvswitch: switch to per-action label counting in conntrack
Similar to commit 70f06c115bcc ("sched: act_ct: switch to per-action
label counting"), we should also switch to per-action label counting
in openvswitch conntrack, as Florian suggested.
The difference is that nf_connlabels_get() is called unconditionally
when creating an ct action in ovs_ct_copy_action(). As with these
flows:
table=0,ip,actions=ct(commit,table=1)
table=1,ip,actions=ct(commit,exec(set_field:0xac->ct_label),table=2)
it needs to make sure the label ext is created in the 1st flow before
the ct is committed in ovs_ct_commit(). Otherwise, the warning in
nf_ct_ext_add() when creating the label ext in the 2nd flow will
be triggered:
WARN_ON(nf_ct_is_confirmed(ct));
Signed-off-by: Xin Long <lucien.xin@gmail.com>
Reviewed-by: Aaron Conole <aconole@redhat.com>
Acked-by: Florian Westphal <fw@strlen.de>
Link: https://patch.msgid.link/6b9347d5c1a0b364e88d900b29a616c3f8e5b1ca.1723483073.git.lucien.xin@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'usr/include')
0 files changed, 0 insertions, 0 deletions