mbox series

[v5,0/3] Enable rte_flow API in ice driver

Message ID 20190621092132.140460-1-qiming.yang@intel.com (mailing list archive)
Headers
Series Enable rte_flow API in ice driver |

Message

Qiming Yang June 21, 2019, 9:21 a.m. UTC 
  This patch set enables the backend of rte_flow, and the generic
filter related functions in ice driver. Supported flows include
ipv4, tcpv4, udpv4, ipv6, tcpv6, udpv6, tunnel, etc. This patch
set depends on shared code update.

---
v2 changes:
 - added UDP tunnel port support.
 - fixed compile issue.
 - added document update.
v3 changes:
 - removed redundancy parser.
 - added License.
 - added VXLAN and NVGRE item support.
v4 changes:
 - fixed some typos.
v5 changes:
 - fixed checkpatch issues.

Qiming Yang (2):
  net/ice: add generic flow API
  net/ice: add UDP tunnel port support

wei zhao (1):
  net/ice: enable switch filter

 drivers/net/ice/Makefile            |   2 +
 drivers/net/ice/ice_ethdev.c        | 116 ++++++
 drivers/net/ice/ice_ethdev.h        |  12 +
 drivers/net/ice/ice_generic_flow.c  | 683 ++++++++++++++++++++++++++++++++++++
 drivers/net/ice/ice_generic_flow.h  | 654 ++++++++++++++++++++++++++++++++++
 drivers/net/ice/ice_switch_filter.c | 521 +++++++++++++++++++++++++++
 drivers/net/ice/ice_switch_filter.h |  24 ++
 drivers/net/ice/meson.build         |   4 +-
 8 files changed, 2015 insertions(+), 1 deletion(-)
 create mode 100644 drivers/net/ice/ice_generic_flow.c
 create mode 100644 drivers/net/ice/ice_generic_flow.h
 create mode 100644 drivers/net/ice/ice_switch_filter.c
 create mode 100644 drivers/net/ice/ice_switch_filter.h

Comments

Aaron Conole June 21, 2019, 2:46 p.m. UTC | #1 
Qiming Yang <qiming.yang@intel.com> writes:

> This patch set enables the backend of rte_flow, and the generic
> filter related functions in ice driver. Supported flows include
> ipv4, tcpv4, udpv4, ipv6, tcpv6, udpv6, tunnel, etc. This patch
> set depends on shared code update.
>
> ---
> v2 changes:
>  - added UDP tunnel port support.
>  - fixed compile issue.
>  - added document update.
> v3 changes:
>  - removed redundancy parser.
>  - added License.
>  - added VXLAN and NVGRE item support.
> v4 changes:
>  - fixed some typos.
> v5 changes:
>  - fixed checkpatch issues.
>

Thanks for fixing up the meson errors.  I notice some build issues from
the Travis server.  Can you take a look?

https://travis-ci.com/ovsrobot/dpdk/builds/116409374

I think the errors are introduced with the first patch, actually.

It might help to use github as an intermediate repository, sign up for
travis, push your changes to your github, and then watch the builds to
ensure that they are compiling properly.  That can help to isolate
environmental issues (for instance, accidentally omitted header files,
etc).

Some of this is documented at doc/guides/contributing/patches.rst but if
something is unclear or should have more detail to explain, lets improve
the docs.