From patchwork Thu Apr 6 13:46:35 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ferruh Yigit X-Patchwork-Id: 23290 Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id DA2C82B87; Thu, 6 Apr 2017 15:47:07 +0200 (CEST) Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id 1ED61108D for ; Thu, 6 Apr 2017 15:47:05 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=intel.com; i=@intel.com; q=dns/txt; s=intel; t=1491486426; x=1523022426; h=from:to:cc:subject:date:message-id:in-reply-to: references; bh=ve+EI0+KkzSSfe3QrYjZhasHi7Z6k7MWgZh/GH1dQqw=; b=nCRkWljd3x+rD817n2y7iJgkEYZLSv5pftt+M32KdumaJ8oNKqgfcY+w XWrmUnq3c4PXtJBbizDpfQ0j/1UfJQ==; Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 06 Apr 2017 06:47:04 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos; i="5.37,160,1488873600"; d="scan'208"; a="1116303598" Received: from sivswdev02.ir.intel.com ([10.237.217.46]) by orsmga001.jf.intel.com with ESMTP; 06 Apr 2017 06:46:38 -0700 From: Ferruh Yigit To: Jerin Jacob , Bruce Richardson , Harry van Haaren , Thomas Monjalon Cc: dev@dpdk.org, Ferruh Yigit Date: Thu, 6 Apr 2017 14:46:35 +0100 Message-Id: <20170406134637.4484-1-ferruh.yigit@intel.com> X-Mailer: git-send-email 2.8.4 In-Reply-To: <20170406121428.16883-1-ferruh.yigit@intel.com> References: <20170406121428.16883-1-ferruh.yigit@intel.com> Subject: [dpdk-dev] [PATCH v2 1/3] net/thunderx: disable pmd for older compilers 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Disable for gcc < 4.7 and icc <= 14.0 PMD uses some compiler builtins and new compiler options. Tested with gcc 4.5.1 and following were not supported: option: -Ofast macros: _Static_assert __ORDER_LITTLE_ENDIAN__ __ORDER_BIG_ENDIAN__ __BYTE_ORDER__ __atomic_fetch_add __ATOMIC_ACQUIRE __atomic_load_n __ATOMIC_RELAXED __atomic_store_n __ATOMIC_RELEASE It is not easy to fix all in PMD, disabling PMD for older compilers. Signed-off-by: Ferruh Yigit --- drivers/net/thunderx/Makefile | 15 +++++++++++++++ mk/toolchain/gcc/rte.toolchain-compat.mk | 4 ++++ mk/toolchain/icc/rte.toolchain-compat.mk | 4 ++++ 3 files changed, 23 insertions(+) diff --git a/drivers/net/thunderx/Makefile b/drivers/net/thunderx/Makefile index 706250b..f2acfcb 100644 --- a/drivers/net/thunderx/Makefile +++ b/drivers/net/thunderx/Makefile @@ -60,8 +60,23 @@ SRCS-$(CONFIG_RTE_LIBRTE_THUNDERX_NICVF_PMD) += nicvf_ethdev.c SRCS-$(CONFIG_RTE_LIBRTE_THUNDERX_NICVF_PMD) += nicvf_bsvf.c SRCS-$(CONFIG_RTE_LIBRTE_THUNDERX_NICVF_PMD) += nicvf_svf.c +# GCC ifeq ($(CONFIG_RTE_TOOLCHAIN_GCC),y) CFLAGS_nicvf_rxtx.o += -fno-prefetch-loop-arrays + +# Disable PMD for gcc < 4.7 in mk/toolchain/gcc/rte.toolchain-compat.mk +ifeq ($(shell test $(GCC_VERSION) -lt 47 && echo 1), 1) +$(warning thunderx pmd not supported by gcc < 4.7) +endif + +# ICC +else ifeq ($(CONFIG_RTE_TOOLCHAIN_ICC),y) + +# Disable PMD for icc <= 14.0 in mk/toolchain/gcc/rte.toolchain-compat.mk +ifeq ($(shell test $(ICC_MAJOR_VERSION) -le 14 && echo 1), 1) +$(warning thunderx pmd not supported by icc <= 14.0) +endif + endif CFLAGS_nicvf_rxtx.o += -Ofast diff --git a/mk/toolchain/gcc/rte.toolchain-compat.mk b/mk/toolchain/gcc/rte.toolchain-compat.mk index 6eed20c..fff14e1 100644 --- a/mk/toolchain/gcc/rte.toolchain-compat.mk +++ b/mk/toolchain/gcc/rte.toolchain-compat.mk @@ -89,4 +89,8 @@ else ifeq ($(shell test $(GCC_VERSION) -lt 42 && echo 1), 1) MACHINE_CFLAGS := $(filter-out -march% -mtune% -msse%,$(MACHINE_CFLAGS)) endif + + ifeq ($(shell test $(GCC_VERSION) -lt 47 && echo 1), 1) + CONFIG_RTE_LIBRTE_THUNDERX_NICVF_PMD=n + endif endif diff --git a/mk/toolchain/icc/rte.toolchain-compat.mk b/mk/toolchain/icc/rte.toolchain-compat.mk index 4134466..2d0e54b 100644 --- a/mk/toolchain/icc/rte.toolchain-compat.mk +++ b/mk/toolchain/icc/rte.toolchain-compat.mk @@ -64,6 +64,10 @@ else MACHINE_CFLAGS := $(patsubst -march=core-avx2,-xCORE-AVX2,$(MACHINE_CFLAGS)) # remove westmere flags MACHINE_CFLAGS := $(filter-out -mpclmul -maes,$(MACHINE_CFLAGS)) + + ifeq ($(shell test $(ICC_MAJOR_VERSION) -le 14 && echo 1), 1) + CONFIG_RTE_LIBRTE_THUNDERX_NICVF_PMD=n + endif endif # if icc version == 12.0 ifeq ($(shell test $(ICC_MAJOR_VERSION) -eq 12 && test $(ICC_MINOR_VERSION) -eq 0 && echo 1), 1)