On 27 Jun 2023, at 12:48, Roi Dayan wrote:

> Hi,
>
> This series adds TC offload support for filtering vxlan tunnels with gbp 
> option.
> First 4 patches do some refactoring and the later patches adds the feature.
>
> Thanks,
> Roi

Thanks for following up on the series!

I’ve applied the changes discussed and applied the series to the master branch!

Cheers,

Eelco


> changelog
>
> v6:
> - fix nits in patch #7.
> - fix test in patch #9 to run test on supported kernel.
>
> v5:
> - Fix byte order issue in probe.
>
> v4:
> - probe TC kernel for vxlan gbp support.
> - add test.
> - style fix in patch 3.
> - log warn instead of err in patch 5.
>
> v3:
> - Add function nl_msg_start_nested_with_flag() to be used
>   with TC fiedls that require the nested flag. currently
>   only vxlan gbp tun opts.
> - Split put flower tunnel opts to sub functions for geneve
>   and vxlan tun opts.
>
> v2:
> - Fix incorrect compat modification in
>   patch "tc: Add vxlan gbp option flower match offload".
>
> Gavin Li (9):
>   tc: Pass tunnel entirely to tunnel option parse and put functions
>   odp-util: Extract vxlan gbp option decoding to a function
>   odp-util: Extract vxlan gbp option encoding to a function
>   netlink: Add new function to add NLA_F_NESTED to nested netlink
>     messages
>   tc: Add vxlan gbp option flower match offload
>   tc: Pass encap entirely to nl_msg_put_act_tunnel_key_set
>   tc: Add vxlan encap action with gbp option offload
>   netdev-tc-offloads: Probe for allowing vxlan gbp support
>   system-offloads-traffic.at: Add vxlan gbp offload test
>
>  acinclude.m4                         |   7 +
>  include/linux/pkt_cls.h              |  13 ++
>  include/linux/tc_act/tc_tunnel_key.h |  17 ++-
>  lib/netdev-offload-tc.c              | 105 ++++++++++++-
>  lib/netlink.c                        |   9 ++
>  lib/netlink.h                        |   1 +
>  lib/odp-util.c                       |  54 ++++---
>  lib/odp-util.h                       |  16 ++
>  lib/tc.c                             | 218 ++++++++++++++++++++-------
>  lib/tc.h                             |  80 +++++-----
>  tests/system-offloads-traffic.at     |  50 ++++++
>  11 files changed, 461 insertions(+), 109 deletions(-)
>
> -- 
> 2.38.0

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

Reply via email to