Skip to content

Commit e8d0c4f

Browse files
authored
Merge pull request #559 from Sahil-u07/fix/shm-race-condition-195
fix(shm): serialise concurrent writers with seq# + POSIX semaphore
2 parents 7201471 + 7e979af commit e8d0c4f

5 files changed

Lines changed: 601 additions & 80 deletions

File tree

.github/workflows/ci.yml

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -124,3 +124,18 @@ jobs:
124124
- name: Validate Dockerfile build
125125
if: steps.filter.outputs.dockerfile == 'true'
126126
run: docker build -f Dockerfile.py -t concore-py-test .
127+
128+
cpp-shm-test:
129+
runs-on: ubuntu-latest
130+
steps:
131+
- uses: actions/checkout@v4
132+
133+
- name: Install g++
134+
run: sudo apt-get update && sudo apt-get install -y g++
135+
136+
- name: Parse concore.hpp standalone
137+
# -fsyntax-only parses concore.hpp without producing an object
138+
# file, so we don't need system SHM semaphores or shared-memory
139+
# segments to succeed. This catches header-level regressions on
140+
# the SHM transport (issue #195) without needing g++ at runtime.
141+
run: g++ -std=c++17 -Wall -Wextra -fsyntax-only tests/test_shm_cpp_smoke.cpp

0 commit comments

Comments
 (0)