From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id EFCAC3777 for ; Wed, 28 Dec 2016 08:06:00 +0100 (CET) Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by orsmga101.jf.intel.com with ESMTP; 27 Dec 2016 23:05:59 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,421,1477983600"; d="scan'208";a="47866797" Received: from dpdk19.sh.intel.com (HELO dpdk19) ([10.239.129.113]) by fmsmga006.fm.intel.com with ESMTP; 27 Dec 2016 23:05:58 -0800 Date: Wed, 28 Dec 2016 15:00:36 +0800 From: Tiwei Bie To: "Xing, Beilei" Cc: "Wu, Jingjing" , "Zhang, Helin" , "dev@dpdk.org" Message-ID: <20161228070035.GA85898@dpdk19> References: <1480679625-4157-1-git-send-email-beilei.xing@intel.com> <1482819984-14120-1-git-send-email-beilei.xing@intel.com> <1482819984-14120-16-git-send-email-beilei.xing@intel.com> <20161228053556.GB28245@dpdk19> <94479800C636CB44BD422CB454846E013158BF48@SHSMSX101.ccr.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <94479800C636CB44BD422CB454846E013158BF48@SHSMSX101.ccr.corp.intel.com> User-Agent: Mutt/1.5.24 (2015-08-30) Subject: Re: [dpdk-dev] [PATCH v2 15/17] net/i40e: add flow flush function X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 28 Dec 2016 07:06:01 -0000 On Wed, Dec 28, 2016 at 02:48:02PM +0800, Xing, Beilei wrote: > > -----Original Message----- > > From: Bie, Tiwei > > Sent: Wednesday, December 28, 2016 1:36 PM > > To: Xing, Beilei > > Cc: Wu, Jingjing ; Zhang, Helin > > ; dev@dpdk.org > > Subject: Re: [dpdk-dev] [PATCH v2 15/17] net/i40e: add flow flush function > > > > On Tue, Dec 27, 2016 at 02:26:22PM +0800, Beilei Xing wrote: > > > This patch adds i40e_flow_flush function to flush all filters for > > > users. And flow director flush function is involved first. > > > > > > Signed-off-by: Beilei Xing > > > --- > > > drivers/net/i40e/i40e_ethdev.h | 3 +++ > > > drivers/net/i40e/i40e_fdir.c | 8 ++------ > > > drivers/net/i40e/i40e_flow.c | 46 > > ++++++++++++++++++++++++++++++++++++++++++ > > > 3 files changed, 51 insertions(+), 6 deletions(-) > > > > > > diff --git a/drivers/net/i40e/i40e_ethdev.h > > > b/drivers/net/i40e/i40e_ethdev.h index b8c7d41..0b736d5 100644 > > > --- a/drivers/net/i40e/i40e_ethdev.h > > > +++ b/drivers/net/i40e/i40e_ethdev.h > > > @@ -786,6 +786,9 @@ i40e_sw_tunnel_filter_lookup(struct > > i40e_tunnel_rule *tunnel_rule, > > > const struct i40e_tunnel_filter_input *input); int > > > i40e_sw_tunnel_filter_del(struct i40e_pf *pf, > > > struct i40e_tunnel_filter *tunnel_filter); > > > +int i40e_sw_fdir_filter_del(struct i40e_pf *pf, > > > + struct i40e_fdir_filter *filter); int > > i40e_fdir_flush(struct > > > +rte_eth_dev *dev); > > > > > > > Why don't declare them as the global functions at the beginning? > > When I implement the store/restore function, I plan this function is only used in i40e_ethdev.c. > I change them to the global functions since I add i40e_flow.c to rework all the flow ops. > These functions are also introduced in this patch set. There is no particular reason to mark them as static at first and then turn them into the global functions in the later patches. So it would be better to declare them as the global ones when introducing them. Best regards, Tiwei Bie