From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id DEFF42BE0 for ; Thu, 26 May 2016 12:40:37 +0200 (CEST) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga102.fm.intel.com with ESMTP; 26 May 2016 03:40:36 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.26,367,1459839600"; d="scan'208";a="974916454" Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.220.104]) by fmsmga001.fm.intel.com with SMTP; 26 May 2016 03:40:35 -0700 Received: by (sSMTP sendmail emulation); Thu, 26 May 2016 11:40:34 +0025 Date: Thu, 26 May 2016 11:40:34 +0100 From: Bruce Richardson To: Stephen Hurd Cc: dev@dpdk.org Message-ID: <20160526104034.GE11928@bricha3-MOBL3> References: <1463179589-82681-1-git-send-email-stephen.hurd@broadcom.com> <1463179589-82681-13-git-send-email-stephen.hurd@broadcom.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1463179589-82681-13-git-send-email-stephen.hurd@broadcom.com> Organization: Intel Shannon Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Subject: Re: [dpdk-dev] [PATCH v2 13/40] bnxt: initial Tx ring code X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 26 May 2016 10:40:38 -0000 On Fri, May 13, 2016 at 03:46:02PM -0700, Stephen Hurd wrote: > Initial implementation of rx_pkt_burst typo: s/rx/tx/ > Add code to allocate rings to bnxt_ring.c > > Signed-off-by: Stephen Hurd > Reviewed-by: Ajit Kumar Khaparde > --- > drivers/net/bnxt/Makefile | 1 + > drivers/net/bnxt/bnxt_cpr.h | 4 +- > drivers/net/bnxt/bnxt_ethdev.c | 5 +- > drivers/net/bnxt/bnxt_ring.c | 140 +++++++++ > drivers/net/bnxt/bnxt_ring.h | 8 + > drivers/net/bnxt/bnxt_txq.c | 42 ++- > drivers/net/bnxt/bnxt_txr.c | 314 ++++++++++++++++++++ > drivers/net/bnxt/bnxt_txr.h | 71 +++++ > drivers/net/bnxt/hsi_struct_def_dpdk.h | 512 +++++++++++++++++++++++++++++++++ > 9 files changed, 1086 insertions(+), 11 deletions(-) > create mode 100644 drivers/net/bnxt/bnxt_txr.c > create mode 100644 drivers/net/bnxt/bnxt_txr.h > > diff --git a/drivers/net/bnxt/Makefile b/drivers/net/bnxt/Makefile > index f6a04f8..0785681 100644 > --- a/drivers/net/bnxt/Makefile > +++ b/drivers/net/bnxt/Makefile > @@ -56,6 +56,7 @@ SRCS-$(CONFIG_RTE_LIBRTE_BNXT_PMD) += bnxt_ring.c > SRCS-$(CONFIG_RTE_LIBRTE_BNXT_PMD) += bnxt_rxq.c > SRCS-$(CONFIG_RTE_LIBRTE_BNXT_PMD) += bnxt_stats.c > SRCS-$(CONFIG_RTE_LIBRTE_BNXT_PMD) += bnxt_txq.c > +SRCS-$(CONFIG_RTE_LIBRTE_BNXT_PMD) += bnxt_txr.c > SRCS-$(CONFIG_RTE_LIBRTE_BNXT_PMD) += bnxt_vnic.c > > # > diff --git a/drivers/net/bnxt/bnxt_cpr.h b/drivers/net/bnxt/bnxt_cpr.h > index e6333fc..f104281 100644 > --- a/drivers/net/bnxt/bnxt_cpr.h > +++ b/drivers/net/bnxt/bnxt_cpr.h > @@ -51,11 +51,11 @@ > > #define B_CP_DB_REARM(cpr, raw_cons) \ > (*(uint32_t *)((cpr)->cp_doorbell) = (DB_CP_REARM_FLAGS | \ > - RING_CMP(&cpr->cp_ring_struct, raw_cons))) > + RING_CMP(cpr->cp_ring_struct, raw_cons))) > > #define B_CP_DIS_DB(cpr, raw_cons) \ > (*(uint32_t *)((cpr)->cp_doorbell) = (DB_CP_FLAGS | \ > - RING_CMP(&cpr->cp_ring_struct, raw_cons))) > + RING_CMP(cpr->cp_ring_struct, raw_cons))) > > struct bnxt_ring_struct; > struct bnxt_cp_ring_info { > diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c > index 786318c..61e856a 100644 > --- a/drivers/net/bnxt/bnxt_ethdev.c > +++ b/drivers/net/bnxt/bnxt_ethdev.c > @@ -44,6 +44,7 @@ > #include "bnxt_rxq.h" > #include "bnxt_stats.h" > #include "bnxt_txq.h" > +#include "bnxt_txr.h" > > #define DRV_MODULE_NAME "bnxt" > static const char bnxt_version[] = > @@ -259,10 +260,8 @@ bnxt_dev_init(struct rte_eth_dev *eth_dev) > goto error; > } > eth_dev->dev_ops = &bnxt_dev_ops; > - /* > - eth_dev->rx_pkt_burst = &bnxt_recv_pkts; > + /* eth_dev->rx_pkt_burst = &bnxt_recv_pkts; */ > eth_dev->tx_pkt_burst = &bnxt_xmit_pkts; > - */ When adding these two lines in the earlier patch, I would suggest just adding them as two separate commented out lines, which will both clear the checkpatch issue flagged before, as well as simplifying the diff here. After this patch is applied, does this mean that the tx burst functions call be successfully called on the NIC? /Bruce