mirror of
https://github.com/pezkuwichain/pezkuwi-subxt.git
synced 2026-08-01 03:35:48 +00:00
479e51c67b
* Insert headers in benchmarks. * Fix benchmarks. * Lower number of headers. * Avoid hardcoding numbers. * Update weights. * cargo fmt --all * Remove todo. * Fix test. * Address review comments. * Fix borrowed value.
282 lines
9.4 KiB
Rust
282 lines
9.4 KiB
Rust
// Copyright 2019-2021 Parity Technologies (UK) Ltd.
|
|
// This file is part of Parity Bridges Common.
|
|
|
|
// Parity Bridges Common is free software: you can redistribute it and/or modify
|
|
// it under the terms of the GNU General Public License as published by
|
|
// the Free Software Foundation, either version 3 of the License, or
|
|
// (at your option) any later version.
|
|
|
|
// Parity Bridges Common is distributed in the hope that it will be useful,
|
|
// but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
// GNU General Public License for more details.
|
|
|
|
// You should have received a copy of the GNU General Public License
|
|
// along with Parity Bridges Common. If not, see <http://www.gnu.org/licenses/>.
|
|
|
|
//! Benchmarks for the GRANDPA Pallet.
|
|
//!
|
|
//! The main dispatchable for the GRANDPA pallet is `submit_finality_proof`, so these benchmarks are
|
|
//! based around that. There are to main factors which affect finality proof verification:
|
|
//!
|
|
//! 1. The number of `votes-ancestries` in the justification
|
|
//! 2. The number of `pre-commits` in the justification
|
|
//!
|
|
//! Vote ancestries are the headers between (`finality_target`, `head_of_chain`], where
|
|
//! `header_of_chain` is a decendant of `finality_target`.
|
|
//!
|
|
//! Pre-commits are messages which are signed by validators at the head of the chain they think is
|
|
//! the best.
|
|
//!
|
|
//! Consider the following:
|
|
//!
|
|
//! / [B'] <- [C']
|
|
//! [A] <- [B] <- [C]
|
|
//!
|
|
//! The common ancestor of both forks is block A, so this is what GRANDPA will finalize. In order to
|
|
//! verify this we will have vote ancestries of [B, C, B', C'] and pre-commits [C, C'].
|
|
//!
|
|
//! Note that the worst case scenario here would be a justification where each validator has it's
|
|
//! own fork which is `SESSION_LENGTH` blocks long.
|
|
//!
|
|
//! As far as benchmarking results go, the only benchmark that should be used in
|
|
//! `pallet-bridge-grandpa` to annotate weights is the `submit_finality_proof` one. The others are
|
|
//! looking at the effects of specific code paths and do not actually reflect the overall worst case
|
|
//! scenario.
|
|
|
|
use crate::*;
|
|
|
|
use bp_test_utils::{
|
|
accounts, authority_list, make_justification_for_header, test_keyring, JustificationGeneratorParams, ALICE,
|
|
TEST_GRANDPA_ROUND, TEST_GRANDPA_SET_ID,
|
|
};
|
|
use frame_benchmarking::{benchmarks_instance_pallet, whitelisted_caller};
|
|
use frame_support::traits::Get;
|
|
use frame_system::RawOrigin;
|
|
use sp_finality_grandpa::AuthorityId;
|
|
use sp_runtime::traits::Zero;
|
|
use sp_std::{vec, vec::Vec};
|
|
|
|
// The maximum number of vote ancestries to include in a justification.
|
|
//
|
|
// In practice this would be limited by the session length (number of blocks a single authority set
|
|
// can produce) of a given chain.
|
|
const MAX_VOTE_ANCESTRIES: u32 = 1000;
|
|
|
|
// The maximum number of pre-commits to include in a justification. In practice this scales with the
|
|
// number of validators.
|
|
const MAX_VALIDATOR_SET_SIZE: u32 = 1024;
|
|
|
|
/// Returns number of first header to be imported.
|
|
///
|
|
/// Since we boostrap the pallet with `HeadersToKeep` already imported headers,
|
|
/// this function computes the next expected header number to import.
|
|
fn header_number<T: Config<I>, I: 'static, N: From<u32>>() -> N {
|
|
(T::HeadersToKeep::get() + 1).into()
|
|
}
|
|
|
|
benchmarks_instance_pallet! {
|
|
// This is the "gold standard" benchmark for this extrinsic, and it's what should be used to
|
|
// annotate the weight in the pallet.
|
|
//
|
|
// The other benchmarks related to `submit_finality_proof` are looking at the effect of specific
|
|
// parameters and are there mostly for seeing how specific codepaths behave.
|
|
submit_finality_proof {
|
|
let v in 1..MAX_VOTE_ANCESTRIES;
|
|
let p in 1..MAX_VALIDATOR_SET_SIZE;
|
|
|
|
let caller: T::AccountId = whitelisted_caller();
|
|
|
|
let authority_list = accounts(p as u16)
|
|
.iter()
|
|
.map(|id| (AuthorityId::from(*id), 1))
|
|
.collect::<Vec<_>>();
|
|
|
|
let init_data = InitializationData {
|
|
header: bp_test_utils::test_header(Zero::zero()),
|
|
authority_list,
|
|
set_id: TEST_GRANDPA_SET_ID,
|
|
is_halted: false,
|
|
};
|
|
|
|
bootstrap_bridge::<T, I>(init_data);
|
|
|
|
let header: BridgedHeader<T, I> = bp_test_utils::test_header(header_number::<T, I, _>());
|
|
let params = JustificationGeneratorParams {
|
|
header: header.clone(),
|
|
round: TEST_GRANDPA_ROUND,
|
|
set_id: TEST_GRANDPA_SET_ID,
|
|
authorities: accounts(p as u16).iter().map(|k| (*k, 1)).collect::<Vec<_>>(),
|
|
votes: v,
|
|
forks: 1,
|
|
};
|
|
|
|
let justification = make_justification_for_header(params);
|
|
|
|
}: _(RawOrigin::Signed(caller), header, justification)
|
|
verify {
|
|
let header: BridgedHeader<T, I> = bp_test_utils::test_header(header_number::<T, I, _>());
|
|
let expected_hash = header.hash();
|
|
|
|
assert_eq!(<BestFinalized<T, I>>::get(), expected_hash);
|
|
assert!(<ImportedHeaders<T, I>>::contains_key(expected_hash));
|
|
}
|
|
|
|
// What we want to check here is the effect of vote ancestries on justification verification
|
|
// do this by varying the number of headers between `finality_target` and `header_of_chain`.
|
|
submit_finality_proof_on_single_fork {
|
|
let v in 1..MAX_VOTE_ANCESTRIES;
|
|
|
|
let caller: T::AccountId = whitelisted_caller();
|
|
|
|
let init_data = InitializationData {
|
|
header: bp_test_utils::test_header(Zero::zero()),
|
|
authority_list: authority_list(),
|
|
set_id: TEST_GRANDPA_SET_ID,
|
|
is_halted: false,
|
|
};
|
|
|
|
bootstrap_bridge::<T, I>(init_data);
|
|
let header: BridgedHeader<T, I> = bp_test_utils::test_header(header_number::<T, I, _>());
|
|
|
|
let params = JustificationGeneratorParams {
|
|
header: header.clone(),
|
|
round: TEST_GRANDPA_ROUND,
|
|
set_id: TEST_GRANDPA_SET_ID,
|
|
authorities: test_keyring(),
|
|
votes: v,
|
|
forks: 1,
|
|
};
|
|
|
|
let justification = make_justification_for_header(params);
|
|
|
|
}: submit_finality_proof(RawOrigin::Signed(caller), header, justification)
|
|
verify {
|
|
let header: BridgedHeader<T, I> = bp_test_utils::test_header(header_number::<T, I, _>());
|
|
let expected_hash = header.hash();
|
|
|
|
assert_eq!(<BestFinalized<T, I>>::get(), expected_hash);
|
|
assert!(<ImportedHeaders<T, I>>::contains_key(expected_hash));
|
|
}
|
|
|
|
// What we want to check here is the effect of many pre-commits on justification verification.
|
|
// We do this by creating many forks, whose head will be used as a signed pre-commit in the
|
|
// final justification.
|
|
submit_finality_proof_on_many_forks {
|
|
let p in 1..MAX_VALIDATOR_SET_SIZE;
|
|
|
|
let caller: T::AccountId = whitelisted_caller();
|
|
|
|
let authority_list = accounts(p as u16)
|
|
.iter()
|
|
.map(|id| (AuthorityId::from(*id), 1))
|
|
.collect::<Vec<_>>();
|
|
|
|
let init_data = InitializationData {
|
|
header: bp_test_utils::test_header(Zero::zero()),
|
|
authority_list,
|
|
set_id: TEST_GRANDPA_SET_ID,
|
|
is_halted: false,
|
|
};
|
|
|
|
bootstrap_bridge::<T, I>(init_data);
|
|
let header: BridgedHeader<T, I> = bp_test_utils::test_header(header_number::<T, I, _>());
|
|
|
|
let params = JustificationGeneratorParams {
|
|
header: header.clone(),
|
|
round: TEST_GRANDPA_ROUND,
|
|
set_id: TEST_GRANDPA_SET_ID,
|
|
authorities: accounts(p as u16).iter().map(|k| (*k, 1)).collect::<Vec<_>>(),
|
|
votes: p,
|
|
forks: p,
|
|
};
|
|
|
|
let justification = make_justification_for_header(params);
|
|
|
|
}: submit_finality_proof(RawOrigin::Signed(caller), header, justification)
|
|
verify {
|
|
let header: BridgedHeader<T, I> = bp_test_utils::test_header(header_number::<T, I, _>());
|
|
let expected_hash = header.hash();
|
|
|
|
assert_eq!(<BestFinalized<T, I>>::get(), expected_hash);
|
|
assert!(<ImportedHeaders<T, I>>::contains_key(expected_hash));
|
|
}
|
|
|
|
// Here we want to find out the overheaded of looking through consensus digests found in a
|
|
// header. As the number of logs in a header grows, how much more work do we require to look
|
|
// through them?
|
|
//
|
|
// Note that this should be the same for looking through scheduled changes and forces changes,
|
|
// which is why we only have one benchmark for this.
|
|
find_scheduled_change {
|
|
// Not really sure what a good bound for this is.
|
|
let n in 1..1000;
|
|
|
|
let mut logs = vec![];
|
|
for i in 0..n {
|
|
// We chose a non-consensus log on purpose since that way we have to look through all
|
|
// the logs in the header
|
|
logs.push(sp_runtime::DigestItem::Other(vec![]));
|
|
}
|
|
|
|
let mut header: BridgedHeader<T, I> = bp_test_utils::test_header(Zero::zero());
|
|
let digest = header.digest_mut();
|
|
*digest = sp_runtime::Digest {
|
|
logs,
|
|
};
|
|
|
|
}: {
|
|
crate::find_scheduled_change(&header)
|
|
}
|
|
|
|
// What we want to check here is how long it takes to read and write the authority set tracked
|
|
// by the pallet as the number of authorities grows.
|
|
read_write_authority_sets {
|
|
// The current max target number of validators on Polkadot/Kusama
|
|
let n in 1..1000;
|
|
|
|
let mut authorities = vec![];
|
|
for i in 0..n {
|
|
authorities.push((ALICE, 1));
|
|
}
|
|
|
|
let authority_set = bp_header_chain::AuthoritySet {
|
|
authorities: authorities.iter().map(|(id, w)| (AuthorityId::from(*id), *w)).collect(),
|
|
set_id: 0
|
|
};
|
|
|
|
<CurrentAuthoritySet<T, I>>::put(&authority_set);
|
|
|
|
}: {
|
|
let authority_set = <CurrentAuthoritySet<T, I>>::get();
|
|
<CurrentAuthoritySet<T, I>>::put(&authority_set);
|
|
}
|
|
}
|
|
|
|
#[cfg(test)]
|
|
mod tests {
|
|
use super::*;
|
|
use frame_support::assert_ok;
|
|
|
|
#[test]
|
|
fn finality_proof_is_valid() {
|
|
mock::run_test(|| {
|
|
assert_ok!(test_benchmark_submit_finality_proof::<mock::TestRuntime>());
|
|
});
|
|
}
|
|
|
|
#[test]
|
|
fn single_fork_finality_proof_is_valid() {
|
|
mock::run_test(|| {
|
|
assert_ok!(test_benchmark_submit_finality_proof_on_single_fork::<mock::TestRuntime>());
|
|
});
|
|
}
|
|
|
|
#[test]
|
|
fn multi_fork_finality_proof_is_valid() {
|
|
mock::run_test(|| {
|
|
assert_ok!(test_benchmark_submit_finality_proof_on_many_forks::<mock::TestRuntime>());
|
|
});
|
|
}
|
|
}
|