[ovs-dev] [PATCH V6 00/12] netdev datapath offload: Support IPv6 and VXLAN encap

[Ilya Maximets]

On 7/8/20 8:38 AM, Eli Britstein wrote:
> This patch set includes enhanced logging to increase debugability, bug
> fixes and additional offloads - IPv6 and VXLAN encap.
> 
> Patches #1-#2:  Enhance log prints for debugability.
> Patches #3-#4:  Bug fixes for partial offloads and Ethernet matching.
> Patches #5-#12: Add support for offloads of IPv6 patterns, partial
>                 TCP/UDP ports, set IPv6 and encap actions
>                 (clone/output).
> 
> v2-v1:
> - Removed redundant out label.
> - Added a patch to fix dl_type match only.
> v3-v2:
> - Rebased, and more elaboration in #7 commit message.
> v4-v3:
> - Rebased, changed order of commits.
> - Testpmd format replaces pervious format.
> - Some comments from Ilya/Harsha addressed.
> v5-v4:
> - Removed redundant trailing '\n' from log messages.
> - Removed 'testpmd' keyword.
> - 'DUMP_TESTPMD_ITEM' -> 'DUMP_PATTERN_ITEM'.
> - Removed *_mask1 variables.
> - Changed logging to dynamic string for tunnel dumps.
> v6-v5:
> - Not introducing/removing 'testpmd' keyword.
> - Minor log format fixes.
> 
> Travis:
> v1: https://travis-ci.org/github/elibritstein/OVS/builds/688413350
> v2: https://travis-ci.org/github/elibritstein/OVS/builds/691375847
> v3: https://travis-ci.org/github/elibritstein/OVS/builds/700534550
> v4: https://travis-ci.org/github/elibritstein/OVS/builds/703556030
> v5: https://travis-ci.org/github/elibritstein/OVS/builds/705679636
> v6: https://travis-ci.org/github/elibritstein/OVS/builds/706044892
> 
> Eli Britstein (10):
>   netdev-offload-dpdk: Log testpmd format for flow create/destroy
>   dpif-netdev: Add mega ufid in flow add/del log
>   dpif-netdev: Don't use zero flow mark
>   netdev-offload-dpdk: Fix Ethernet matching for type only
>   netdev-offload-dpdk: Support partial TCP/UDP port matching
>   netdev-offload-dpdk: Remove pre-validate of patterns function
>   netdev-offload-dpdk: Add IPv6 pattern matching
>   netdev-offload-dpdk: Support offload of set IPv6 actions
>   netdev-offload-dpdk: Support offload of clone tnl_push/output actions
>   netdev-offload-dpdk: Support tnl/push using vxlan encap attribute
> 
> Ilya Maximets (2):
>   netdev: Allow storing dpif type into netdev structure.
>   netdev-offload: Use dpif type instead of class.
> 
>  Documentation/howto/dpdk.rst  |   4 +-
>  NEWS                          |   3 +
>  lib/dpif-netdev.c             |  36 +-
>  lib/dpif-netlink.c            |  23 +-
>  lib/dpif.c                    |  21 +-
>  lib/netdev-offload-dpdk.c     | 883 ++++++++++++++++++++++++++----------------
>  lib/netdev-offload-tc.c       |   3 +-
>  lib/netdev-offload.c          |  52 ++-
>  lib/netdev-offload.h          |  16 +-
>  lib/netdev-provider.h         |   3 +-
>  lib/netdev.c                  |  16 +
>  lib/netdev.h                  |   2 +
>  ofproto/ofproto-dpif-upcall.c |   5 +-
>  tests/dpif-netdev.at          |  20 +-
>  tests/ofproto-macros.at       |   3 +-
>  15 files changed, 680 insertions(+), 410 deletions(-)
> 

Thanks!

I fixed the NEWS line in patch #8 as "IPv6 TCP/UDP ports" doesn't sound
correct to me and also stripped line breaks from VLOG in patch #2.
With that, applied to master branch.

Best regards, Ilya Maximets.