From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 4341FA0524; Fri, 27 Nov 2020 10:39:27 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 1AFAEC938; Fri, 27 Nov 2020 10:39:26 +0100 (CET) Received: from shelob.oktetlabs.ru (shelob.oktetlabs.ru [91.220.146.113]) by dpdk.org (Postfix) with ESMTP id 7FBFCC930 for ; Fri, 27 Nov 2020 10:39:23 +0100 (CET) Received: from [192.168.38.17] (aros.oktetlabs.ru [192.168.38.17]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by shelob.oktetlabs.ru (Postfix) with ESMTPSA id 0D7757F54B; Fri, 27 Nov 2020 12:39:22 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 shelob.oktetlabs.ru 0D7757F54B DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=oktetlabs.ru; s=default; t=1606469962; bh=MSHCwWI4ID5zHL/sE8rhQlsImtn1PX3tW+Dz+DdQ55M=; h=Subject:To:Cc:References:From:Date:In-Reply-To; b=EHt6NnXujhSDAMkTzbER5EShuLpSkCtmR43Aw7FxFFa5zZGE5b/yXxre8y7PToDdD DLu1Ew6aulXixQ5n5McGL8892CIqRe6gkunRemIPvX68sp33CDzYbNEVjovs0YPPtx S9x+fy9EjnSqQ5xe5Y/uwhuLi7pdsLdLpJvuYysw= To: David Marchand , Ruifeng Wang Cc: Jerin Jacob , Honnappa Nagarahalli , dev , nd , Feifei Wang , Jerin Jacob References: <20201127080903.26817-1-ruifeng.wang@arm.com> <20201127080903.26817-2-ruifeng.wang@arm.com> From: Andrew Rybchenko Organization: OKTET Labs Message-ID: Date: Fri, 27 Nov 2020 12:39:21 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.4.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH 1/2] eal/arm: fix gcc build for optimization level 0 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" On 11/27/20 12:25 PM, David Marchand wrote: > On Fri, Nov 27, 2020 at 9:51 AM Jerin Jacob wrote: >> >> On Fri, Nov 27, 2020 at 2:09 PM David Marchand >> wrote: >>> >>> On Fri, Nov 27, 2020 at 9:10 AM Ruifeng Wang wrote: >>>> >>>> Gcc build with '-O0' on platforms with RTE_ARM_FEATURE_ATOMICS set >>>> failed for: >>>> ../lib/librte_efd/rte_efd.c >>>> Assembler messages: >>>> 3866: Error: selected processor does not support `crc32cb w0,w0,w1' >>>> 3890: Error: selected processor does not support `crc32ch w0,w0,w1' >>>> 3914: Error: selected processor does not support `crc32cw w0,w0,w1' >>>> 3938: Error: selected processor does not support `crc32cx w0,w0,x1' >>>> >>>> This was caused by an architecture specifier added for Clang. >>>> Unlike Clang, Gcc considers each inline assembly block to be dependent >>>> and therefor, the architecture specifier impacts assemble of some >>>> blocks require certain extension support. >>> >>> I reproduced this error with -O0 for config/arm/arm64_octeontx2_linux_gcc. >>> The patch looks simple enough and it only affects ARM builds. >>> >>> Jerin, >>> Are you okay with this change for 20.11? >>> Or should we wait 21.02? >> >> I could reproduce the issue with meson build -Dbuildtype=debug >> --cross-file config/arm/arm64_octeontx2_linux_gcc >> >> This fix is OK. Request to merge for 20.11 >> >> Acked-by: Jerin Jacob > > Ruifeng, Andrew, > > I get a compilation error with this patch on sfc using gcc and > config/arm/arm64_octeontx2_linux_gcc config + --buildtype=debug and/or > -Dc_args=-O0 . > > aarch64-linux-gnu-gcc (GNU Toolchain for the A-profile Architecture > 8.2-2019.01 (arm-rel-8.28)) 8.2.1 20180802 > > FAILED: drivers/common/sfc_efx/base/932d09a@@sfc_base@sta/efx_crc32.c.o > aarch64-linux-gnu-gcc > -Idrivers/common/sfc_efx/base/932d09a@@sfc_base@sta > -Idrivers/common/sfc_efx/base -I../../dpdk/drivers/common/sfc_efx/base > -Idrivers/common/sfc_efx -I../../dpdk/drivers/common/sfc_efx -I. > -I../../dpdk/ -Iconfig -I../../dpdk/config -Ilib/librte_eal/include > -I../../dpdk/lib/librte_eal/include -Ilib/librte_eal/linux/include > -I../../dpdk/lib/librte_eal/linux/include -Ilib/librte_eal/arm/include > -I../../dpdk/lib/librte_eal/arm/include -Ilib/librte_eal/common > -I../../dpdk/lib/librte_eal/common -Ilib/librte_eal > -I../../dpdk/lib/librte_eal -Ilib/librte_kvargs > -I../../dpdk/lib/librte_kvargs > -Ilib/librte_telemetry/../librte_metrics > -I../../dpdk/lib/librte_telemetry/../librte_metrics > -Ilib/librte_telemetry -I../../dpdk/lib/librte_telemetry > -fdiagnostics-color=always -pipe -D_FILE_OFFSET_BITS=64 -Wall > -Winvalid-pch -Werror -g -include rte_config.h -Wextra -Wcast-qual > -Wdeprecated -Wformat -Wformat-nonliteral -Wformat-security > -Wmissing-declarations -Wmissing-prototypes -Wnested-externs > -Wold-style-definition -Wpointer-arith -Wsign-compare > -Wstrict-prototypes -Wundef -Wwrite-strings -Wno-packed-not-aligned > -Wno-missing-field-initializers -D_GNU_SOURCE -fPIC > -march=armv8.2-a+crc+crypto+lse -DALLOW_EXPERIMENTAL_API > -DALLOW_INTERNAL_API -Wno-format-truncation -Wdisabled-optimization > -Waggregate-return -Wbad-function-cast -Wno-sign-compare > -Wno-unused-parameter -Wno-unused-variable -Wno-empty-body > -Wno-unused-but-set-variable -MD -MQ > 'drivers/common/sfc_efx/base/932d09a@@sfc_base@sta/efx_crc32.c.o' -MF > 'drivers/common/sfc_efx/base/932d09a@@sfc_base@sta/efx_crc32.c.o.d' -o > 'drivers/common/sfc_efx/base/932d09a@@sfc_base@sta/efx_crc32.c.o' -c > ../../dpdk/drivers/common/sfc_efx/base/efx_crc32.c > In file included from ../../dpdk/lib/librte_eal/arm/include/rte_atomic.h:9, > from ../../dpdk/lib/librte_eal/include/generic/rte_pause.h:19, > from ../../dpdk/lib/librte_eal/arm/include/rte_pause_64.h:19, > from ../../dpdk/lib/librte_eal/arm/include/rte_pause.h:13, > from > ../../dpdk/lib/librte_eal/include/generic/rte_spinlock.h:25, > from ../../dpdk/lib/librte_eal/arm/include/rte_spinlock.h:17, > from ../../dpdk/drivers/common/sfc_efx/efsys.h:15, > from ../../dpdk/drivers/common/sfc_efx/base/efx.h:11, > from ../../dpdk/drivers/common/sfc_efx/base/efx_crc32.c:7: ... > If I waive those warnings: > - next to fail is net/igc (fixed by next patch so ok) > - and then next to fail is net/sfc. As I understand it is not common/sfc_efx specific issue since it is simply header inclusion. I guess it is the just first one to build. I have no good ideas what to do with it. So, help from ARM experts would be appreciated. Andrew.