Re: [dpdk-dev] [PATCH v7 0/3] Enable rte_flow API in ice driver

From: "Yang, Qiming" <qiming.yang@intel.com>
To: "Yigit, Ferruh" <ferruh.yigit@intel.com>,
	Aaron Conole <aconole@redhat.com>
Cc: "dev@dpdk.org" <dev@dpdk.org>
Subject: Re: [dpdk-dev] [PATCH v7 0/3] Enable rte_flow API in ice driver
Date: Wed, 26 Jun 2019 08:26:38 +0000	[thread overview]
Message-ID: <F5DF4F0E3AFEF648ADC1C3C33AD4DBF17A4ED3D4@SHSMSX101.ccr.corp.intel.com> (raw)
In-Reply-To: <47f5df91-b872-f59b-4de3-a704e6ca586a@intel.com>

Sure, I'll add the information in cover letter. 

> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Wednesday, June 26, 2019 3:43 PM
> To: Yang, Qiming <qiming.yang@intel.com>; Aaron Conole
> <aconole@redhat.com>
> Cc: dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v7 0/3] Enable rte_flow API in ice driver
> 
> On 6/26/2019 2:52 AM, Yang, Qiming wrote:
> > Hi,
> > It seems these error induced by lack of some base code which are merged
> in dpdk-next-net-intel branch.
> > It's not this patch set's issue.
> 
> If there is a dependency to another patchset, can you please put this
> information into cover letter?
> 
> >
> > Qiming
> > -----Original Message-----
> > From: Aaron Conole [mailto:aconole@redhat.com]
> > Sent: Tuesday, June 25, 2019 10:58 PM
> > To: Yang, Qiming <qiming.yang@intel.com>
> > Cc: dev@dpdk.org
> > Subject: Re: [dpdk-dev] [PATCH v7 0/3] Enable rte_flow API in ice
> > driver
> >
> > 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.
> >> v6 changes:
> >>  - fixed one uninitialize issue.
> >> v7 changes:
> >>  - fixed queue action validation.
> >
> > Seems there is still compliation problems - is some dependency not
> correctly updated?
> >
> > https://travis-ci.com/ovsrobot/dpdk/jobs/210680563
> >
> >> 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  | 696
> >> ++++++++++++++++++++++++++++++++++++
> >>  drivers/net/ice/ice_generic_flow.h  | 614
> >> +++++++++++++++++++++++++++++++  drivers/net/ice/ice_switch_filter.c
> >> +++++++++++++++++++++++++++++++ |
> >> 512 ++++++++++++++++++++++++++  drivers/net/ice/ice_switch_filter.h |
> 24 ++
> >>  drivers/net/ice/meson.build         |   4 +-
> >>  8 files changed, 1979 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