On Tue, Dec 03, 2019 at 12:13:11PM +0100, Ilya Maximets wrote:
> These patches are necessary to enable vxlan offloading for
> userspace datapath via DPDK flow offloading API (not tested).
> They allows to properly distinguish userspace tunnels from the
> system tunnels on netdev level in order to choose appropriate
> flow offloading API provider.  Before these patches it was not
> possible because tunneling ports are implemented by the same
> netdev-vport classes and uses same dpif port names, so only
> ofproto really knows which netdevs assigned to each dpif.

This series looks good to me.

Reviewed-by: Simon Horman <simon.hor...@netronome.com>

> 
> RFC v3:
>   * Rebase on current master.
> 
> RFC v2:
>   * Added 2 patches for using dpif type.
>   * Last patch updated to use netdev dpif_type instead of traversing
>     dpif ports.
> 
> 
> Ilya Maximets (4):
>   netdev: Allow storing dpif type into netdev structure.
>   netdev-offload: Use dpif type instead of class.
>   netdev-offload: Allow offloading to netdev without ifindex.
>   netdev-offload: Disallow offloading to unrelated tunneling vports.
> 
>  lib/dpif-netdev.c             | 13 +++----
>  lib/dpif-netlink.c            | 23 ++++++------
>  lib/dpif.c                    | 21 ++++++-----
>  lib/netdev-offload-dpdk.c     |  8 +++++
>  lib/netdev-offload-tc.c       | 12 +++++--
>  lib/netdev-offload.c          | 68 ++++++++++++++++++-----------------
>  lib/netdev-offload.h          | 16 ++++-----
>  lib/netdev-provider.h         |  3 +-
>  lib/netdev.c                  | 16 +++++++++
>  lib/netdev.h                  |  2 ++
>  ofproto/ofproto-dpif-upcall.c |  5 ++-
>  11 files changed, 114 insertions(+), 73 deletions(-)
> 
> -- 
> 2.17.1
> 
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to