On Fri, Sep 18, 2020 at 9:05 PM Dumitru Ceara <[email protected]> wrote:

> On 9/18/20 5:29 PM, [email protected] wrote:
> > From: Numan Siddique <[email protected]>
> >
> > If the value of pkt_mark in the router policy options is greater than
> 2147483647, we
> > are ignoring it. This is because we use smap_get_int(). This patch fixes
> this issue
> > by adding a wrapper function - ovn_smap_get_uint(). The OpenvSwitch
> library doesn't have
> > this smap helper function.
> >
> > ovn_smap_get_uint() needs to be removed once the OpenvSwitch library has
> this helper
> > function. A comment has been added in the code for this.
> >
> > Fixes: a123ef0fb8fd("Support packet metadata marking for logical router
> policies.")
> > Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1878248
> > Reported-by: Alexander Constantinescu <[email protected]>
> > Signed-off-by: Numan Siddique <[email protected]>
> > ---
>
> Looks good to me.
>
> Acked-by: Dumitru Ceara <[email protected]>
>

Thanks Dumitru for the review. I applied this patch to master and
branch-20.09.

Numan


>
> Thanks,
> Dumitru
>
> _______________________________________________
> 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