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 F2EA7A04F3; Wed, 8 Jan 2020 17:06:13 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id D6F7A1DA91; Wed, 8 Jan 2020 17:06:13 +0100 (CET) Received: from us-smtp-1.mimecast.com (us-smtp-delivery-1.mimecast.com [207.211.31.120]) by dpdk.org (Postfix) with ESMTP id 08A841DA8B for ; Wed, 8 Jan 2020 17:06:12 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1578499572; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=XOFawCvL7aIx/Q5Pfl+uXw/9IpBv+VQQrq5uSNlR+fc=; b=cWu1Gs+48JPGXXvMR/ako4HGySYPc8x5HBHt/KCp6YYkrXoUZ0GElow5ozeziwq/fNJaWH pZPIuy/TdBgUcKCd+Vo9Wvl2roV2X5wf1lBbg8xqDOPhRCYOonJ08UViU/CXQ2Y08tkhXC Ofc/psHgsko52r8tZQOQgchrmN0ULwM= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-289-Vv_udwI_MTC8wrtpQUCzhQ-1; Wed, 08 Jan 2020 11:06:09 -0500 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 7C3668CD008; Wed, 8 Jan 2020 16:06:07 +0000 (UTC) Received: from dhcp-25.97.bos.redhat.com (unknown [10.18.25.108]) by smtp.corp.redhat.com (Postfix) with ESMTPS id E37F6601B3; Wed, 8 Jan 2020 16:06:03 +0000 (UTC) From: Aaron Conole To: Honnappa Nagarahalli Cc: Ruifeng Wang , "maicolgabriel\@hotmail.com" , "thomas\@monjalon.net" , "ferruh.yigit\@intel.com" , "arybchenko\@solarflare.com" , "dev\@dpdk.org" , "david.marchand\@redhat.com" , Gavin Hu , nd References: <20191218053902.193417-1-ruifeng.wang@arm.com> <20191223070833.144628-1-ruifeng.wang@arm.com> <20191223070833.144628-3-ruifeng.wang@arm.com> Date: Wed, 08 Jan 2020 11:06:03 -0500 In-Reply-To: (Honnappa Nagarahalli's message of "Tue, 7 Jan 2020 14:40:09 +0000") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux) MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 X-MC-Unique: Vv_udwI_MTC8wrtpQUCzhQ-1 X-Mimecast-Spam-Score: 0 Content-Type: text/plain Content-Transfer-Encoding: quoted-printable Subject: Re: [dpdk-dev] [PATCH v3 2/2] ci: add travis ci support for aarch64 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" Honnappa Nagarahalli writes: > > >> > >> > > Add Travis compilation jobs for aarch64. gcc/clang compilations for >> > > static/shared libraries are added. >> > > >> > > Some limitations for current aarch64 Travis support: >> > > 1. Container is used. Huge page is not available due to security rea= son. >> > > 2. Missing kernel header package in Xenial distribution. >> > > >> > > Solutions to address the limitations: >> > > 1. Not to add unit test for now. And run tests with no-huge in futur= e. >> > > 2. Use Bionic distribution for all aarch64 jobs. >> > > >> > > Signed-off-by: Ruifeng Wang >> > > Reviewed-by: Gavin Hu >> > > --- >> > >> > Can't we achieve the same thing by setting >> > >> > arch: >> > - amd64 >> > - arm64 >> > >> > in the build matrix? Or will that also force the intel builds to use >> > the container infrastructure (in which case the no-huge support needs = to be >> fixed)? >>=20 >> No, container infrastructure will not be imposed to intel builds. >> AFAIN, Travis infrastructure for a specific CPU arch is provided as is, = and >> there is no config option to control. >> The problem with just adding 'arch' in build matrix is that RUN_TESTS on >> arm64 is not supported by now (Travis limitation). 'env' with RUN_TESTS = will >> fail. >> > >> > One thing I wonder, isn't is possible to use qemu-user to do the amd64 >> > unit tests? Then do we really need some changes to do the native buil= d? >>=20 >> Do you mean to use qemu-user to do unit tests for non-x86 arch? >> Changes will be needed as well to enable qemu-user to do unit test. >> Since Travis support multi CPU arch, I think native build and test is si= mpler >> and more natural. > Yes, prefer to run the tests natively as the infrastructure is > available and will improve further from here. Agreed. But we can't run the aarch64 tests for now, no matter which way we slice it. >>=20 >> > Does it buy us anything *today* given the cost of the hugepage restric= tion? >> > Will that ever be resolved (I didn't see so from the docs on travis)? >>=20 >> The hugepage issue has been reported to Travis. I think it will be resol= ved. >> But no set dates yet. >> > >> > > .ci/linux-setup.sh | 11 +++++++---- >> > > .travis.yml | 42 +++++++++++++++++++++++++++++++++++++++++- >> > > 2 files changed, 48 insertions(+), 5 deletions(-) >> > > >> > > diff --git a/.ci/linux-setup.sh b/.ci/linux-setup.sh index >> > > dfb9d4a20..a92978037 100755 >> > > --- a/.ci/linux-setup.sh >> > > +++ b/.ci/linux-setup.sh >> > > @@ -3,7 +3,10 @@ >> > > # need to install as 'root' since some of the unit tests won't run >> > > without it sudo python3 -m pip install --upgrade meson >> > > >> > > -# setup hugepages >> > > -cat /proc/meminfo >> > > -sudo sh -c 'echo 1024 > /proc/sys/vm/nr_hugepages' >> > > -cat /proc/meminfo >> > > +# hugepage settings are skipped on aarch64 due to environment >> > > +limitation if [ "$TRAVIS_ARCH" !=3D "aarch64" ]; then >> > > + # setup hugepages >> > > + cat /proc/meminfo >> > > + sudo sh -c 'echo 1024 > /proc/sys/vm/nr_hugepages' >> > > + cat /proc/meminfo >> > > +fi >> > > diff --git a/.travis.yml b/.travis.yml index 8f90d06f2..980c7605d >> > > 100644 >> > > --- a/.travis.yml >> > > +++ b/.travis.yml >> > > @@ -115,6 +115,46 @@ matrix: >> > > apt: >> > > packages: >> > > - *extra_packages >> > > - >> > > + - env: DEF_LIB=3D"static" >> > > + arch: arm64 >> > > + compiler: gcc >> > > + dist: bionic >> > > + addons: >> > > + apt: >> > > + packages: >> > > + - *required_packages >> > > + - env: DEF_LIB=3D"shared" >> > > + arch: arm64 >> > > + compiler: gcc >> > > + dist: bionic >> > > + addons: >> > > + apt: >> > > + packages: >> > > + - *required_packages >> > > + - env: DEF_LIB=3D"static" >> > > + arch: arm64 >> > > + dist: bionic >> > > + compiler: clang >> > > + addons: >> > > + apt: >> > > + packages: >> > > + - *required_packages >> > > + - env: DEF_LIB=3D"shared" >> > > + arch: arm64 >> > > + dist: bionic >> > > + compiler: clang >> > > + addons: >> > > + apt: >> > > + packages: >> > > + - *required_packages >> > > + - env: DEF_LIB=3D"shared" OPTS=3D"-Denable_kmods=3Dfalse" BUILD_D= OCS=3D1 >> > > + arch: arm64 >> > > + compiler: gcc >> > > + dist: bionic >> > > + addons: >> > > + apt: >> > > + packages: >> > > + - *required_packages >> > > + - *doc_packages >> > > >> > > script: ./.ci/${TRAVIS_OS_NAME}-build.sh >>=20