Skip to content

Enable address sanitizer for mpi4py runs #10

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
devreal wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Enable address sanitizer for mpi4py runs #10

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
c6fc05d
Enable ASAN for mpi4py in CI
devreal Jan 14, 2026
7f5eea7
Remove disable of ASLR and apt update
devreal Jan 14, 2026
1af95bb
Enable ASAN for all mpi4py tests and increase optimizations
devreal Jan 14, 2026
b8a4c15
ASAN: explicitly disable stack-use-after-return check
devreal Jan 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
42 changes: 38 additions & 4 deletions .github/workflows/ompi_mpi4py.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,18 +20,25 @@ permissions:

jobs:
test:
runs-on: ubuntu-22.04
# We need Unbuntu 24.04 (over 22.04) due to a kernel bug,
# see https://github.com/google/sanitizers/issues/856.
runs-on: ubuntu-24.04
timeout-minutes: 30
env:
MPI4PY_TEST_SPAWN: true
# disable ASAN while building
ASAN_OPTIONS: verify_asan_link_order=0,detect_odr_violation=0,abort_on_error=0
# disable leak detection
LSAN_OPTIONS: detect_leaks=0,exitcode=0

steps:
- name: Configure hostname
run: echo 127.0.0.1 `hostname` | sudo tee -a /etc/hosts > /dev/null
if: ${{ runner.os == 'Linux' || runner.os == 'macOS' }}

- name: Install depencencies
run: sudo apt-get install -y -q
libnuma-dev
libnuma-dev libasan8
if: ${{ runner.os == 'Linux' }}

- name: Checkout Open MPI
Expand Down Expand Up @@ -59,7 +66,8 @@ jobs:
--disable-oshmem
--disable-silent-rules
--prefix=/opt/openmpi
LDFLAGS=-Wl,-rpath,/opt/openmpi/lib
CFLAGS="-O2 -fno-omit-frame-pointer -g -fsanitize=address"
LDFLAGS="-Wl,-rpath,/opt/openmpi/lib -fsanitize=address"
working-directory: mpi-build

- name: Build MPI
Expand Down Expand Up @@ -115,6 +123,21 @@ jobs:
env:
CFLAGS: "-O0"

- name: Setting up ASAN environment
# LD_PRELOAD is needed to make sure ASAN is the first thing loaded
# as it will otherwise complain.
# Leak detection is currently disabled because of the size of the report.
# The patcher is disabled because ASAN fails if code mmaps data at fixed
# memory addresses, see https://github.com/open-mpi/ompi/issues/12819.
# ODR violation detection is disabled until #13469 is fixed
# Disabling stack use after return detection to reduce slowdown, per
# https://github.com/llvm/llvm-project/issues/64190.
run: |
echo LD_PRELOAD=/usr/lib/x86_64-linux-gnu/libasan.so.8 >> $GITHUB_ENV
echo ASAN_OPTIONS=detect_odr_violation=0,abort_on_error=1,detect_stack_use_after_return=0 >> $GITHUB_ENV
echo LSAN_OPTIONS=detect_leaks=0,exitcode=0 >> $GITHUB_ENV
echo OMPI_MCA_memory=^patcher >> $GITHUB_ENV

- name: Test mpi4py (singleton)
run: python test/main.py -v -x TestExcErrhandlerNull
if: ${{ true }}
Expand Down Expand Up @@ -145,6 +168,18 @@ jobs:
if: ${{ true }}
timeout-minutes: 10

- name: Show MPI (ASAN)
run: ompi_info

- name: Show MPICC (ASAN)
run: mpicc -show

- name: Disabling ASAN environment
run: |
echo LD_PRELOAD= >> $GITHUB_ENV
echo ASAN_OPTIONS=verify_asan_link_order=0,detect_odr_violation=0,abort_on_error=0 >> $GITHUB_ENV
echo LSAN_OPTIONS=detect_leaks=0,exitcode=0 >> $GITHUB_ENV

- name: Relocate Open MPI installation
run: mv /opt/openmpi /opt/ompi
- name: Update PATH and set OPAL_PREFIX and LD_LIBRARY_PATH
Expand All @@ -157,4 +192,3 @@ jobs:
run: python test/main.py -v -x TestExcErrhandlerNull
if: ${{ true }}
timeout-minutes: 10

Loading