Skip to content

Fix benchmark start barrier deadlock - #83

Merged
LimiNode merged 1 commit into
codex/-odrfrom
codex/investigate-ci-linux-test-failure-fensn1
Dec 3, 2025
Merged

Fix benchmark start barrier deadlock#83
LimiNode merged 1 commit into
codex/-odrfrom
codex/investigate-ci-linux-test-failure-fensn1

Conversation

@LimiNode

@LimiNode LimiNode commented Dec 3, 2025

Copy link
Copy Markdown
Owner

Summary

  • add a dedicated readiness condition variable in the benchmark barrier so the main thread cannot miss wakeups when producers wait on the start flag
  • keep producer start coordination unchanged while ensuring multi-producer warmups complete without watchdog timeouts

Testing

  • LOGIT_BENCH_TOTAL=20000 LOGIT_BENCH_WARMUP=2000 ./build/logit_bench

Codex Task

@LimiNode
LimiNode merged commit 522f48d into codex/-odr Dec 3, 2025
1 check passed
@LimiNode
LimiNode deleted the codex/investigate-ci-linux-test-failure-fensn1 branch December 4, 2025 03:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant