Skip to content

Make prover thread count and aggregated-proof buffer configurable.#2796

Merged
bkolad merged 7 commits intodevfrom
blaze/prover_thread_count
May 1, 2026
Merged

Make prover thread count and aggregated-proof buffer configurable.#2796
bkolad merged 7 commits intodevfrom
blaze/prover_thread_count

Conversation

@bkolad
Copy link
Copy Markdown
Member

@bkolad bkolad commented Apr 30, 2026

Description

This PR introduces the folloing changes:

  1. Add prover_thread_count_override: Option<NonZero<usize>> to ProofManagerConfig. When None (default), prover_thread_count() keeps its previous value of 2 * aggregated_proof_block_jump + 1; when Some(n), the override is used directly.
  2. Add max_number_of_aggregated_proofs_in_memory: NonZero<usize> to ProofManagerConfig and thread it through start_zk_workflow_in_backgroundZkProofManager. This becomes the capacity of the intake → aggregator mpsc (previously hardcoded to 1),
    so operators can let more completed aggregation windows queue in memory before back-pressure stalls intake.
  3. Update the runner's config validation: max_number_of_transitions_in_db & max_number_of_transitions_in_memory
  4. Set max_number_of_aggregated_proofs_in_memory = 5 in the example/demo TOMLs and test rollup builders, and bump max_number_of_transitions_in_db/_in_memory accordingly so the new validation passes.
  • I have updated CHANGELOG.md with a new entry if my PR makes any breaking changes or fixes a bug. If my PR removes a feature or changes its behavior, I provide help for users on how to migrate to the new behavior.
  • I have carefully reviewed all my Cargo.toml changes before opening the PRs. (Are all new dependencies necessary? Is any module dependency leaked into the full-node (hint: it shouldn't)?)

Linked Issues

@bkolad bkolad changed the title WIP prover_thread_count Make prover thread count and aggregated-proof buffer configurable Apr 30, 2026
@bkolad bkolad changed the title Make prover thread count and aggregated-proof buffer configurable Make prover thread count and aggregated-proof buffer configurable. Apr 30, 2026
@bkolad bkolad force-pushed the blaze/prover_thread_count branch from d4a3ae8 to e8bc2c6 Compare April 30, 2026 09:12
Comment thread crates/full-node/full-node-configs/src/runner.rs
Comment thread crates/full-node/full-node-configs/src/runner.rs
@bkolad bkolad marked this pull request as ready for review April 30, 2026 09:24
Comment thread crates/full-node/full-node-configs/src/runner.rs Outdated
bkolad and others added 4 commits April 30, 2026 16:16
Co-authored-by: Theodore Bugnet <24320578+theodorebugnet@users.noreply.github.com>
@bkolad bkolad force-pushed the blaze/prover_thread_count branch from db8de28 to dd88c25 Compare April 30, 2026 14:16
@bkolad bkolad merged commit 0bc3cb9 into dev May 1, 2026
29 of 30 checks passed
@bkolad bkolad deleted the blaze/prover_thread_count branch May 1, 2026 07:04
@github-actions github-actions Bot locked and limited conversation to collaborators May 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants