From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id 480AD2986 for ; Wed, 17 May 2017 16:43:43 +0200 (CEST) Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga104.jf.intel.com with ESMTP; 17 May 2017 07:43:43 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.38,354,1491289200"; d="scan'208";a="88397387" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.81]) ([10.237.220.81]) by orsmga004.jf.intel.com with ESMTP; 17 May 2017 07:43:41 -0700 To: Michael Lilja , helin.zhang@intel.com, jingjing.wu@intel.com Cc: dev@dpdk.org References: <20170517103807.18746-1-ml@napatech.com> <20170517143120.18306-1-ml@napatech.com> From: Ferruh Yigit Message-ID: Date: Wed, 17 May 2017 15:43:41 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: <20170517143120.18306-1-ml@napatech.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH v7] net/i40e: improved FDIR programming times 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, 17 May 2017 14:43:44 -0000 On 5/17/2017 3:31 PM, Michael Lilja wrote: > Previously, the FDIR programming time is +11ms on i40e. > This patch will result in an average programming time of > 22usec with a max of 60usec . > > Signed-off-by: Michael Lilja Sorry for multiple, minor change requests ... > > --- > v7: > * Code style changes > > v6: > * Fixed code style issues > > v5: > * Reinitialization of "i" inconsistent with original intent > > v4: > * Code style fix > > v3: > * Replaced commit message > > v2: > * Code style fix > > v1: > * Initial version > --- > --- > drivers/net/i40e/i40e_fdir.c | 22 +++++++++++----------- > 1 file changed, 11 insertions(+), 11 deletions(-) > > diff --git a/drivers/net/i40e/i40e_fdir.c b/drivers/net/i40e/i40e_fdir.c > index 28cc554f5..1192d5831 100644 > --- a/drivers/net/i40e/i40e_fdir.c > +++ b/drivers/net/i40e/i40e_fdir.c > @@ -76,6 +76,7 @@ > /* Wait count and interval for fdir filter programming */ > #define I40E_FDIR_WAIT_COUNT 10 > #define I40E_FDIR_WAIT_INTERVAL_US 1000 > +#define I40E_FDIR_MAX_WAIT (I40E_FDIR_WAIT_COUNT * I40E_FDIR_WAIT_INTERVAL_US) It looks like I40E_FDIR_WAIT_COUNT and I40E_FDIR_WAIT_INTERVAL_US not used anywhere else, is there any value to keep them? why not: #define I40E_FDIR_MAX_WAIT_US 10000 /* 10 ms */ > > /* Wait count and interval for fdir filter flush */ > #define I40E_FDIR_FLUSH_RETRY 50 > @@ -1295,28 +1296,27 @@ i40e_fdir_filter_programming(struct i40e_pf *pf, > /* Update the tx tail register */ > rte_wmb(); > I40E_PCI_REG_WRITE(txq->qtx_tail, txq->tx_tail); > - > - for (i = 0; i < I40E_FDIR_WAIT_COUNT; i++) { > - rte_delay_us(I40E_FDIR_WAIT_INTERVAL_US); > + for (i = 0; i < I40E_FDIR_MAX_WAIT; i++) { > if ((txdp->cmd_type_offset_bsz & > rte_cpu_to_le_64(I40E_TXD_QW1_DTYPE_MASK)) == > rte_cpu_to_le_64(I40E_TX_DESC_DTYPE_DESC_DONE)) > break; > + rte_delay_us(1); > } > - if (i >= I40E_FDIR_WAIT_COUNT) { > + if (i >= I40E_FDIR_MAX_WAIT) { > PMD_DRV_LOG(ERR, "Failed to program FDIR filter:" > " time out to get DD on tx queue."); > return -ETIMEDOUT; > } > /* totally delay 10 ms to check programming status*/ > - rte_delay_us((I40E_FDIR_WAIT_COUNT - i) * I40E_FDIR_WAIT_INTERVAL_US); > - if (i40e_check_fdir_programming_status(rxq) < 0) { > - PMD_DRV_LOG(ERR, "Failed to program FDIR filter:" > - " programming status reported."); > - return -ENOSYS; > + for (; i < I40E_FDIR_MAX_WAIT; i++) { > + if (i40e_check_fdir_programming_status(rxq) >= 0) > + return 0; > + rte_delay_us(1); > } > - > - return 0; > + PMD_DRV_LOG(ERR, "Failed to program FDIR filter:" > + " programming status reported."); > + return -ETIMEDOUT; > } > > /* >