Skip to content

docs(audits): stdlib sync channel overhead in transfer hot path#3825

Merged
oferchen merged 1 commit into
masterfrom
docs/stdlib-sync-channel-1592
May 7, 2026
Merged

docs(audits): stdlib sync channel overhead in transfer hot path#3825
oferchen merged 1 commit into
masterfrom
docs/stdlib-sync-channel-1592

Conversation

@oferchen
Copy link
Copy Markdown
Owner

@oferchen oferchen commented May 7, 2026

Summary

  • Survey of remaining std::sync::mpsc and Arc<Mutex<channel>> sites on the receiver-side transfer hot path after PR Explicitly disable Windows x86/aarch64 cross targets #1744 migrated work_queue.rs to crossbeam_channel::bounded.
  • Per-channel-op cost table comparing std, crossbeam, crossbeam_queue::ArrayQueue, and parking_lot::Mutex<VecDeque<T>> backends.
  • Criterion bench plan (crates/engine/benches/channel_overhead.rs) sweeping 1/4/16/64 producers across each backend, with end-to-end checks on real receiver workloads.
  • Migration recommendations: highest-confidence target is the Arc<Mutex<mpsc::Receiver>> shared queue at crates/signature/src/async_gen.rs:218 (site B). Two further bounded-channel migrations conditional on bench data. No parking_lot migration recommended.

Test plan

  • Doc-only change; rendering verified locally.
  • No code or test impact; CI fmt+clippy must still pass on existing tree.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label May 7, 2026
@oferchen oferchen merged commit 4e51014 into master May 7, 2026
8 checks passed
@oferchen oferchen deleted the docs/stdlib-sync-channel-1592 branch May 8, 2026 03:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant