Skip to content

Commit 75bbeea

Browse files
alexgghgithub-actions[bot]
authored andcommitted
Increase the number of pvf execute workers (#7116)
Reference hardware requirements have been bumped to at least 8 cores so we can no allocate 50% of that capacity to PVF execution. --------- Signed-off-by: Alexandru Gheorghe <[email protected]> (cherry picked from commit f0eec07)
1 parent 268a928 commit 75bbeea

File tree

2 files changed

+11
-8
lines changed

2 files changed

+11
-8
lines changed

polkadot/node/service/src/lib.rs

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -944,14 +944,9 @@ pub fn new_full<
944944
secure_validator_mode,
945945
prep_worker_path,
946946
exec_worker_path,
947-
pvf_execute_workers_max_num: execute_workers_max_num.unwrap_or_else(
948-
|| match config.chain_spec.identify_chain() {
949-
// The intention is to use this logic for gradual increasing from 2 to 4
950-
// of this configuration chain by chain until it reaches production chain.
951-
Chain::Polkadot | Chain::Kusama => 2,
952-
Chain::Rococo | Chain::Westend | Chain::Unknown => 4,
953-
},
954-
),
947+
// Default execution workers is 4 because we have 8 cores on the reference hardware,
948+
// and this accounts for 50% of that cpu capacity.
949+
pvf_execute_workers_max_num: execute_workers_max_num.unwrap_or(4),
955950
pvf_prepare_workers_soft_max_num: prepare_workers_soft_max_num.unwrap_or(1),
956951
pvf_prepare_workers_hard_max_num: prepare_workers_hard_max_num.unwrap_or(2),
957952
})

prdoc/pr_7116.prdoc

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
title: Increase the number of pvf execution workers from 2 to 4
2+
doc:
3+
- audience: Node Dev
4+
description: |-
5+
Increase the number of pvf execution workers from 2 to 4.
6+
crates:
7+
- name: polkadot-service
8+
bump: patch

0 commit comments

Comments
 (0)