Merge pull request #1878 from pguyot/w40/run-tests-on-aarch64-runner

Use new arm64 runners for Linux tests

Continuation of:
- #1890

These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).

SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later

Author: bettio

.github/workflows/build-and-test-other.yaml

@@ -70,7 +70,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        arch: ["arm32v5", "arm32v7", "arm64v8"]
+        arch: ["arm32v5", "arm32v7"]
 
         include:
         - arch: "arm32v5"
@@ -100,12 +100,6 @@ jobs:
           cflags: "-mcpu=cortex-a7 -mfloat-abi=hard -O2 -mthumb -mthumb-interwork -D_FILE_OFFSET_BITS=64"
           cmake_opts: "-DAVM_WARNINGS_ARE_ERRORS=ON"
 
-        - arch: "arm64v8"
-          platform: "arm64/v8"
-          tag: "bookworm"
-          cflags: "-O2"
-          cmake_opts: "-DAVM_WARNINGS_ARE_ERRORS=ON"
-
         # Required for testing big endian archs
         - arch: "s390x"
           platform: "s390x"

.github/workflows/build-and-test.yaml

@@ -301,6 +301,26 @@ jobs:
           cmake_opts_other: "-DAVM_DISABLE_JIT=OFF"
           jit_target_arch: "x86_64"
 
+        # arm64 builds
+        - os: "ubuntu-24.04-arm"
+          cc: "cc"
+          cxx: "c++"
+          cflags: "-O2"
+          otp: "28"
+          elixir_version: "1.17"
+          rebar3_version: "3.24.0"
+          cmake_opts_other: "-DAVM_WARNINGS_ARE_ERRORS=OFF"
+
+        - os: "ubuntu-24.04-arm"
+          cc: "cc"
+          cxx: "c++"
+          cflags: ""
+          otp: "28"
+          elixir_version: "1.17"
+          rebar3_version: "3.24.0"
+          cmake_opts_other: "-DAVM_DISABLE_JIT=OFF"
+          jit_target_arch: "aarch64"
+
     env:
       ImageOS: ${{ matrix.container == 'ubuntu:20.04' && 'ubuntu20' || matrix.os == 'ubuntu-20.04' && 'ubuntu20' || matrix.os == 'ubuntu-22.04' && 'ubuntu22' || matrix.os == 'ubuntu-24.04' && 'ubuntu24' || 'ubuntu24' }}
       CC: ${{ matrix.cc }}
@@ -324,8 +344,16 @@ jobs:
       run: sudo apt update -y
 
     - name: "Install deps"
+      if: matrix.container != ''
       run: sudo apt install -y ${{ matrix.compiler_pkgs}} cmake gperf zlib1g-dev doxygen valgrind libmbedtls-dev
 
+    - name: "Install deps"
+      if: matrix.container == ''
+      run: |
+        sudo apt install -y ${{ matrix.compiler_pkgs}} cmake gperf zlib1g-dev doxygen libmbedtls-dev libc6-dbg
+        # Get a more recent valgrind
+        sudo snap install valgrind --classic
+
     - name: "Checkout repo"
       uses: actions/checkout@v4
       with:
@@ -387,7 +415,7 @@ jobs:
 
     # Test
     - name: "Test: test-erlang"
-      timeout-minutes: 15
+      timeout-minutes: 30
       working-directory: build
       run: |
         ulimit -c unlimited