[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240708223839.GA18283@breakpoint.cc>
Date: Tue, 9 Jul 2024 00:38:39 +0200
From: Florian Westphal <fw@...len.de>
To: Xin Long <lucien.xin@...il.com>
Cc: Florian Westphal <fw@...len.de>, Ilya Maximets <i.maximets@....org>,
network dev <netdev@...r.kernel.org>, dev@...nvswitch.org,
Marcelo Ricardo Leitner <marcelo.leitner@...il.com>,
Jiri Pirko <jiri@...nulli.us>, Davide Caratti <dcaratti@...hat.com>,
Jamal Hadi Salim <jhs@...atatu.com>,
Eric Dumazet <edumazet@...gle.com>,
Cong Wang <xiyou.wangcong@...il.com>, kuba@...nel.org,
Paolo Abeni <pabeni@...hat.com>, davem@...emloft.net,
Pablo Neira Ayuso <pablo@...filter.org>,
Aaron Conole <aconole@...hat.com>
Subject: Re: [PATCH net-next 3/3] openvswitch: set IPS_CONFIRMED in tmpl
status only when commit is set in conntrack
Xin Long <lucien.xin@...il.com> wrote:
> I can avoid this warning by not allocating ext for commit ct in ovs:
>
> @@ -426,7 +426,7 @@ static int ovs_ct_set_labels(struct nf_conn *ct,
> struct sw_flow_key *key,
> struct nf_conn_labels *cl;
> int err;
>
> - cl = ovs_ct_get_conn_labels(ct);
> + cl = nf_ct_labels_find(ct);
> if (!cl)
> return -ENOSPC;
>
> However, the test case would fail, although the failure can be worked around
> by setting ct_label in the 1st rule:
>
> table=0,priority=30,in_port=1,ip,nw_dst=172.1.1.2,actions=ct(commit,nat(dst=10.1.1.2:80),exec(set_field:0x01->ct_label),table=1)
>
> So I'm worrying our change may break some existing OVS user cases.
Then ovs_ct_limit_init() and nf_connlabels_get() need to be called
once on the first conntrack operatation, regardless if labels are asked
for or not.
Not nice but still better than current state.
ovs_ct_execute() perhaps?
Powered by blists - more mailing lists