From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH v4 10/17] net/axgbe: add transmit and receive data path apis Date: Thu, 5 Apr 2018 12:34:54 +0100 Message-ID: <01a55d4b-8a22-eb4a-39d8-328130ede0e8@intel.com> References: <1520584954-130575-1-git-send-email-Ravi1.kumar@amd.com> <1522910389-35530-1-git-send-email-Ravi1.kumar@amd.com> <1522910389-35530-10-git-send-email-Ravi1.kumar@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit To: Ravi Kumar , dev@dpdk.org Return-path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id AAC831CB5F for ; Thu, 5 Apr 2018 13:34:57 +0200 (CEST) In-Reply-To: <1522910389-35530-10-git-send-email-Ravi1.kumar@amd.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" On 4/5/2018 7:39 AM, Ravi Kumar wrote: > Supported scalar implementation for RX data path > Supported scalar and vector implementation for TX data path > > Signed-off-by: Ravi Kumar > --- > drivers/net/axgbe/Makefile | 1 + > drivers/net/axgbe/axgbe_ethdev.c | 22 +- > drivers/net/axgbe/axgbe_rxtx.c | 429 +++++++++++++++++++++++++++++++++ > drivers/net/axgbe/axgbe_rxtx.h | 19 ++ > drivers/net/axgbe/axgbe_rxtx_vec_sse.c | 93 +++++++ > 5 files changed, 563 insertions(+), 1 deletion(-) > create mode 100644 drivers/net/axgbe/axgbe_rxtx_vec_sse.c > > diff --git a/drivers/net/axgbe/Makefile b/drivers/net/axgbe/Makefile > index 9fd7b5e..aff7917 100644 > --- a/drivers/net/axgbe/Makefile > +++ b/drivers/net/axgbe/Makefile > @@ -24,5 +24,6 @@ SRCS-$(CONFIG_RTE_LIBRTE_AXGBE_PMD) += axgbe_mdio.c > SRCS-$(CONFIG_RTE_LIBRTE_AXGBE_PMD) += axgbe_phy_impl.c > SRCS-$(CONFIG_RTE_LIBRTE_AXGBE_PMD) += axgbe_i2c.c > SRCS-$(CONFIG_RTE_LIBRTE_AXGBE_PMD) += axgbe_rxtx.c > +SRCS-$(CONFIG_RTE_LIBRTE_AXGBE_PMD) += axgbe_rxtx_vec_sse.c This needs to be protected with x86 checks. PMD is enabled by default in config, which means it will be enabled for other architectures too, like arm and ibm, and this file will cause build error for them.