mirror of
https://github.com/pezkuwichain/pezkuwi-subxt.git
synced 2026-05-01 00:47:55 +00:00
Sebastian Kunert
9a650c46fd
This PR provides the infrastructure for the pov-reclaim mechanism discussed in #209. The goal is to provide the current proof size to the runtime so it can be used to reclaim storage weight. ## New Host Function - A new host function is provided [here](https://github.com/skunert/polkadot-sdk/blob/5b317fda3be205f4136f10d4490387ccd4f9765d/cumulus/primitives/pov-reclaim/src/lib.rs#L23). It returns the size of the current proof size to the runtime. If recording is not enabled, it returns 0. ## Implementation Overview - Implement option to enable proof recording during import in the client. This is currently enabled for `polkadot-parachain`, `parachain-template` and the cumulus test node. - Make the proof recorder ready for no-std. It was previously only enabled for std environments, but we need to record the proof size in `validate_block` too. - Provide a recorder implementation that only the records the size of incoming nodes and does not store the nodes itself. - Fix benchmarks that were broken by async backing changes - Provide new externalities extension that is registered by default if proof recording is enabled. - I think we should discuss the naming, pov-reclaim was more intuitive to me, but we could also go with clawback like in the issue. ## Impact of proof recording during import With proof recording: 6.3058 Kelem/s Without proof recording: 6.3427 Kelem/s The measured impact on the importing performance is quite low on my machine using the block import benchmark. With proof recording I am seeing a performance hit of 0.585%. --------- Co-authored-by: command-bot <> Co-authored-by: Davide Galassi <davxy@datawok.net> Co-authored-by: Bastian Köcher <git@kchr.de>
79 lines
2.4 KiB
Rust
79 lines
2.4 KiB
Rust
// This file is part of Substrate.
|
|
|
|
// Copyright (C) Parity Technologies (UK) Ltd.
|
|
// SPDX-License-Identifier: Apache-2.0
|
|
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
//
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
|
|
use crate::utils::generate_runtime_interface_include;
|
|
|
|
use proc_macro2::{Span, TokenStream};
|
|
|
|
use syn::{Ident, ItemTrait, Result};
|
|
|
|
use inflector::Inflector;
|
|
|
|
use quote::quote;
|
|
|
|
mod bare_function_interface;
|
|
mod host_function_interface;
|
|
mod trait_decl_impl;
|
|
|
|
/// Custom keywords supported by the `runtime_interface` attribute.
|
|
pub mod keywords {
|
|
// Custom keyword `wasm_only` that can be given as attribute to [`runtime_interface`].
|
|
syn::custom_keyword!(wasm_only);
|
|
// Disable tracing-macros added to the [`runtime_interface`] by specifying this optional entry
|
|
syn::custom_keyword!(no_tracing);
|
|
}
|
|
|
|
/// Implementation of the `runtime_interface` attribute.
|
|
///
|
|
/// It expects the trait definition the attribute was put above and if this should be an wasm only
|
|
/// interface.
|
|
pub fn runtime_interface_impl(
|
|
trait_def: ItemTrait,
|
|
is_wasm_only: bool,
|
|
tracing: bool,
|
|
) -> Result<TokenStream> {
|
|
let bare_functions = bare_function_interface::generate(&trait_def, is_wasm_only, tracing)?;
|
|
let crate_include = generate_runtime_interface_include();
|
|
let mod_name = Ident::new(&trait_def.ident.to_string().to_snake_case(), Span::call_site());
|
|
let trait_decl_impl = trait_decl_impl::process(&trait_def, is_wasm_only)?;
|
|
let host_functions = host_function_interface::generate(&trait_def, is_wasm_only)?;
|
|
let vis = trait_def.vis;
|
|
let attrs = &trait_def.attrs;
|
|
|
|
let res = quote! {
|
|
#( #attrs )*
|
|
#vis mod #mod_name {
|
|
use super::*;
|
|
#crate_include
|
|
|
|
#bare_functions
|
|
|
|
#trait_decl_impl
|
|
|
|
#host_functions
|
|
}
|
|
};
|
|
|
|
let res = expander::Expander::new("runtime_interface")
|
|
.dry(std::env::var("EXPAND_MACROS").is_err())
|
|
.verbose(true)
|
|
.write_to_out_dir(res)
|
|
.expect("Does not fail because of IO in OUT_DIR; qed");
|
|
|
|
Ok(res)
|
|
}
|