Set a proper proof size block limit

prdoc/pr_10535.prdoc

@@ -0,0 +1,14 @@
+title: Set a proper proof size block limit
+doc:
+- audience: Runtime Dev
+  description: |-
+    The current block limit of the revive dev node defined its `proof_size` as `u64::MAX`.
+
+    This is a reasonable setting for a standalone chain as the PoV as a limiting resource is only relevant for parachains.
+
+    However, this gives some confusing gas mapping calculations: they are correct and consistent but the resulting `proof_size` weights are unexpectedly high.
+
+    This PR sets the `proof_size` of the block limit to the same value as the Polkadot Asset Hub.
+crates:
+- name: revive-dev-runtime
+  bump: patch

substrate/frame/revive/dev-node/runtime/Cargo.toml

@@ -19,6 +19,7 @@ polkadot-sdk = { workspace = true, features = [
 	"pallet-transaction-payment",
 	"pallet-transaction-payment-rpc-runtime-api",
 	"parachains-common",
+	"polkadot-primitives",
 	"polkadot-runtime-common",
 	"runtime",
 	"with-tracing",

substrate/frame/revive/dev-node/runtime/src/lib.rs

@@ -269,8 +269,10 @@ const AVERAGE_ON_INITIALIZE_RATIO: Perbill = Perbill::from_percent(10);
 /// by  Operational  extrinsics.
 const NORMAL_DISPATCH_RATIO: Perbill = Perbill::from_percent(75);
 /// We allow for 2 seconds of compute with a 6 second average block time, with maximum proof size.
-const MAXIMUM_BLOCK_WEIGHT: Weight =
-	Weight::from_parts(WEIGHT_REF_TIME_PER_SECOND.saturating_mul(2), u64::MAX);
+const MAXIMUM_BLOCK_WEIGHT: Weight = Weight::from_parts(
+	WEIGHT_REF_TIME_PER_SECOND.saturating_mul(2),
+	polkadot_primitives::MAX_POV_SIZE as u64,
+);
 
 parameter_types! {
 	pub const Version: RuntimeVersion = VERSION;
@@ -298,6 +300,7 @@ parameter_types! {
 #[derive_impl(frame_system::config_preludes::SolochainDefaultConfig)]
 impl frame_system::Config for Runtime {
 	type Block = Block;
+	type BlockWeights = RuntimeBlockWeights;
 	type Version = Version;
 	type AccountId = AccountId;
 	type Hash = Hash;