> -----Original Message----- > From: Juraj Linkeš <juraj.lin...@pantheon.tech> > Sent: Wednesday, November 11, 2020 7:46 PM > To: Honnappa Nagarahalli <honnappa.nagaraha...@arm.com>; > bruce.richard...@intel.com; Ruifeng Wang <ruifeng.w...@arm.com>; Phil > Yang <phil.y...@arm.com>; vcchu...@amazon.com; Dharmik Thakkar > <dharmik.thak...@arm.com>; jerinjac...@gmail.com; > hemant.agra...@nxp.com; Ajit Khaparde (ajit.khapa...@broadcom.com) > <ajit.khapa...@broadcom.com>; ferruh.yi...@intel.com; > acon...@redhat.com > Cc: dev@dpdk.org; nd <n...@arm.com>; nd <n...@arm.com> > Subject: RE: [PATCH v8 09/14] ci: switch to generic Arm builds > > > > > -----Original Message----- > > From: Honnappa Nagarahalli <honnappa.nagaraha...@arm.com> > > Sent: Sunday, November 8, 2020 3:43 PM > > To: Juraj Linkeš <juraj.lin...@pantheon.tech>; > > bruce.richard...@intel.com; Ruifeng Wang <ruifeng.w...@arm.com>; > Phil > > Yang <phil.y...@arm.com>; vcchu...@amazon.com; Dharmik Thakkar > > <dharmik.thak...@arm.com>; jerinjac...@gmail.com; > > hemant.agra...@nxp.com; Ajit Khaparde > > (ajit.khapa...@broadcom.com) <ajit.khapa...@broadcom.com>; > > ferruh.yi...@intel.com; acon...@redhat.com > > Cc: dev@dpdk.org; nd <n...@arm.com>; Honnappa Nagarahalli > > <honnappa.nagaraha...@arm.com>; nd <n...@arm.com> > > Subject: RE: [PATCH v8 09/14] ci: switch to generic Arm builds > > > > <snip> > > > > > > > > Switch to generic build on arm Travis machines to avoid differences > > > in build configuration caused by different Arm hardware. > > > > > > Signed-off-by: Juraj Linkeš <juraj.lin...@pantheon.tech> > > > --- > > > .ci/linux-build.sh | 4 ++++ > > > 1 file changed, 4 insertions(+) > > > > > > diff --git a/.ci/linux-build.sh b/.ci/linux-build.sh index > > > d079801d7..7fe0fcfd8 > > > 100755 > > > --- a/.ci/linux-build.sh > > > +++ b/.ci/linux-build.sh > > > @@ -33,6 +33,10 @@ if [ "$AARCH64" = "1" ]; then > > > OPTS="$OPTS --cross-file config/arm/arm64_armv8_linux_gcc" > > > fi > > > > > > +if [ "$(uname -m)" = "aarch64" ]; then > > > + OPTS="$OPTS -Dmachine=generic" > > > +fi > > > + > > I do not think we should change it by default. We would need to test > > generic builds as well as the SoC specific build. > > > > I'll drop the patch. Testing without it revealed a failure in clang Graviton2 > build (only in that one build, other build are working fine): > ../lib/librte_eal/arm/include/rte_atomic_64.h:76:39: error: instruction > requires: lse > > Here's the build with full log: https://travis- > ci.com/github/jlinkes/dpdk/jobs/433176503 > > I think we need to add a commit that fixes the failure but I don't know how > to fix it.
Can you try '-mcpu=native' instead of '-march=native -mtune=native' for native build? Clang doesn't support '-march=native'. Reference can be found here: https://community.arm.com/developer/tools-software/tools/b/tools-software-ides-blog/posts/compiler-flags-across-architectures-march-mtune-and-mcpu > > > > > > if [ "$BUILD_DOCS" = "1" ]; then > > > OPTS="$OPTS -Denable_docs=true" > > > fi > > > -- > > > 2.20.1
