[ovs-dev] [PATCH ovn v2 1/3] northd.c: Remove redundant condition check in ovn_dp_group_add_with_reference().

Numan Siddique numans at ovn.org
Tue Oct 5 03:07:42 UTC 2021


On Mon, Oct 4, 2021 at 5:29 AM Anton Ivanov
<anton.ivanov at cambridgegreys.com> wrote:
>
>
> On 03/10/2021 23:45, Han Zhou wrote:
> > Signed-off-by: Han Zhou <hzhou at ovn.org>
> > ---
> >   northd/northd.c | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/northd/northd.c b/northd/northd.c
> > index cf2467fe1..027c5b170 100644
> > --- a/northd/northd.c
> > +++ b/northd/northd.c
> > @@ -4475,7 +4475,7 @@ ovn_dp_group_add_with_reference(struct ovn_lflow *lflow_ref,
> >           return false;
> >       }
> >
> > -    if (use_parallel_build && use_logical_dp_groups) {
> > +    if (use_parallel_build) {
> >           ovs_mutex_lock(&lflow_ref->odg_lock);
> >           hmapx_add(&lflow_ref->od_group, od);
> >           ovs_mutex_unlock(&lflow_ref->odg_lock);
> +1
>

Acked-by: Numan Siddique <numans at ovn.org>

Numan

> --
> Anton R. Ivanov
> Cambridgegreys Limited. Registered in England. Company Number 10273661
> https://www.cambridgegreys.com/
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>


More information about the dev mailing list