[Tarantool-patches] [PATCH luajit v3 0/3] build: fixed making tests with ASAN on OSX

[Tarantool-patches] [PATCH luajit v3 0/3] build: fixed making tests with ASAN on OSX

[Evgeniy Temirgaleev via Tarantool-patches]

This patch fixes building of the LuaJIT tests with sanitizer checks
on macOS. The fix is in the first commit. The second commit enables
sanitizer checks on macOS in CI. The third contains 'other changes'
which is helpful to debug new CI behavior.

Changes in v3:
  - compiler selection implemented in 'setup-sanitizers-macos' action.

Some MacOS/Arch/Compiler/ASAN options cases lead to build/test fail.
Such cases were excluded from the workflow matrix. See for details:
  - https://github.com/tarantool/tarantool/issues/13018
  - https://github.com/tarantool/tarantool/issues/13019

Branch: https://github.com/tarantool/luajit/tree/tmr_g/lj-noticket-fix-test-build-mac

Temir Galeev (3):
  test: fixed making tests with ASAN on OSX
  ci: enabled sanitizer tests for macOS
  ci: added a common way to disable perf workflow

 .../README.md                                 |   0
 .../action.yml                                |   8 +-
 .../actions/setup-sanitizers-macos/README.md  |  15 +++
 .../actions/setup-sanitizers-macos/action.yml |  73 +++++++++++++
 .github/workflows/performance.yml             |   1 +
 .github/workflows/sanitizers-testing.yml      | 101 ++++++++++++++++--
 test/LuaJIT-tests/CMakeLists.txt              |   3 +-
 test/tarantool-tests/CMakeLists.txt           |   3 +-
 8 files changed, 191 insertions(+), 13 deletions(-)
 rename .github/actions/{setup-sanitizers => setup-sanitizers-linux}/README.md (100%)
 rename .github/actions/{setup-sanitizers => setup-sanitizers-linux}/action.yml (76%)
 create mode 100644 .github/actions/setup-sanitizers-macos/README.md
 create mode 100644 .github/actions/setup-sanitizers-macos/action.yml

-- 
2.49.0

[Tarantool-patches] [PATCH luajit v3 1/3] test: fixed making tests with ASAN on OSX

[Evgeniy Temirgaleev via Tarantool-patches]

From: Temir Galeev <temir.galeev@bk.ru>

LuaJIT tests use fixup for Linux to run binaries made with
the ASAN option. Our LibRealPath module is used for it.
This module doesn't support OSX and breaks the make process.

So, we disable the fixup for OSX to allow LuaJIT making
and running the tests with ASAN enabled.
---
 test/LuaJIT-tests/CMakeLists.txt    | 3 ++-
 test/tarantool-tests/CMakeLists.txt | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/test/LuaJIT-tests/CMakeLists.txt b/test/LuaJIT-tests/CMakeLists.txt
index c1dbde35..42967cbc 100644
--- a/test/LuaJIT-tests/CMakeLists.txt
+++ b/test/LuaJIT-tests/CMakeLists.txt
@@ -17,7 +17,8 @@ else()
   list(APPEND LUAJIT_TESTS_ENV LD_LIBRARY_PATH="${LD_LIBRARY_PATH}")
 endif()
 
-if(LUAJIT_USE_ASAN)
+if(LUAJIT_USE_ASAN
+   AND NOT CMAKE_SYSTEM_NAME STREQUAL "Darwin")
   # When running LuaJIT-tests under ASAN, the internal ASAN check
   # failed:
   # AddressSanitizer: CHECK failed: asan_interceptors.cpp:356
diff --git a/test/tarantool-tests/CMakeLists.txt b/test/tarantool-tests/CMakeLists.txt
index 682a883a..e9930e22 100644
--- a/test/tarantool-tests/CMakeLists.txt
+++ b/test/tarantool-tests/CMakeLists.txt
@@ -183,7 +183,8 @@ endforeach()
 # required that the ASan library go first in the `LD_PRELOAD`
 # list. Set it manually. The test will append it to the executed
 # process.
-if(LUAJIT_USE_ASAN AND CMAKE_C_COMPILER_ID STREQUAL "GNU")
+if(LUAJIT_USE_ASAN AND CMAKE_C_COMPILER_ID STREQUAL "GNU"
+   AND NOT CMAKE_SYSTEM_NAME STREQUAL "Darwin")
   LibRealPath(LIB_ASAN libasan.so)
   AppendTestEnvVar(
     "test/${TEST_SUITE_NAME}/lj-522-fix-dlerror-return-null.test.lua"
-- 
2.49.0

Re: [Tarantool-patches] [PATCH luajit v3 1/3] test: fixed making tests with ASAN on OSX

[Sergey Kaplun via Tarantool-patches]

Hi, Evgeniy!
Thanks for the patch and fixes.
LGTM, with the minor nit below.

On 06.08.26, Evgeniy Temirgaleev wrote:
> From: Temir Galeev <temir.galeev@bk.ru>

<snipped>

> ---
>  test/LuaJIT-tests/CMakeLists.txt    | 3 ++-
>  test/tarantool-tests/CMakeLists.txt | 3 ++-
>  2 files changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/test/LuaJIT-tests/CMakeLists.txt b/test/LuaJIT-tests/CMakeLists.txt
> index c1dbde35..42967cbc 100644
> --- a/test/LuaJIT-tests/CMakeLists.txt
> +++ b/test/LuaJIT-tests/CMakeLists.txt
> @@ -17,7 +17,8 @@ else()
>    list(APPEND LUAJIT_TESTS_ENV LD_LIBRARY_PATH="${LD_LIBRARY_PATH}")
>  endif()
>  
> -if(LUAJIT_USE_ASAN)
> +if(LUAJIT_USE_ASAN
> +   AND NOT CMAKE_SYSTEM_NAME STREQUAL "Darwin")

Minor: Looks like these lines may be joint.

>    # When running LuaJIT-tests under ASAN, the internal ASAN check
>    # failed:
>    # AddressSanitizer: CHECK failed: asan_interceptors.cpp:356
> diff --git a/test/tarantool-tests/CMakeLists.txt b/test/tarantool-tests/CMakeLists.txt
> index 682a883a..e9930e22 100644
> --- a/test/tarantool-tests/CMakeLists.txt
> +++ b/test/tarantool-tests/CMakeLists.txt
> @@ -183,7 +183,8 @@ endforeach()
>  # required that the ASan library go first in the `LD_PRELOAD`
>  # list. Set it manually. The test will append it to the executed
>  # process.
> -if(LUAJIT_USE_ASAN AND CMAKE_C_COMPILER_ID STREQUAL "GNU")
> +if(LUAJIT_USE_ASAN AND CMAKE_C_COMPILER_ID STREQUAL "GNU"
> +   AND NOT CMAKE_SYSTEM_NAME STREQUAL "Darwin")
>    LibRealPath(LIB_ASAN libasan.so)
>    AppendTestEnvVar(
>      "test/${TEST_SUITE_NAME}/lj-522-fix-dlerror-return-null.test.lua"
> -- 
> 2.49.0
> 

-- 
Best regards,
Sergey Kaplun

Re: [Tarantool-patches] [PATCH luajit v3 1/3] test: fixed making tests with ASAN on OSX

[Evgeniy Temirgaleev via Tarantool-patches]

[-- Attachment #1: Type: text/plain, Size: 2175 bytes --]

Hi, Sergey! Thanks for review!

> 
> From: Sergey Kaplun <skaplun@tarantool.org>
> To: Evgeniy Temirgaleev <e.temirgaleev@tarantool.org>
> Cc: Sergey Bronnikov <sergeyb@tarantool.org>, tarantool-patches@dev.tarantool.org
> 
> Date: Tuesday, August 18, 2026 1:25 PM +03:00
> Hi, Evgeniy!
> Thanks for the patch and fixes.
> LGTM, with the minor nit below.
> 
> On 06.08.26, Evgeniy Temirgaleev wrote:
> > From: Temir Galeev <temir.galeev@bk.ru>
> 
> <snipped>
> 
> > ---
> > test/LuaJIT-tests/CMakeLists.txt | 3 ++-
> > test/tarantool-tests/CMakeLists.txt | 3 ++-
> > 2 files changed, 4 insertions(+), 2 deletions(-)
> >
> > diff --git a/test/LuaJIT-tests/CMakeLists.txt
> b/test/LuaJIT-tests/CMakeLists.txt
> > index c1dbde35..42967cbc 100644
> > --- a/test/LuaJIT-tests/CMakeLists.txt
> > +++ b/test/LuaJIT-tests/CMakeLists.txt
> > @@ -17,7 +17,8 @@ else()
> > list(APPEND LUAJIT_TESTS_ENV LD_LIBRARY_PATH="${LD_LIBRARY_PATH}")
> > endif()
> >
> > -if(LUAJIT_USE_ASAN)
> > +if(LUAJIT_USE_ASAN
> > + AND NOT CMAKE_SYSTEM_NAME STREQUAL "Darwin")
> 
> Minor: Looks like these lines may be joint.
> 

It was so. But when I added this guard to the second place, I decided to make the change same in both places.

> 
> 
> > # When running LuaJIT-tests under ASAN, the internal ASAN check
> > # failed:
> > # AddressSanitizer: CHECK failed: asan_interceptors.cpp:356
> > diff --git a/test/tarantool-tests/CMakeLists.txt
> b/test/tarantool-tests/CMakeLists.txt
> > index 682a883a..e9930e22 100644
> > --- a/test/tarantool-tests/CMakeLists.txt
> > +++ b/test/tarantool-tests/CMakeLists.txt
> > @@ -183,7 +183,8 @@ endforeach()
> > # required that the ASan library go first in the `LD_PRELOAD`
> > # list. Set it manually. The test will append it to the executed
> > # process.
> > -if(LUAJIT_USE_ASAN AND CMAKE_C_COMPILER_ID STREQUAL "GNU")
> > +if(LUAJIT_USE_ASAN AND CMAKE_C_COMPILER_ID STREQUAL "GNU"
> > + AND NOT CMAKE_SYSTEM_NAME STREQUAL "Darwin")
> > LibRealPath(LIB_ASAN libasan.so)
> > AppendTestEnvVar(
> > "test/${TEST_SUITE_NAME}/lj-522-fix-dlerror-return-null.test.lua"
> > --
> > 2.49.0
> >
> 
> --
> Best regards,
> Sergey Kaplun
> 

Best regards,
Evgeniy Temirgaleev

[-- Attachment #2: Type: text/html, Size: 3391 bytes --]

[Tarantool-patches] [PATCH luajit v3 2/3] ci: enabled sanitizer tests for macOS

[Evgeniy Temirgaleev via Tarantool-patches]

From: Temir Galeev <temir.galeev@bk.ru>

The arm64 and x86_64 architectures with clang/gcc compiler were added
to the matrix.
---
 .../README.md                                 |   0
 .../action.yml                                |   8 +-
 .../actions/setup-sanitizers-macos/README.md  |  15 +++
 .../actions/setup-sanitizers-macos/action.yml |  73 +++++++++++++
 .github/workflows/sanitizers-testing.yml      | 101 ++++++++++++++++--
 5 files changed, 186 insertions(+), 11 deletions(-)
 rename .github/actions/{setup-sanitizers => setup-sanitizers-linux}/README.md (100%)
 rename .github/actions/{setup-sanitizers => setup-sanitizers-linux}/action.yml (76%)
 create mode 100644 .github/actions/setup-sanitizers-macos/README.md
 create mode 100644 .github/actions/setup-sanitizers-macos/action.yml

diff --git a/.github/actions/setup-sanitizers/README.md b/.github/actions/setup-sanitizers-linux/README.md
similarity index 100%
rename from .github/actions/setup-sanitizers/README.md
rename to .github/actions/setup-sanitizers-linux/README.md
diff --git a/.github/actions/setup-sanitizers/action.yml b/.github/actions/setup-sanitizers-linux/action.yml
similarity index 76%
rename from .github/actions/setup-sanitizers/action.yml
rename to .github/actions/setup-sanitizers-linux/action.yml
index 8642d553..18f5a75d 100644
--- a/.github/actions/setup-sanitizers/action.yml
+++ b/.github/actions/setup-sanitizers-linux/action.yml
@@ -20,13 +20,17 @@ runs:
     - name: Install build and test dependencies
       run: |
         apt -y update
+        echo Available compilers:
+        export CC_FAMILY=`echo ${CC_NAME} | sed 's/-.*$//'`
+        apt list | grep -Pe "^${CC_FAMILY}-[0-9]+/"
+        # Try to install
         apt -y install ${CC_NAME} libstdc++-10-dev cmake ninja-build make perl
       shell: bash
       env:
         CC_NAME: ${{ inputs.cc_name }}
-    - name: Set specific C compiler as a default toolchain
+    - name: Set specific C compiler as a default toolchain for cmake
       run: |
-        echo CC=${CC_NAME} | tee -a $GITHUB_ENV
+        echo CMAKE_C_COMPILER=${CC_NAME} | tee -a $GITHUB_ENV
       shell: bash
       env:
         CC_NAME: ${{ inputs.cc_name }}
diff --git a/.github/actions/setup-sanitizers-macos/README.md b/.github/actions/setup-sanitizers-macos/README.md
new file mode 100644
index 00000000..9ac5eb38
--- /dev/null
+++ b/.github/actions/setup-sanitizers-macos/README.md
@@ -0,0 +1,15 @@
+# Setup environment for sanitizers on macOS
+
+Action setups the environment on macOS runners (install requirements, setup the
+workflow environment, etc) for testing with sanitizers enabled.
+
+Requires input:
+- cc_name as versioned C compiler: gcc-ver or clang-ver.
+
+## How to use Github Action from Github workflow
+
+Add the following code to the running steps before LuaJIT configuration:
+```
+- uses: ./.github/actions/setup-sanitizers-macos
+  if: ${{ matrix.OS == 'macOS' }}
+```
diff --git a/.github/actions/setup-sanitizers-macos/action.yml b/.github/actions/setup-sanitizers-macos/action.yml
new file mode 100644
index 00000000..9836ea03
--- /dev/null
+++ b/.github/actions/setup-sanitizers-macos/action.yml
@@ -0,0 +1,73 @@
+name: Setup CI environment for testing with sanitizers on macOS
+description: Common part to tweak macOS CI runner environment for sanitizers
+inputs:
+  cc_name:
+    description: C compiler name (for example, gcc-12)
+    required: false
+    default: clang-21
+runs:
+  using: composite
+  steps:
+    - name: Get compiler version from cc_name
+      shell: bash
+      env:
+        CC_NAME: ${{ inputs.cc_name }}
+      run: |
+        echo CC_VERSION=`echo ${CC_NAME} | sed 's/.*-//'` | tee -a $GITHUB_ENV
+    - name: Get brew formula from cc_name
+      shell: bash
+      env:
+        CC_FORMULA_NAME: |-
+          ${{ case(
+            startsWith(inputs.cc_name, 'gcc'), 'gcc',
+            startsWith(inputs.cc_name, 'clang'), 'llvm',
+            'MISCONFIG'
+          ) }}
+      run: |
+        echo CC_FORMULA=${CC_FORMULA_NAME}@${CC_VERSION} | tee -a $GITHUB_ENV
+        echo Available formulas: `brew search ${CC_FORMULA_NAME}`
+    - name: Setup CI environment
+      uses: ./.github/actions/setup
+    - name: Set CMAKE_BUILD_PARALLEL_LEVEL
+      shell: bash
+      run: |
+        # Set CMAKE_BUILD_PARALLEL_LEVEL environment variable to
+        # limit the number of parallel jobs for build/test step.
+        NPROC=$(sysctl -n hw.logicalcpu 2>/dev/null)
+        echo CMAKE_BUILD_PARALLEL_LEVEL=$(($NPROC + 1)) | tee -a $GITHUB_ENV
+    - name: Set MACOSX_DEPLOYMENT_TARGERT
+      shell: bash
+      run: |
+        # Set required MACOSX_DEPLOYMENT_TARGERT environment
+        # variable for Makefile.original build.
+        # See https://github.com/LuaJIT/LuaJIT/issues/484,
+        # https://github.com/LuaJIT/LuaJIT/issues/653.
+        echo MACOSX_DEPLOYMENT_TARGET=$(sw_vers -productVersion) | tee -a $GITHUB_ENV
+    - name: Install build and test dependencies
+      shell: bash
+      run: |
+        # Install brew using the command from Homebrew repository
+        # instructions: https://github.com/Homebrew/install.
+        # XXX: 'echo' command below is required since brew
+        # installation script obliges the one to enter a newline
+        # for confirming the installation via Ruby script.
+        brew update ||
+          echo | /usr/bin/ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)"
+        # Try to install the packages either upgrade it to avoid
+        # of fails if the package already exists with the previous
+        # version.
+        brew install --force ${CC_FORMULA} cmake make ninja perl ||
+          brew upgrade ${CC_FORMULA} cmake make ninja perl
+    - name: Set specific C compiler as a default toolchain for cmake
+      shell: bash
+      env:
+        CC_NAME: ${{ inputs.cc_name }}
+      run: |
+        echo CMAKE_C_COMPILER=${CC_NAME} | tee -a $GITHUB_ENV
+        echo CMAKE_PREFIX_PATH="`brew --prefix ${CC_FORMULA}`" | tee -a $GITHUB_ENV
+    - name: Log installed compilers
+      shell: bash
+      run: |
+        echo default clang: `clang --version`
+        echo default gcc: `gcc --version`
+        echo cmake compiler: `${CMAKE_PREFIX_PATH}/bin/${CMAKE_C_COMPILER} --version`
diff --git a/.github/workflows/sanitizers-testing.yml b/.github/workflows/sanitizers-testing.yml
index 4bf7d023..fe550b81 100644
--- a/.github/workflows/sanitizers-testing.yml
+++ b/.github/workflows/sanitizers-testing.yml
@@ -31,17 +31,41 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        # XXX: Let's start with only Linux/x86_64
+        ARCH: [ARM64, x86_64]
         BUILDTYPE: [Debug, Release]
-        CC: [gcc-10, clang-11]
+        OS: [Linux, macOS]
+        # There are different top-level versions available for Linux and macOS runners.
+        CC: [gcc-10, clang-11, gcc-15, clang-21]
         include:
           - BUILDTYPE: Debug
             CMAKEFLAGS: -DCMAKE_BUILD_TYPE=Debug -DLUA_USE_ASSERT=ON -DLUA_USE_APICHECK=ON
           - BUILDTYPE: Release
             CMAKEFLAGS: -DCMAKE_BUILD_TYPE=RelWithDebInfo
-    runs-on: [self-hosted, regular, Linux, x86_64]
+        exclude:
+          # On current runners with Linux/ARM64 environment and
+          # with LUAJIT_USE_SYSMALLOC=ON the system allocator returns addresses
+          # with 48-bit set. Thus checkptrGC() fails with new Lua state pointer
+          # and luajit fails to start with 'cannot create state: not enough memory'
+          # error. So, we exclude these cases.
+          - ARCH: ARM64
+            OS: Linux
+          # Exclude nonsuitable OS/compiler pairs.
+          - OS: macOS
+            CC: gcc-10
+          - OS: macOS
+            CC: clang-11
+          - OS: Linux
+            CC: gcc-15
+          - OS: Linux
+            CC: clang-21
+          # Exclude macOS/ARM64/gcc case due to some tests are failed.
+          # Details: https://github.com/tarantool/tarantool/issues/13018
+          - ARCH: ARM64
+            OS: macOS
+            CC: gcc-15
+    runs-on: [self-hosted, regular, '${{ matrix.OS }}', '${{ matrix.ARCH }}']
     name: >
-      LuaJIT with ASan and UBSan (Linux/x86_64)
+      LuaJIT with ASan and UBSan (${{ matrix.OS }}/${{ matrix.ARCH }})
       ${{ matrix.BUILDTYPE }}
       CC:${{ matrix.CC }}
       GC64:ON SYSMALLOC:ON
@@ -51,7 +75,13 @@ jobs:
           fetch-depth: 0
           submodules: recursive
       - name: setup Linux for sanitizers
-        uses: ./.github/actions/setup-sanitizers
+        if: ${{ matrix.OS == 'Linux' }}
+        uses: ./.github/actions/setup-sanitizers-linux
+        with:
+          cc_name: ${{ matrix.CC }}
+      - name: setup macOS for sanitizers
+        if: ${{ matrix.OS == 'macOS' }}
+        uses: ./.github/actions/setup-sanitizers-macos
         with:
           cc_name: ${{ matrix.CC }}
       - name: configure
@@ -70,18 +100,46 @@ jobs:
           cmake -S . -B ${{ env.BUILDDIR }}
           -G Ninja
           ${{ matrix.CMAKEFLAGS }}
+          -DCMAKE_C_COMPILER=${CMAKE_C_COMPILER}
+          -DCMAKE_PREFIX_PATH=${CMAKE_PREFIX_PATH}
           -DLUAJIT_ENABLE_GC64=ON
           -DLUAJIT_USE_ASAN=ON
           -DLUAJIT_USE_SYSMALLOC=ON
           -DLUAJIT_USE_UBSAN=ON
+      - name: Check for possible compiler misconfig
+        working-directory: ${{ env.BUILDDIR }}
+        env:
+          CC_NAME: ${{ matrix.CC }}
+        run: grep CMakeCache.txt -e CMAKE_C_COMPILER:STRING | grep ${CC_NAME}
       - name: build
         run: cmake --build . --parallel
         working-directory: ${{ env.BUILDDIR }}
-      - name: test
+
+      # Enable as much checks as possible. See more info here:
+      # https://github.com/google/sanitizers/wiki/AddressSanitizerFlags,
+      # https://github.com/google/sanitizers/wiki/SanitizerCommonFlags.
+      - name: setup sanitizer options for Linux
+        if: ${{ matrix.OS == 'Linux' }}
+        env:
+          ASAN_OPTIONS: "                    \
+            detect_invalid_pointer_pairs=1:  \
+            detect_leaks=1:                  \
+            detect_stack_use_after_return=1: \
+            dump_instruction_bytes=1:        \
+            heap_profile=0:                  \
+            print_suppressions=0:            \
+            symbolize=1:                     \
+            unmap_shadow_on_exit=1:          \
+          "
+          UBSAN_OPTIONS: "
+            print_stacktrace=1 \
+          "
+        run: |
+          echo ASAN_OPTIONS=${ASAN_OPTIONS} | tee -a $GITHUB_ENV
+          echo UBSAN_OPTIONS=${UBSAN_OPTIONS} | tee -a $GITHUB_ENV
+      - name: setup sanitizer options for macOS (common)
+        if: ${{ matrix.OS == 'macOS' && matrix.ARCH != 'ARM64' && matrix.CC != 'clang-21' }}
         env:
-          # Enable as much checks as possible. See more info here:
-          # https://github.com/google/sanitizers/wiki/AddressSanitizerFlags,
-          # https://github.com/google/sanitizers/wiki/SanitizerCommonFlags.
           ASAN_OPTIONS: "                    \
             detect_invalid_pointer_pairs=1:  \
             detect_leaks=1:                  \
@@ -95,5 +153,30 @@ jobs:
           UBSAN_OPTIONS: "
             print_stacktrace=1 \
           "
+        run: |
+          echo ASAN_OPTIONS=${ASAN_OPTIONS} | tee -a $GITHUB_ENV
+          echo UBSAN_OPTIONS=${UBSAN_OPTIONS} | tee -a $GITHUB_ENV
+      - name: setup sanitizer options for macOS (ARM64/clang-21)
+        if: ${{ matrix.OS == 'macOS' && matrix.ARCH == 'ARM64' && matrix.CC == 'clang-21' }}
+        # detect_leaks is disabled due to some tests build is failed.
+        # Details: https://github.com/tarantool/tarantool/issues/13019
+        env:
+          ASAN_OPTIONS: "                    \
+            detect_invalid_pointer_pairs=1:  \
+            detect_leaks=0:                  \
+            detect_stack_use_after_return=1: \
+            dump_instruction_bytes=1:        \
+            heap_profile=0:                  \
+            print_suppressions=0:            \
+            symbolize=1:                     \
+            unmap_shadow_on_exit=1:          \
+          "
+          UBSAN_OPTIONS: "
+            print_stacktrace=1 \
+          "
+        run: |
+          echo ASAN_OPTIONS=${ASAN_OPTIONS} | tee -a $GITHUB_ENV
+          echo UBSAN_OPTIONS=${UBSAN_OPTIONS} | tee -a $GITHUB_ENV
+      - name: test
         run: cmake --build . --parallel --target LuaJIT-test
         working-directory: ${{ env.BUILDDIR }}
-- 
2.49.0

Re: [Tarantool-patches] [PATCH luajit v3 2/3] ci: enabled sanitizer tests for macOS

[Sergey Kaplun via Tarantool-patches]

Hi, Evgeniy!
Thanks for the patch and fixes!
Generally LGTM, but please clarify my questions below.

On 06.08.26, Evgeniy Temirgaleev wrote:
> From: Temir Galeev <temir.galeev@bk.ru>
> 
> The arm64 and x86_64 architectures with clang/gcc compiler were added
> to the matrix.
> ---
>  .../README.md                                 |   0
>  .../action.yml                                |   8 +-
>  .../actions/setup-sanitizers-macos/README.md  |  15 +++
>  .../actions/setup-sanitizers-macos/action.yml |  73 +++++++++++++
>  .github/workflows/sanitizers-testing.yml      | 101 ++++++++++++++++--
>  5 files changed, 186 insertions(+), 11 deletions(-)
>  rename .github/actions/{setup-sanitizers => setup-sanitizers-linux}/README.md (100%)
>  rename .github/actions/{setup-sanitizers => setup-sanitizers-linux}/action.yml (76%)
>  create mode 100644 .github/actions/setup-sanitizers-macos/README.md
>  create mode 100644 .github/actions/setup-sanitizers-macos/action.yml
> 
> diff --git a/.github/actions/setup-sanitizers/README.md b/.github/actions/setup-sanitizers-linux/README.md
> similarity index 100%
> rename from .github/actions/setup-sanitizers/README.md
> rename to .github/actions/setup-sanitizers-linux/README.md
> diff --git a/.github/actions/setup-sanitizers/action.yml b/.github/actions/setup-sanitizers-linux/action.yml
> similarity index 76%
> rename from .github/actions/setup-sanitizers/action.yml
> rename to .github/actions/setup-sanitizers-linux/action.yml
> index 8642d553..18f5a75d 100644
> --- a/.github/actions/setup-sanitizers/action.yml
> +++ b/.github/actions/setup-sanitizers-linux/action.yml
> @@ -20,13 +20,17 @@ runs:
>      - name: Install build and test dependencies
>        run: |
>          apt -y update
> +        echo Available compilers:
> +        export CC_FAMILY=`echo ${CC_NAME} | sed 's/-.*$//'`
> +        apt list | grep -Pe "^${CC_FAMILY}-[0-9]+/"
> +        # Try to install
>          apt -y install ${CC_NAME} libstdc++-10-dev cmake ninja-build make perl
>        shell: bash
>        env:
>          CC_NAME: ${{ inputs.cc_name }}
> -    - name: Set specific C compiler as a default toolchain
> +    - name: Set specific C compiler as a default toolchain for cmake
>        run: |
> -        echo CC=${CC_NAME} | tee -a $GITHUB_ENV
> +        echo CMAKE_C_COMPILER=${CC_NAME} | tee -a $GITHUB_ENV
>        shell: bash
>        env:
>          CC_NAME: ${{ inputs.cc_name }}

These changes look like debugging of the workflow. Are they necessary?

> diff --git a/.github/actions/setup-sanitizers-macos/README.md b/.github/actions/setup-sanitizers-macos/README.md
> new file mode 100644
> index 00000000..9ac5eb38
> --- /dev/null
> +++ b/.github/actions/setup-sanitizers-macos/README.md

<snipped>

> diff --git a/.github/actions/setup-sanitizers-macos/action.yml b/.github/actions/setup-sanitizers-macos/action.yml
> new file mode 100644
> index 00000000..9836ea03
> --- /dev/null
> +++ b/.github/actions/setup-sanitizers-macos/action.yml
> @@ -0,0 +1,73 @@

<snipped>

> +    - name: Set CMAKE_BUILD_PARALLEL_LEVEL
> +      shell: bash
> +      run: |
> +        # Set CMAKE_BUILD_PARALLEL_LEVEL environment variable to
> +        # limit the number of parallel jobs for build/test step.
> +        NPROC=$(sysctl -n hw.logicalcpu 2>/dev/null)
> +        echo CMAKE_BUILD_PARALLEL_LEVEL=$(($NPROC + 1)) | tee -a $GITHUB_ENV
> +    - name: Set MACOSX_DEPLOYMENT_TARGERT
> +      shell: bash
> +      run: |
> +        # Set required MACOSX_DEPLOYMENT_TARGERT environment
> +        # variable for Makefile.original build.
> +        # See https://github.com/LuaJIT/LuaJIT/issues/484,
> +        # https://github.com/LuaJIT/LuaJIT/issues/653.
> +        echo MACOSX_DEPLOYMENT_TARGET=$(sw_vers -productVersion) | tee -a $GITHUB_ENV
> +    - name: Install build and test dependencies
> +      shell: bash
> +      run: |
> +        # Install brew using the command from Homebrew repository
> +        # instructions: https://github.com/Homebrew/install.
> +        # XXX: 'echo' command below is required since brew
> +        # installation script obliges the one to enter a newline
> +        # for confirming the installation via Ruby script.
> +        brew update ||
> +          echo | /usr/bin/ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)"
> +        # Try to install the packages either upgrade it to avoid
> +        # of fails if the package already exists with the previous
> +        # version.
> +        brew install --force ${CC_FORMULA} cmake make ninja perl ||
> +          brew upgrade ${CC_FORMULA} cmake make ninja perl

Can we reuse setup-macos action instead these 3 steps?

<snipped>

> diff --git a/.github/workflows/sanitizers-testing.yml b/.github/workflows/sanitizers-testing.yml
> index 4bf7d023..fe550b81 100644
> --- a/.github/workflows/sanitizers-testing.yml
> +++ b/.github/workflows/sanitizers-testing.yml


<snipped>

> +        exclude:

<snipped>

> +          - OS: Linux
> +            CC: gcc-15
> +          - OS: Linux
> +            CC: clang-21

May we bump gcc and clang instead for Linux as well (I suppose its
about time :)) ?

I suppose it helps to avoid too many excludes. Also, there is no need
for testing these specific compilers instead of modern ones in
sanitizers build.

If there are any issues when upgrading compilers for sanitizer builds
(infrastructure or tests failures) -- feel free to ignore.

<snipped>


> -- 
> 2.49.0
> 

-- 
Best regards,
Sergey Kaplun

Re: [Tarantool-patches] [PATCH luajit v3 2/3] ci: enabled sanitizer tests for macOS

[Evgeniy Temirgaleev via Tarantool-patches]

[-- Attachment #1: Type: text/plain, Size: 7162 bytes --]

Hi, Sergey! Thanks for review!

> 
> From: Sergey Kaplun <skaplun@tarantool.org>
> To: Evgeniy Temirgaleev <e.temirgaleev@tarantool.org>
> Cc: Sergey Bronnikov <sergeyb@tarantool.org>, tarantool-patches@dev.tarantool.org
> 
> Date: Tuesday, August 18, 2026 1:47 PM +03:00
> Hi, Evgeniy!
> Thanks for the patch and fixes!
> Generally LGTM, but please clarify my questions below.
> 
> On 06.08.26, Evgeniy Temirgaleev wrote:
> > From: Temir Galeev <temir.galeev@bk.ru>
> >
> > The arm64 and x86_64 architectures with clang/gcc compiler were added
> > to the matrix.
> > ---
> > .../README.md | 0
> > .../action.yml | 8 +-
> > .../actions/setup-sanitizers-macos/README.md | 15 +++
> > .../actions/setup-sanitizers-macos/action.yml | 73 +++++++++++++
> > .github/workflows/sanitizers-testing.yml | 101 ++++++++++++++++--
> > 5 files changed, 186 insertions(+), 11 deletions(-)
> > rename .github/actions/{setup-sanitizers =>
> setup-sanitizers-linux}/README.md (100%)
> > rename .github/actions/{setup-sanitizers =>
> setup-sanitizers-linux}/action.yml (76%)
> > create mode 100644 .github/actions/setup-sanitizers-macos/README.md
> > create mode 100644 .github/actions/setup-sanitizers-macos/action.yml
> >
> > diff --git a/.github/actions/setup-sanitizers/README.md
> b/.github/actions/setup-sanitizers-linux/README.md
> > similarity index 100%
> > rename from .github/actions/setup-sanitizers/README.md
> > rename to .github/actions/setup-sanitizers-linux/README.md
> > diff --git a/.github/actions/setup-sanitizers/action.yml
> b/.github/actions/setup-sanitizers-linux/action.yml
> > similarity index 76%
> > rename from .github/actions/setup-sanitizers/action.yml
> > rename to .github/actions/setup-sanitizers-linux/action.yml
> > index 8642d553..18f5a75d 100644
> > --- a/.github/actions/setup-sanitizers/action.yml
> > +++ b/.github/actions/setup-sanitizers-linux/action.yml
> > @@ -20,13 +20,17 @@ runs:
> > - name: Install build and test dependencies
> > run: |
> > apt -y update
> > + echo Available compilers:
> > + export CC_FAMILY=`echo ${CC_NAME} | sed 's/-.*$//'`
> > + apt list | grep -Pe "^${CC_FAMILY}-[0-9]+/"
> > + # Try to install
> > apt -y install ${CC_NAME} libstdc++-10-dev cmake ninja-build make perl
> > shell: bash
> > env:
> > CC_NAME: ${{ inputs.cc_name }}
> > - - name: Set specific C compiler as a default toolchain
> > + - name: Set specific C compiler as a default toolchain for cmake
> > run: |
> > - echo CC=${CC_NAME} | tee -a $GITHUB_ENV
> > + echo CMAKE_C_COMPILER=${CC_NAME} | tee -a $GITHUB_ENV
> > shell: bash
> > env:
> > CC_NAME: ${{ inputs.cc_name }}
> 
> These changes look like debugging of the workflow. Are they necessary?
> 

I think it’s a useful information: Which compiler we can select just now with the current runner? For example, it used to answer your question below.
I added it advisedly, as it done for macOS action.

> 
> 
> > diff --git a/.github/actions/setup-sanitizers-macos/README.md
> b/.github/actions/setup-sanitizers-macos/README.md
> > new file mode 100644
> > index 00000000..9ac5eb38
> > --- /dev/null
> > +++ b/.github/actions/setup-sanitizers-macos/README.md
> 
> <snipped>
> 
> > diff --git a/.github/actions/setup-sanitizers-macos/action.yml
> b/.github/actions/setup-sanitizers-macos/action.yml
> > new file mode 100644
> > index 00000000..9836ea03
> > --- /dev/null
> > +++ b/.github/actions/setup-sanitizers-macos/action.yml
> > @@ -0,0 +1,73 @@
> 
> <snipped>
> 
> > + - name: Set CMAKE_BUILD_PARALLEL_LEVEL
> > + shell: bash
> > + run: |
> > + # Set CMAKE_BUILD_PARALLEL_LEVEL environment variable to
> > + # limit the number of parallel jobs for build/test step.
> > + NPROC=$(sysctl -n hw.logicalcpu 2>/dev/null)
> > + echo CMAKE_BUILD_PARALLEL_LEVEL=$(($NPROC + 1)) | tee -a $GITHUB_ENV
> > + - name: Set MACOSX_DEPLOYMENT_TARGERT
> > + shell: bash
> > + run: |
> > + # Set required MACOSX_DEPLOYMENT_TARGERT environment
> > + # variable for Makefile.original build.
> > + # See https://github.com/LuaJIT/LuaJIT/issues/484,
> > + # https://github.com/LuaJIT/LuaJIT/issues/653.
> > + echo MACOSX_DEPLOYMENT_TARGET=$(sw_vers -productVersion) | tee -a
> $GITHUB_ENV
> > + - name: Install build and test dependencies
> > + shell: bash
> > + run: |
> > + # Install brew using the command from Homebrew repository
> > + # instructions: https://github.com/Homebrew/install.
> > + # XXX: 'echo' command below is required since brew
> > + # installation script obliges the one to enter a newline
> > + # for confirming the installation via Ruby script.
> > + brew update ||
> > + echo | /usr/bin/ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install
> )"
> > + # Try to install the packages either upgrade it to avoid
> > + # of fails if the package already exists with the previous
> > + # version.
> > + brew install --force ${CC_FORMULA} cmake make ninja perl ||
> > + brew upgrade ${CC_FORMULA} cmake make ninja perl
> 
> Can we reuse setup-macos action instead these 3 steps?
> 

Yes, we can do it on top of the «ci: unused gcc removed from macOS setup» patch.
So, we need to merge it first. And then I shall update this patch.
Thanks for the notice.

> 
> 
> <snipped>
> 
> > diff --git a/.github/workflows/sanitizers-testing.yml
> b/.github/workflows/sanitizers-testing.yml
> > index 4bf7d023..fe550b81 100644
> > --- a/.github/workflows/sanitizers-testing.yml
> > +++ b/.github/workflows/sanitizers-testing.yml
> 
> 
> <snipped>
> 
> > + exclude:
> 
> <snipped>
> 
> > + - OS: Linux
> > + CC: gcc-15
> > + - OS: Linux
> > + CC: clang-21
> 
> May we bump gcc and clang instead for Linux as well (I suppose its
> about time :)) ?
> 
> I suppose it helps to avoid too many excludes. Also, there is no need
> for testing these specific compilers instead of modern ones in
> sanitizers build.
> 
> If there are any issues when upgrading compilers for sanitizer builds
> (infrastructure or tests failures) -- feel free to ignore.
> 
> <snipped>
> 

I tried to use the top versions available for macOS and for Linux. We can’t use the same versions with our current setup.

With the macOS runner we have:
Available formulas: ... gcc gcc@10 gcc@11 gcc@12 gcc@13 gcc@14 gcc@15 gcc@9 …
Available formulas: ... llvm llvm@14 llvm@15 llvm@16 llvm@17 llvm@18 llvm@19 llvm@20 llvm@21 ...

With the Linux runner we have:

gcc-10/focal-updates,now 10.5.0-1ubuntu1~20.04 amd64 [installed]
gcc-7/focal 7.5.0-6ubuntu2 amd64
gcc-8/focal 8.4.0-3ubuntu2 amd64
gcc-9/focal-updates,now 9.4.0-1ubuntu1~20.04.2 amd64 [installed,automatic]

clang-10/focal,now 1:10.0.0-4ubuntu1 amd64 [installed,automatic]
clang-11/focal-updates,now 1:11.0.0-2~ubuntu20.04.1 amd64 [installed]
clang-12/focal-updates 1:12.0.0-3ubuntu1~20.04.5 amd64
clang-18/focal-updates 1:18.1.8-11~20.04.2 amd64
clang-7/focal 1:7.0.1-12 amd64
clang-8/focal 1:8.0.1-9 amd64
clang-9/focal 1:9.0.1-12 amd64

The clang18 have broken dependencies with the current version of setup-sanitizers-linux. So I leave the current version in the context of ‘fix sanitizers for macOS patch’.

> 
> 
> 
> > --
> > 2.49.0
> >
> 
> --
> Best regards,
> Sergey Kaplun
> 

Best regards,
Evgeniy Temirgaleev

[-- Attachment #2: Type: text/html, Size: 10381 bytes --]

Re: [Tarantool-patches] [PATCH luajit v3 2/3] ci: enabled sanitizer tests for macOS

[Sergey Kaplun via Tarantool-patches]

Hi, Evgeniy!
Thanks for the answers!
LGTM, with minor suggestion below.

On 18.08.26, Evgeniy Temirgaleev wrote:
> Hi, Sergey! Thanks for review!
> 
> > 
> > From: Sergey Kaplun <skaplun@tarantool.org>

<snipped>

> > b/.github/actions/setup-sanitizers-linux/action.yml
> > > similarity index 76%
> > > rename from .github/actions/setup-sanitizers/action.yml
> > > rename to .github/actions/setup-sanitizers-linux/action.yml
> > > index 8642d553..18f5a75d 100644
> > > --- a/.github/actions/setup-sanitizers/action.yml
> > > +++ b/.github/actions/setup-sanitizers-linux/action.yml
> > > @@ -20,13 +20,17 @@ runs:
> > > - name: Install build and test dependencies
> > > run: |
> > > apt -y update
> > > + echo Available compilers:
> > > + export CC_FAMILY=`echo ${CC_NAME} | sed 's/-.*$//'`
> > > + apt list | grep -Pe "^${CC_FAMILY}-[0-9]+/"
> > > + # Try to install
> > > apt -y install ${CC_NAME} libstdc++-10-dev cmake ninja-build make perl
> > > shell: bash
> > > env:
> > > CC_NAME: ${{ inputs.cc_name }}
> > > - - name: Set specific C compiler as a default toolchain
> > > + - name: Set specific C compiler as a default toolchain for cmake
> > > run: |
> > > - echo CC=${CC_NAME} | tee -a $GITHUB_ENV
> > > + echo CMAKE_C_COMPILER=${CC_NAME} | tee -a $GITHUB_ENV
> > > shell: bash
> > > env:
> > > CC_NAME: ${{ inputs.cc_name }}
> > 
> > These changes look like debugging of the workflow. Are they necessary?
> > 
> 
> I think it’s a useful information: Which compiler we can select just now with the current runner? For example, it used to answer your question below.
> I added it advisedly, as it done for macOS action.

Got it. Please add the comment and the note about it in the commit
message to avoid confusion.

> 
> > 
> > 
> > > diff --git a/.github/actions/setup-sanitizers-macos/README.md
> > b/.github/actions/setup-sanitizers-macos/README.md
> > > new file mode 100644
> > > index 00000000..9ac5eb38
> > > --- /dev/null
> > > +++ b/.github/actions/setup-sanitizers-macos/README.md
> > 
> > <snipped>
> > 
> > > diff --git a/.github/actions/setup-sanitizers-macos/action.yml
> > b/.github/actions/setup-sanitizers-macos/action.yml
> > > new file mode 100644
> > > index 00000000..9836ea03
> > > --- /dev/null
> > > +++ b/.github/actions/setup-sanitizers-macos/action.yml
> > > @@ -0,0 +1,73 @@
> > 
> > <snipped>
> > 
> > > + - name: Set CMAKE_BUILD_PARALLEL_LEVEL
> > > + shell: bash
> > > + run: |
> > > + # Set CMAKE_BUILD_PARALLEL_LEVEL environment variable to
> > > + # limit the number of parallel jobs for build/test step.
> > > + NPROC=$(sysctl -n hw.logicalcpu 2>/dev/null)
> > > + echo CMAKE_BUILD_PARALLEL_LEVEL=$(($NPROC + 1)) | tee -a $GITHUB_ENV
> > > + - name: Set MACOSX_DEPLOYMENT_TARGERT
> > > + shell: bash
> > > + run: |
> > > + # Set required MACOSX_DEPLOYMENT_TARGERT environment
> > > + # variable for Makefile.original build.
> > > + # See https://github.com/LuaJIT/LuaJIT/issues/484,
> > > + # https://github.com/LuaJIT/LuaJIT/issues/653.
> > > + echo MACOSX_DEPLOYMENT_TARGET=$(sw_vers -productVersion) | tee -a
> > $GITHUB_ENV
> > > + - name: Install build and test dependencies
> > > + shell: bash
> > > + run: |
> > > + # Install brew using the command from Homebrew repository
> > > + # instructions: https://github.com/Homebrew/install.
> > > + # XXX: 'echo' command below is required since brew
> > > + # installation script obliges the one to enter a newline
> > > + # for confirming the installation via Ruby script.
> > > + brew update ||
> > > + echo | /usr/bin/ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install
> > )"
> > > + # Try to install the packages either upgrade it to avoid
> > > + # of fails if the package already exists with the previous
> > > + # version.
> > > + brew install --force ${CC_FORMULA} cmake make ninja perl ||
> > > + brew upgrade ${CC_FORMULA} cmake make ninja perl
> > 
> > Can we reuse setup-macos action instead these 3 steps?
> > 
> 
> Yes, we can do it on top of the «ci: unused gcc removed from macOS setup» patch.
> So, we need to merge it first.

Do we? Don't understand why it is required. The gcc instalation and
upgrade looks harmless, IINM.

>                                 And then I shall update this patch.
> Thanks for the notice.
> 
> > 
> > 
> > <snipped>
> > 
> > > diff --git a/.github/workflows/sanitizers-testing.yml
> > b/.github/workflows/sanitizers-testing.yml
> > > index 4bf7d023..fe550b81 100644
> > > --- a/.github/workflows/sanitizers-testing.yml
> > > +++ b/.github/workflows/sanitizers-testing.yml
> > 
> > 
> > <snipped>
> > 
> > > + exclude:
> > 
> > <snipped>
> > 
> > > + - OS: Linux
> > > + CC: gcc-15
> > > + - OS: Linux
> > > + CC: clang-21
> > 
> > May we bump gcc and clang instead for Linux as well (I suppose its
> > about time :)) ?
> > 
> > I suppose it helps to avoid too many excludes. Also, there is no need
> > for testing these specific compilers instead of modern ones in
> > sanitizers build.
> > 
> > If there are any issues when upgrading compilers for sanitizer builds
> > (infrastructure or tests failures) -- feel free to ignore.
> > 
> > <snipped>
> > 
> 
> I tried to use the top versions available for macOS and for Linux. We can’t use the same versions with our current setup.
> 
> With the macOS runner we have:
> Available formulas: ... gcc gcc@10 gcc@11 gcc@12 gcc@13 gcc@14 gcc@15 gcc@9 …
> Available formulas: ... llvm llvm@14 llvm@15 llvm@16 llvm@17 llvm@18 llvm@19 llvm@20 llvm@21 ...
> 
> With the Linux runner we have:
> 
> gcc-10/focal-updates,now 10.5.0-1ubuntu1~20.04 amd64 [installed]
> gcc-7/focal 7.5.0-6ubuntu2 amd64
> gcc-8/focal 8.4.0-3ubuntu2 amd64
> gcc-9/focal-updates,now 9.4.0-1ubuntu1~20.04.2 amd64 [installed,automatic]
> 
> clang-10/focal,now 1:10.0.0-4ubuntu1 amd64 [installed,automatic]
> clang-11/focal-updates,now 1:11.0.0-2~ubuntu20.04.1 amd64 [installed]
> clang-12/focal-updates 1:12.0.0-3ubuntu1~20.04.5 amd64
> clang-18/focal-updates 1:18.1.8-11~20.04.2 amd64
> clang-7/focal 1:7.0.1-12 amd64
> clang-8/focal 1:8.0.1-9 amd64
> clang-9/focal 1:9.0.1-12 amd64
> 
> The clang18 have broken dependencies with the current version of setup-sanitizers-linux. So I leave the current version in the context of ‘fix sanitizers for macOS patch’.

Ignore then. Thanks for the explanation!

> 
> > 
> > 
> > 
> > > --
> > > 2.49.0
> > >
> > 
> > --
> > Best regards,
> > Sergey Kaplun
> > 
> 
> Best regards,
> Evgeniy Temirgaleev

-- 
Best regards,
Sergey Kaplun

[Tarantool-patches] [PATCH luajit v3 3/3] ci: added a common way to disable perf workflow

[Evgeniy Temirgaleev via Tarantool-patches]

From: Temir Galeev <temir.galeev@bk.ru>

This allows a quick disable of all but one workflow to debug
a new CI behavior.

The way is to substitue '**-notest' with 'your-branch-name'
temporarily on other workflows while debugging the target workflow.
---
 .github/workflows/performance.yml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.github/workflows/performance.yml b/.github/workflows/performance.yml
index fe22eb4b..a872a265 100644
--- a/.github/workflows/performance.yml
+++ b/.github/workflows/performance.yml
@@ -3,6 +3,7 @@ name: Performance
 on:
   push:
     branches-ignore:
+      - '**-notest'
       - '**-noperf'
       - 'tarantool/release/**'
       - 'upstream-**'
-- 
2.49.0

Re: [Tarantool-patches] [PATCH luajit v3 3/3] ci: added a common way to disable perf workflow

[Sergey Kaplun via Tarantool-patches]

LGTM.

-- 
Best regards,
Sergey Kaplun