On Mon, Apr 8, 2019 at 11:54 PM Ben Pfaff <[email protected]> wrote:

> Signed-off-by: Ben Pfaff <[email protected]>
> ---
>  lib/meta-flow.c | 6 ++++++
>

Acked-by: Numan Siddique <[email protected]>


>  1 file changed, 6 insertions(+)
>
> diff --git a/lib/meta-flow.c b/lib/meta-flow.c
> index 1724e46e1b97..e3274b97f335 100644
> --- a/lib/meta-flow.c
> +++ b/lib/meta-flow.c
> @@ -2287,6 +2287,12 @@ mf_set(const struct mf_field *mf,
>          *err_str = NULL;
>      }
>
> +    /* The cases where 'mask' is all-1-bits or all-0-bits were already
> handled
> +     * above[*], so the code below only needs to work for the remaining
> cases
> +     * of a nontrivial mask.
> +     *
> +     * [*] Except where the field is a tunnel metadata field and 'mask' is
> +     *     all-0-bits; see above. */
>      switch (mf->id) {
>      case MFF_CT_ZONE:
>      case MFF_CT_NW_PROTO:
> --
> 2.20.1
>
> _______________________________________________
> dev mailing list
> [email protected]
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to