From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Wu, Jingjing" Subject: Re: [PATCH v5 00/17] net/i40e: consistent filter API Date: Wed, 4 Jan 2017 06:40:29 +0000 Message-ID: <9BB6961774997848B5B42BEC655768F810CC2A06@SHSMSX103.ccr.corp.intel.com> References: <1483068352-32272-1-git-send-email-beilei.xing@intel.com> <1483500187-124740-1-git-send-email-beilei.xing@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Cc: "dev@dpdk.org" To: "Xing, Beilei" , "Zhang, Helin" Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id A1F5C5A44 for ; Wed, 4 Jan 2017 07:40:44 +0100 (CET) In-Reply-To: <1483500187-124740-1-git-send-email-beilei.xing@intel.com> Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" > -----Original Message----- > From: Xing, Beilei > Sent: Wednesday, January 4, 2017 11:23 AM > To: Wu, Jingjing ; Zhang, Helin > Cc: dev@dpdk.org > Subject: [PATCH v5 00/17] net/i40e: consistent filter API >=20 > The patch set depends on Adrien's Generic flow API(rte_flow). >=20 > The patches mainly finish following functions: > 1) Store and restore all kinds of filters. > 2) Parse all kinds of filters. > 3) Add flow validate function. > 4) Add flow create function. > 5) Add flow destroy function. > 6) Add flow flush function. >=20 > v5 changes: > Change some local variable name. > Add removing i40e_flow_list during device unint. > Fix compile error when gcc compile option isn't '-O0'. >=20 > v4 changes: > Change I40E_TCI_MASK with 0xFFFF to align with testpmd. > Modidy the stats show when restoring filters. >=20 > v3 changes: > Set the related cause pointer to a non-NULL value when error happens. > Change return value when error happens. > Modify filter_del parameter with key. > Malloc filter after checking when delete a filter. > Delete meaningless initialization. > Add return value when there's error. > Change global variable definition. > Modify some function declaration. >=20 > v2 changes: > Add i40e_flow.c, all flow ops are implemented in the file. > Change the whole implementation of all parse flow functions. > Update error info for all flow ops. > Add flow_list to store flows created. >=20 > Beilei Xing (17): > net/i40e: store ethertype filter > net/i40e: store tunnel filter > net/i40e: store flow director filter > net/i40e: restore ethertype filter > net/i40e: restore tunnel filter > net/i40e: restore flow director filter > net/i40e: add flow validate function > net/i40e: parse flow director filter > net/i40e: parse tunnel filter > net/i40e: add flow create function > net/i40e: add flow destroy function > net/i40e: destroy ethertype filter > net/i40e: destroy tunnel filter > net/i40e: destroy flow directory filter > net/i40e: add flow flush function > net/i40e: flush ethertype filters > net/i40e: flush tunnel filters >=20 > drivers/net/i40e/Makefile | 2 + > drivers/net/i40e/i40e_ethdev.c | 526 ++++++++++-- > drivers/net/i40e/i40e_ethdev.h | 173 ++++ > drivers/net/i40e/i40e_fdir.c | 140 +++- > drivers/net/i40e/i40e_flow.c | 1772 > ++++++++++++++++++++++++++++++++++++++++ > 5 files changed, 2547 insertions(+), 66 deletions(-) create mode 100644 > drivers/net/i40e/i40e_flow.c >=20 Acked-by: Jingjing Wu Thanks Jingjing