Re: [ovs-dev] [PATCH v3 6/7] tc: Pass encap entirely to nl_msg_put_act_tunnel_key_set

[Eelco Chaudron]

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

> From: Gavin Li <gav...@nvidia.com>
>
> Most of the data members of struct tc_action{ } are defined as anonymous
> struct in place. Instead of passing all members of an anonymous struct,
> which is not flexible to new members being added, expose encap as named
> struct and pass it entirely.
>
> Signed-off-by: Gavin Li <gav...@nvidia.com>
> Reviewed-by: Roi Dayan <r...@nvidia.com>
> Reviewed-by: Simon Horman <simon.hor...@corigine.com>

This looks good to me! Thanks for cleaning this up, looks way better this way.

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

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