Re: [ovs-dev] [PATCH v3 1/7] tc: Pass tunnel entirely to tunnel option parse and put functions

[Eelco Chaudron]

On 15 May 2023, at 10:23, Roi Dayan wrote:

> From: Gavin Li <gav...@nvidia.com>
>
> Tc flower tunnel key options were encoded in nl_msg_put_flower_tunnel_opts
> and decoded in nl_parse_flower_tunnel_opts. Only geneve was supported.
>
> To avoid adding more arguments to the function to support more vxlan
> options in the future, change the function arguments to pass tunnel
> entirely to it instead of keep adding new arguments.
>
> Signed-off-by: Gavin Li <gav...@nvidia.com>
> Reviewed-by: Roi Dayan <r...@nvidia.com>
> Reviewed-by: Simon Horman <simon.hor...@corigine.com>

The changes look good to me.

Acked-by: Eelco Chaudron <echau...@redhat.com>

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev