From 3684dfec5e543c1e7479f29f8de3750fdd018a89 Mon Sep 17 00:00:00 2001
From: s0me0ne-unkn0wn <48632512+s0me0ne-unkn0wn@users.noreply.github.com>
Date: Wed, 16 Oct 2024 14:12:03 +0200
Subject: [PATCH 1/2] [stable2409] Backport #5887 (#6082)

---
 polkadot/runtime/parachains/Cargo.toml          |  7 ++++---
 .../runtime/parachains/src/configuration.rs     | 10 +++++++---
 .../parachains/src/configuration/tests.rs       |  2 +-
 prdoc/pr_6082.prdoc                             | 17 +++++++++++++++++
 4 files changed, 29 insertions(+), 7 deletions(-)
 create mode 100644 prdoc/pr_6082.prdoc

diff --git a/polkadot/runtime/parachains/Cargo.toml b/polkadot/runtime/parachains/Cargo.toml
index 3a745a2738a3..926e1ed3bf4c 100644
--- a/polkadot/runtime/parachains/Cargo.toml
+++ b/polkadot/runtime/parachains/Cargo.toml
@@ -18,9 +18,10 @@ scale-info = { features = ["derive"], workspace = true }
 serde = { features = ["alloc", "derive"], workspace = true }
 derive_more = { workspace = true, default-features = true }
 bitflags = { workspace = true }
-sp-api.workspace = true
-sp-inherents.workspace = true
-sp-io.workspace = true
+
+sp-api = { workspace = true }
+sp-inherents = { workspace = true }
+sp-io = { workspace = true }
 sp-runtime = { features = ["serde"], workspace = true }
 sp-session.workspace = true
 sp-staking = { features = ["serde"], workspace = true }
diff --git a/polkadot/runtime/parachains/src/configuration.rs b/polkadot/runtime/parachains/src/configuration.rs
index d09962ef2b44..34ccac57a67d 100644
--- a/polkadot/runtime/parachains/src/configuration.rs
+++ b/polkadot/runtime/parachains/src/configuration.rs
@@ -29,7 +29,7 @@ use polkadot_parachain_primitives::primitives::{
 use polkadot_primitives::{
 	ApprovalVotingParams, AsyncBackingParams, Balance, ExecutorParamError, ExecutorParams,
 	NodeFeatures, SessionIndex, LEGACY_MIN_BACKING_VOTES, MAX_CODE_SIZE, MAX_HEAD_DATA_SIZE,
-	MAX_POV_SIZE, ON_DEMAND_MAX_QUEUE_MAX_SIZE,
+	ON_DEMAND_MAX_QUEUE_MAX_SIZE,
 };
 use sp_runtime::{traits::Zero, Perbill, Percent};
 
@@ -46,6 +46,10 @@ use polkadot_primitives::vstaging::SchedulerParams;
 
 const LOG_TARGET: &str = "runtime::configuration";
 
+// This value is derived from network layer limits. See `sc_network::MAX_RESPONSE_SIZE` and
+// `polkadot_node_network_protocol::POV_RESPONSE_SIZE`.
+const POV_SIZE_HARD_LIMIT: u32 = 16 * 1024 * 1024;
+
 /// All configuration of the runtime with respect to paras.
 #[derive(
 	Clone,
@@ -310,7 +314,7 @@ pub enum InconsistentError<BlockNumber> {
 	MaxCodeSizeExceedHardLimit { max_code_size: u32 },
 	/// `max_head_data_size` exceeds the hard limit of `MAX_HEAD_DATA_SIZE`.
 	MaxHeadDataSizeExceedHardLimit { max_head_data_size: u32 },
-	/// `max_pov_size` exceeds the hard limit of `MAX_POV_SIZE`.
+	/// `max_pov_size` exceeds the hard limit of `POV_SIZE_HARD_LIMIT`.
 	MaxPovSizeExceedHardLimit { max_pov_size: u32 },
 	/// `minimum_validation_upgrade_delay` is less than `paras_availability_period`.
 	MinimumValidationUpgradeDelayLessThanChainAvailabilityPeriod {
@@ -377,7 +381,7 @@ where
 			})
 		}
 
-		if self.max_pov_size > MAX_POV_SIZE {
+		if self.max_pov_size > POV_SIZE_HARD_LIMIT {
 			return Err(MaxPovSizeExceedHardLimit { max_pov_size: self.max_pov_size })
 		}
 
diff --git a/polkadot/runtime/parachains/src/configuration/tests.rs b/polkadot/runtime/parachains/src/configuration/tests.rs
index dad8b6458e10..0d20399e471b 100644
--- a/polkadot/runtime/parachains/src/configuration/tests.rs
+++ b/polkadot/runtime/parachains/src/configuration/tests.rs
@@ -210,7 +210,7 @@ fn invariants() {
 		);
 
 		assert_err!(
-			Configuration::set_max_pov_size(RuntimeOrigin::root(), MAX_POV_SIZE + 1),
+			Configuration::set_max_pov_size(RuntimeOrigin::root(), POV_SIZE_HARD_LIMIT + 1),
 			Error::<Test>::InvalidNewValue
 		);
 
diff --git a/prdoc/pr_6082.prdoc b/prdoc/pr_6082.prdoc
new file mode 100644
index 000000000000..3ee6ac05a11a
--- /dev/null
+++ b/prdoc/pr_6082.prdoc
@@ -0,0 +1,17 @@
+# Schema: Polkadot SDK PRDoc Schema (prdoc) v1.0.0
+# See doc at https://raw.githubusercontent.com/paritytech/polkadot-sdk/master/prdoc/schema_user.json
+
+title: "Set reasonable hard limit for PoV size config value"
+
+doc:
+  - audience:
+    - Runtime Dev
+    - Runtime User
+    description: |
+      Sets the hard limit of the `max_pov_size` host configuration parameter to correspond to the
+      actual network-related limit rather than to a random constant.
+
+crates:
+  - name: polkadot-runtime-parachains
+    bump: patch
+

From 1bf94f488011e8f3324ffb82d85e7b4cdd110e7d Mon Sep 17 00:00:00 2001
From: Dmitry Sinyavin <dmitry.sinyavin@parity.io>
Date: Wed, 27 Nov 2024 13:58:00 +0100
Subject: [PATCH 2/2] Rename prdoc

---
 prdoc/{pr_6082.prdoc => pr_6674.prdoc} | 0
 1 file changed, 0 insertions(+), 0 deletions(-)
 rename prdoc/{pr_6082.prdoc => pr_6674.prdoc} (100%)

diff --git a/prdoc/pr_6082.prdoc b/prdoc/pr_6674.prdoc
similarity index 100%
rename from prdoc/pr_6082.prdoc
rename to prdoc/pr_6674.prdoc