[ovs-dev] [PATCH ovn v2 3/5] ovn-northd: Remove lflow_add_unique.
Dumitru Ceara
dceara at redhat.com
Fri Jun 18 18:57:27 UTC 2021
On 6/18/21 5:51 PM, Dumitru Ceara wrote:
> On 6/11/21 9:35 PM, Han Zhou wrote:
>> This patch removes the workaround when adding multicast group related
>> lflows, because the multicast group dependency problem is fixed in
>> ovn-controller in the previous commit.
>>
>> This patch also removes the UniqueFlow/AnnotatedFlow usage in northd
>> DDlog implementation for the same reason.
>>
>> Signed-off-by: Han Zhou <hzhou at ovn.org>
>> ---
>> northd/ovn-northd.c | 89 ++++++-----------
>> northd/ovn_northd.dl | 233 +++++++++++++++++++------------------------
>> tests/ovn-northd.at | 2 +-
>> 3 files changed, 137 insertions(+), 187 deletions(-)
>>
>> diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c
>> index 005c1fc86..411b14adf 100644
>> --- a/northd/ovn-northd.c
>> +++ b/northd/ovn-northd.c
>> @@ -3663,9 +3663,6 @@ build_ports(struct northd_context *ctx,
>> sset_destroy(&active_ha_chassis_grps);
>> }
>
> [...]
>
>> @@ -6447,9 +6425,8 @@ build_lswitch_rport_arp_req_self_orig_flow(struct ovn_port *op,
>>
>> ds_put_format(&match, "eth.src == %s && (arp.op == 1 || nd_ns)",
>> ds_cstr(ð_src));
>> - ovn_lflow_add_unique(lflows, od, S_SWITCH_IN_L2_LKUP, priority,
>> - ds_cstr(&match),
>> - "outport = \""MC_FLOOD_L2"\"; output;");
>> + ovn_lflow_add(lflows, od, S_SWITCH_IN_L2_LKUP, priority, ds_cstr(&match),
>> + "outport = \""MC_FLOOD_L2"\"; output;");
>>
>> sset_destroy(&all_eth_addrs);
>> ds_destroy(ð_src);
>> @@ -6502,7 +6479,7 @@ build_lswitch_rport_arp_req_flow_for_ip(struct sset *ips,
>> ds_put_format(&actions, "clone {outport = %s; output; }; "
>> "outport = \""MC_FLOOD_L2"\"; output;",
>> patch_op->json_key);
>> - ovn_lflow_add_unique_with_hint(lflows, od, S_SWITCH_IN_L2_LKUP,
>> + ovn_lflow_add_with_hint(lflows, od, S_SWITCH_IN_L2_LKUP,
>> priority, ds_cstr(&match),
>> ds_cstr(&actions), stage_hint);
>
> Nit: indentation.
>
> Otherwise:
>
> Acked-by: Dumitru Ceara <dceara at redhat.com>
>
Actually, I'm not so sure about the ack anymore, with this applied some
ovn-northd-ddlog tests fail, including:
772: ovn -- logical gatapath groups -- ovn-northd-ddlog -- dp-groups=no
FAILED (ovn-macros.at:413)
Regards,
Dumitru
More information about the dev
mailing list