Re: [dpdk-dev] [PATCH v3 2/2] ci: add travis ci support for aarch64
From: Aaron Conole <aconole@redhat.com>
Date: 2020-01-06 13:34:36
Ruifeng Wang [off-list ref] 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 reason. 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 future. 2. Use Bionic distribution for all aarch64 jobs. Signed-off-by: Ruifeng Wang <redacted> Reviewed-by: Gavin Hu <redacted> ---
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)? 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 build? Does it buy us anything *today* given the cost of the hugepage restriction? Will that ever be resolved (I didn't see so from the docs on travis)?
quoted hunk ↗ jump to hunk
.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" != "aarch64" ]; then + # setup hugepages + cat /proc/meminfo + sudo sh -c 'echo 1024 > /proc/sys/vm/nr_hugepages' + cat /proc/meminfo +fidiff --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="static" + arch: arm64 + compiler: gcc + dist: bionic + addons: + apt: + packages: + - *required_packages + - env: DEF_LIB="shared" + arch: arm64 + compiler: gcc + dist: bionic + addons: + apt: + packages: + - *required_packages + - env: DEF_LIB="static" + arch: arm64 + dist: bionic + compiler: clang + addons: + apt: + packages: + - *required_packages + - env: DEF_LIB="shared" + arch: arm64 + dist: bionic + compiler: clang + addons: + apt: + packages: + - *required_packages + - env: DEF_LIB="shared" OPTS="-Denable_kmods=false" BUILD_DOCS=1 + arch: arm64 + compiler: gcc + dist: bionic + addons: + apt: + packages: + - *required_packages + - *doc_packages script: ./.ci/${TRAVIS_OS_NAME}-build.sh