> -----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. > > > if [ "$BUILD_DOCS" = "1" ]; then > > OPTS="$OPTS -Denable_docs=true" > > fi > > -- > > 2.20.1
