diff --git a/Cargo.lock b/Cargo.lock
index af5c3b6309319db0323c51a2a62a1ff19782b2a3..78c18446082ba44f121d6e73fa4780293d9bf539 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -4341,6 +4341,7 @@ dependencies = [
"pallet-message-queue",
"parity-scale-codec",
"polkadot-parachain-primitives",
+ "polkadot-runtime-common",
"polkadot-runtime-parachains",
"rand",
"sc-client-api",
@@ -26925,13 +26926,10 @@ name = "xcm-executor-integration-tests"
version = "1.0.0"
dependencies = [
"frame-support",
- "frame-system",
"futures",
- "pallet-sudo",
"pallet-transaction-payment",
"pallet-xcm",
"parity-scale-codec",
- "polkadot-runtime-parachains",
"polkadot-test-client",
"polkadot-test-runtime",
"polkadot-test-service",
diff --git a/bridges/bin/runtime-common/Cargo.toml b/bridges/bin/runtime-common/Cargo.toml
index d7a66d335eedda128183904bb348b4c9a411cd38..e3b243cd61f756db0bd757f80c5b785da1e5c5eb 100644
--- a/bridges/bin/runtime-common/Cargo.toml
+++ b/bridges/bin/runtime-common/Cargo.toml
@@ -86,7 +86,6 @@ runtime-benchmarks = [
"pallet-utility/runtime-benchmarks",
"sp-runtime/runtime-benchmarks",
"sp-trie",
- "xcm/runtime-benchmarks",
]
integrity-test = ["static_assertions"]
test-helpers = ["bp-runtime/test-helpers", "sp-trie"]
diff --git a/bridges/modules/xcm-bridge-hub-router/Cargo.toml b/bridges/modules/xcm-bridge-hub-router/Cargo.toml
index 113722fad6ea7157d03b58dd21edf15927da8e25..3eb747097b3a0c45add84455a63b1eb4baca369e 100644
--- a/bridges/modules/xcm-bridge-hub-router/Cargo.toml
+++ b/bridges/modules/xcm-bridge-hub-router/Cargo.toml
@@ -50,7 +50,6 @@ runtime-benchmarks = [
"frame-system/runtime-benchmarks",
"sp-runtime/runtime-benchmarks",
"xcm-builder/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
"frame-support/try-runtime",
diff --git a/bridges/modules/xcm-bridge-hub/Cargo.toml b/bridges/modules/xcm-bridge-hub/Cargo.toml
index d358f05183172e8f16fa92e386d08941aa26d254..8c494b0d7cfee6f4a3a050df3bac5af0f0282d45 100644
--- a/bridges/modules/xcm-bridge-hub/Cargo.toml
+++ b/bridges/modules/xcm-bridge-hub/Cargo.toml
@@ -62,7 +62,6 @@ runtime-benchmarks = [
"sp-runtime/runtime-benchmarks",
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
"frame-support/try-runtime",
diff --git a/bridges/snowbridge/pallets/inbound-queue/Cargo.toml b/bridges/snowbridge/pallets/inbound-queue/Cargo.toml
index 8ff155ac018946ca3e808595173598f305eda59d..363620ff90a41d83617c990e45422f386c121c45 100644
--- a/bridges/snowbridge/pallets/inbound-queue/Cargo.toml
+++ b/bridges/snowbridge/pallets/inbound-queue/Cargo.toml
@@ -79,7 +79,6 @@ runtime-benchmarks = [
"snowbridge-router-primitives/runtime-benchmarks",
"sp-runtime/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
"frame-support/try-runtime",
diff --git a/bridges/snowbridge/pallets/system/Cargo.toml b/bridges/snowbridge/pallets/system/Cargo.toml
index 46ae5987433dca9e34fea54f49bb35434acdc9a8..3f820aae4044bf9e55a144c21f269b9de38ff779 100644
--- a/bridges/snowbridge/pallets/system/Cargo.toml
+++ b/bridges/snowbridge/pallets/system/Cargo.toml
@@ -64,7 +64,6 @@ runtime-benchmarks = [
"snowbridge-core/runtime-benchmarks",
"sp-runtime/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
"frame-support/try-runtime",
diff --git a/bridges/snowbridge/primitives/core/Cargo.toml b/bridges/snowbridge/primitives/core/Cargo.toml
index 1a2b98ebc7b61796b224b11921dbd8dc42dead8e..93a655e0dfeefc7b2c4cc3331b81878fc437d2a8 100644
--- a/bridges/snowbridge/primitives/core/Cargo.toml
+++ b/bridges/snowbridge/primitives/core/Cargo.toml
@@ -60,6 +60,4 @@ runtime-benchmarks = [
"polkadot-parachain-primitives/runtime-benchmarks",
"sp-runtime/runtime-benchmarks",
"xcm-builder/runtime-benchmarks",
- "xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
diff --git a/bridges/snowbridge/primitives/router/Cargo.toml b/bridges/snowbridge/primitives/router/Cargo.toml
index 9e3ce7767b7fce43ae4f4f25634fc8d2f45e6ea8..32cb4731c4bf42cd07d8aee346770f1caac2e839 100644
--- a/bridges/snowbridge/primitives/router/Cargo.toml
+++ b/bridges/snowbridge/primitives/router/Cargo.toml
@@ -48,5 +48,4 @@ runtime-benchmarks = [
"snowbridge-core/runtime-benchmarks",
"sp-runtime/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
diff --git a/bridges/snowbridge/runtime/runtime-common/Cargo.toml b/bridges/snowbridge/runtime/runtime-common/Cargo.toml
index 7bf84eadb091723a44d13d8aa9dcb8de0d2c4be9..2cf9a4dbe0fcfa354b12348fe77e560096c18053 100644
--- a/bridges/snowbridge/runtime/runtime-common/Cargo.toml
+++ b/bridges/snowbridge/runtime/runtime-common/Cargo.toml
@@ -42,5 +42,4 @@ runtime-benchmarks = [
"snowbridge-core/runtime-benchmarks",
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
diff --git a/bridges/snowbridge/runtime/test-common/Cargo.toml b/bridges/snowbridge/runtime/test-common/Cargo.toml
index 34171935f0591c0b8cd0495d7e498f1be48389bb..abf8422b553926dad968bbd3dd762025fc3f0e04 100644
--- a/bridges/snowbridge/runtime/test-common/Cargo.toml
+++ b/bridges/snowbridge/runtime/test-common/Cargo.toml
@@ -83,6 +83,5 @@ runtime-benchmarks = [
"snowbridge-pallet-system/runtime-benchmarks",
"sp-runtime/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
fast-runtime = []
diff --git a/cumulus/pallets/dmp-queue/Cargo.toml b/cumulus/pallets/dmp-queue/Cargo.toml
index 45ad608581aff6ca941f5e26df3fc7606d7f3167..3a7489208382de6d62f1e2d3d97724c7e90b39bc 100644
--- a/cumulus/pallets/dmp-queue/Cargo.toml
+++ b/cumulus/pallets/dmp-queue/Cargo.toml
@@ -51,7 +51,6 @@ runtime-benchmarks = [
"frame-support/runtime-benchmarks",
"frame-system/runtime-benchmarks",
"sp-runtime/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
diff --git a/cumulus/pallets/parachain-system/Cargo.toml b/cumulus/pallets/parachain-system/Cargo.toml
index 4bbcb96896fb1f83f07dde5adb5fff5e7a336c9d..4fb431762075987b9ca26066d0a3324e2411d470 100644
--- a/cumulus/pallets/parachain-system/Cargo.toml
+++ b/cumulus/pallets/parachain-system/Cargo.toml
@@ -31,15 +31,14 @@ sp-std.workspace = true
sp-trie.workspace = true
sp-version.workspace = true
polkadot-parachain-primitives = { features = ["wasm-api"], workspace = true }
-polkadot-runtime-parachains = { workspace = true }
-xcm = { workspace = true }
-xcm-builder = { workspace = true }
-
-# Cumulus
-cumulus-pallet-parachain-system-proc-macro = { workspace = true }
-cumulus-primitives-core = { workspace = true }
-cumulus-primitives-parachain-inherent = { workspace = true }
-cumulus-primitives-proof-size-hostfunction = { workspace = true }
+polkadot-runtime-parachains.workspace = true
+polkadot-runtime-common = { optional = true, workspace = true }
+xcm.workspace = true
+xcm-builder.workspace = true
+cumulus-pallet-parachain-system-proc-macro.workspace = true
+cumulus-primitives-core.workspace = true
+cumulus-primitives-parachain-inherent.workspace = true
+cumulus-primitives-proof-size-hostfunction.workspace = true
[dev-dependencies]
assert_matches = { workspace = true }
@@ -77,6 +76,7 @@ std = [
"log/std",
"pallet-message-queue/std",
"polkadot-parachain-primitives/std",
+ "polkadot-runtime-common/std",
"polkadot-runtime-parachains/std",
"scale-info/std",
"sp-core/std",
@@ -99,16 +99,17 @@ runtime-benchmarks = [
"frame-system/runtime-benchmarks",
"pallet-message-queue/runtime-benchmarks",
"polkadot-parachain-primitives/runtime-benchmarks",
+ "polkadot-runtime-common/runtime-benchmarks",
"polkadot-runtime-parachains/runtime-benchmarks",
"sp-runtime/runtime-benchmarks",
"xcm-builder/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
"frame-support/try-runtime",
"frame-system/try-runtime",
"pallet-message-queue/try-runtime",
+ "polkadot-runtime-common?/try-runtime",
"polkadot-runtime-parachains/try-runtime",
"sp-runtime/try-runtime",
]
diff --git a/cumulus/pallets/parachain-system/src/lib.rs b/cumulus/pallets/parachain-system/src/lib.rs
index 0fa759357f653dc98948f38d768ea4ddf749ab92..39fc8321a072ea9380e728ce19a94a064921aa82 100644
--- a/cumulus/pallets/parachain-system/src/lib.rs
+++ b/cumulus/pallets/parachain-system/src/lib.rs
@@ -1636,7 +1636,7 @@ impl<T: Config> InspectMessageQueues for Pallet<T> {
}
#[cfg(feature = "runtime-benchmarks")]
-impl<T: Config> polkadot_runtime_parachains::EnsureForParachain for Pallet<T> {
+impl<T: Config> polkadot_runtime_common::xcm_sender::EnsureForParachain for Pallet<T> {
fn ensure(para_id: ParaId) {
if let ChannelStatus::Closed = Self::get_channel_status(para_id) {
Self::open_outbound_hrmp_channel_for_benchmarks_or_tests(para_id)
diff --git a/cumulus/pallets/xcmp-queue/Cargo.toml b/cumulus/pallets/xcmp-queue/Cargo.toml
index b0ce5bd35ac7ebcf2ad67473cc0e952183c40184..7c69e6fd2d24589c441b9781943ce35612788678 100644
--- a/cumulus/pallets/xcmp-queue/Cargo.toml
+++ b/cumulus/pallets/xcmp-queue/Cargo.toml
@@ -73,7 +73,6 @@ runtime-benchmarks = [
"sp-runtime/runtime-benchmarks",
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
"frame-support/try-runtime",
diff --git a/cumulus/parachains/common/Cargo.toml b/cumulus/parachains/common/Cargo.toml
index b435408d0c22516ef8c39b3492069aaeb92abd0d..0ae9bfcc460808a7d1880e81d3fb5e681b0ccd5f 100644
--- a/cumulus/parachains/common/Cargo.toml
+++ b/cumulus/parachains/common/Cargo.toml
@@ -85,5 +85,4 @@ runtime-benchmarks = [
"polkadot-primitives/runtime-benchmarks",
"sp-runtime/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
diff --git a/cumulus/parachains/integration-tests/emulated/common/src/impls.rs b/cumulus/parachains/integration-tests/emulated/common/src/impls.rs
index 9dad323aa19c72a03d2ed5229b114fcb9d7c0d1b..c0d42cf2758e9431c87e4187eb42b9ab703f7f54 100644
--- a/cumulus/parachains/integration-tests/emulated/common/src/impls.rs
+++ b/cumulus/parachains/integration-tests/emulated/common/src/impls.rs
@@ -370,8 +370,6 @@ macro_rules! impl_send_transact_helpers_for_relay_chain {
let destination: $crate::impls::Location = <Self as RelayChain>::child_location_of(recipient);
let xcm = $crate::impls::xcm_transact_unpaid_execution(call, $crate::impls::OriginKind::Superuser);
- $crate::impls::dmp::Pallet::<<Self as $crate::impls::Chain>::Runtime>::make_parachain_reachable(recipient);
-
// Send XCM `Transact`
$crate::impls::assert_ok!(<Self as [<$chain RelayPallet>]>::XcmPallet::send(
root_origin,
diff --git a/cumulus/parachains/integration-tests/emulated/common/src/macros.rs b/cumulus/parachains/integration-tests/emulated/common/src/macros.rs
index cd2b41e5198f8de43b332b4bd404739206ff364b..b776cafb2545e8e25b7e137fbf448ee35293a66a 100644
--- a/cumulus/parachains/integration-tests/emulated/common/src/macros.rs
+++ b/cumulus/parachains/integration-tests/emulated/common/src/macros.rs
@@ -23,7 +23,6 @@ pub use pallet_message_queue;
pub use pallet_xcm;
// Polkadot
-pub use polkadot_runtime_parachains::dmp::Pallet as Dmp;
pub use xcm::{
prelude::{
AccountId32, All, Asset, AssetId, BuyExecution, DepositAsset, ExpectTransactStatus,
@@ -157,8 +156,6 @@ macro_rules! test_relay_is_trusted_teleporter {
// Send XCM message from Relay
<$sender_relay>::execute_with(|| {
- $crate::macros::Dmp::<<$sender_relay as $crate::macros::Chain>::Runtime>::make_parachain_reachable(<$receiver_para>::para_id());
-
assert_ok!(<$sender_relay as [<$sender_relay Pallet>]>::XcmPallet::limited_teleport_assets(
origin.clone(),
bx!(para_destination.clone().into()),
diff --git a/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-rococo/src/tests/hybrid_transfers.rs b/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-rococo/src/tests/hybrid_transfers.rs
index fb95c361f089188a5bb35e6f7daf9f49c868136b..baec7d20f4156657134ba8963adc61208c29266f 100644
--- a/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-rococo/src/tests/hybrid_transfers.rs
+++ b/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-rococo/src/tests/hybrid_transfers.rs
@@ -13,8 +13,6 @@
// See the License for the specific language governing permissions and
// limitations under the License.
-use rococo_system_emulated_network::rococo_emulated_chain::rococo_runtime::Dmp;
-
use super::reserve_transfer::*;
use crate::{
imports::*,
@@ -779,8 +777,6 @@ fn transfer_native_asset_from_relay_to_para_through_asset_hub() {
xcm: xcm_on_final_dest,
}]);
- Dmp::make_parachain_reachable(AssetHubRococo::para_id());
-
// First leg is a teleport, from there a local-reserve-transfer to final dest
<Rococo as RococoPallet>::XcmPallet::transfer_assets_using_type_and_then(
t.signed_origin,
diff --git a/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-rococo/src/tests/reserve_transfer.rs b/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-rococo/src/tests/reserve_transfer.rs
index 407a581afeb97c0ff9b09666d75f793557d32035..d642e877f00243dab5d4a7ccc11a977a5c816062 100644
--- a/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-rococo/src/tests/reserve_transfer.rs
+++ b/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-rococo/src/tests/reserve_transfer.rs
@@ -14,7 +14,6 @@
// limitations under the License.
use crate::imports::*;
-use rococo_system_emulated_network::rococo_emulated_chain::rococo_runtime::Dmp;
use sp_core::{crypto::get_public_from_string_or_panic, sr25519};
fn relay_to_para_sender_assertions(t: RelayToParaTest) {
@@ -488,11 +487,6 @@ pub fn para_to_para_through_hop_receiver_assertions<Hop: Clone>(t: Test<PenpalA,
}
fn relay_to_para_reserve_transfer_assets(t: RelayToParaTest) -> DispatchResult {
- let Junction::Parachain(para_id) = *t.args.dest.chain_location().last().unwrap() else {
- unimplemented!("Destination is not a parachain?")
- };
-
- Dmp::make_parachain_reachable(para_id);
<Rococo as RococoPallet>::XcmPallet::limited_reserve_transfer_assets(
t.signed_origin,
bx!(t.args.dest.into()),
@@ -552,13 +546,6 @@ fn para_to_system_para_reserve_transfer_assets(t: ParaToSystemParaTest) -> Dispa
fn para_to_para_through_relay_limited_reserve_transfer_assets(
t: ParaToParaThroughRelayTest,
) -> DispatchResult {
- let Junction::Parachain(para_id) = *t.args.dest.chain_location().last().unwrap() else {
- unimplemented!("Destination is not a parachain?")
- };
-
- Rococo::ext_wrapper(|| {
- Dmp::make_parachain_reachable(para_id);
- });
<PenpalA as PenpalAPallet>::PolkadotXcm::limited_reserve_transfer_assets(
t.signed_origin,
bx!(t.args.dest.into()),
diff --git a/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-rococo/src/tests/treasury.rs b/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-rococo/src/tests/treasury.rs
index 8648c8ce9311d05430468b01510f8f88514d5e08..69111d38bcac3b8d7278be81170d8158ba0bf4ea 100644
--- a/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-rococo/src/tests/treasury.rs
+++ b/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-rococo/src/tests/treasury.rs
@@ -29,7 +29,6 @@ use frame_support::{
use parachains_common::AccountId;
use polkadot_runtime_common::impls::VersionedLocatableAsset;
use rococo_runtime_constants::currency::GRAND;
-use rococo_system_emulated_network::rococo_emulated_chain::rococo_runtime::Dmp;
use xcm_executor::traits::ConvertLocation;
// Fund Treasury account on Asset Hub from Treasury account on Relay Chain with ROCs.
@@ -65,7 +64,6 @@ fn spend_roc_on_asset_hub() {
treasury_balance * 2,
));
- Dmp::make_parachain_reachable(1000);
let native_asset = Location::here();
let asset_hub_location: Location = [Parachain(1000)].into();
let treasury_location: Location = (Parent, PalletInstance(18)).into();
@@ -201,8 +199,6 @@ fn create_and_claim_treasury_spend_in_usdt() {
// create a conversion rate from `asset_kind` to the native currency.
assert_ok!(AssetRate::create(root.clone(), Box::new(asset_kind.clone()), 2.into()));
- Dmp::make_parachain_reachable(1000);
-
// create and approve a treasury spend.
assert_ok!(Treasury::spend(
root,
diff --git a/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-westend/src/tests/hybrid_transfers.rs b/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-westend/src/tests/hybrid_transfers.rs
index 91ebdda16828a6fa6039c73d64dc226da9be1521..0686bd71d085438912e71215b3857fac7a7bda4c 100644
--- a/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-westend/src/tests/hybrid_transfers.rs
+++ b/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-westend/src/tests/hybrid_transfers.rs
@@ -13,8 +13,6 @@
// See the License for the specific language governing permissions and
// limitations under the License.
-use westend_system_emulated_network::westend_emulated_chain::westend_runtime::Dmp;
-
use super::reserve_transfer::*;
use crate::{
imports::*,
@@ -780,8 +778,6 @@ fn transfer_native_asset_from_relay_to_penpal_through_asset_hub() {
xcm: xcm_on_final_dest,
}]);
- Dmp::make_parachain_reachable(AssetHubWestend::para_id());
-
// First leg is a teleport, from there a local-reserve-transfer to final dest
<Westend as WestendPallet>::XcmPallet::transfer_assets_using_type_and_then(
t.signed_origin,
diff --git a/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-westend/src/tests/reserve_transfer.rs b/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-westend/src/tests/reserve_transfer.rs
index dc36fed42932c5ed219c87c4f20bc86e111443bd..707e8adc8a566df1357c63500bbe230a8aaa9cf4 100644
--- a/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-westend/src/tests/reserve_transfer.rs
+++ b/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-westend/src/tests/reserve_transfer.rs
@@ -15,7 +15,6 @@
use crate::{create_pool_with_wnd_on, foreign_balance_on, imports::*};
use sp_core::{crypto::get_public_from_string_or_panic, sr25519};
-use westend_system_emulated_network::westend_emulated_chain::westend_runtime::Dmp;
fn relay_to_para_sender_assertions(t: RelayToParaTest) {
type RuntimeEvent = <Westend as Chain>::RuntimeEvent;
@@ -488,11 +487,6 @@ pub fn para_to_para_through_hop_receiver_assertions<Hop: Clone>(t: Test<PenpalA,
}
fn relay_to_para_reserve_transfer_assets(t: RelayToParaTest) -> DispatchResult {
- let Junction::Parachain(para_id) = *t.args.dest.chain_location().last().unwrap() else {
- unimplemented!("Destination is not a parachain?")
- };
-
- Dmp::make_parachain_reachable(para_id);
<Westend as WestendPallet>::XcmPallet::limited_reserve_transfer_assets(
t.signed_origin,
bx!(t.args.dest.into()),
@@ -539,13 +533,6 @@ fn para_to_system_para_reserve_transfer_assets(t: ParaToSystemParaTest) -> Dispa
fn para_to_para_through_relay_limited_reserve_transfer_assets(
t: ParaToParaThroughRelayTest,
) -> DispatchResult {
- let Junction::Parachain(para_id) = *t.args.dest.chain_location().last().unwrap() else {
- unimplemented!("Destination is not a parachain?")
- };
-
- Westend::ext_wrapper(|| {
- Dmp::make_parachain_reachable(para_id);
- });
<PenpalA as PenpalAPallet>::PolkadotXcm::limited_reserve_transfer_assets(
t.signed_origin,
bx!(t.args.dest.into()),
diff --git a/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-westend/src/tests/treasury.rs b/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-westend/src/tests/treasury.rs
index 3b53557fc05c614046c548e8b01a97b971dcd45b..c303e6411d33b7a4485aa0132855d0237a26eb1b 100644
--- a/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-westend/src/tests/treasury.rs
+++ b/cumulus/parachains/integration-tests/emulated/tests/assets/asset-hub-westend/src/tests/treasury.rs
@@ -20,7 +20,6 @@ use emulated_integration_tests_common::{
};
use frame_support::traits::fungibles::{Inspect, Mutate};
use polkadot_runtime_common::impls::VersionedLocatableAsset;
-use westend_system_emulated_network::westend_emulated_chain::westend_runtime::Dmp;
use xcm_executor::traits::ConvertLocation;
#[test]
@@ -59,8 +58,6 @@ fn create_and_claim_treasury_spend() {
// create a conversion rate from `asset_kind` to the native currency.
assert_ok!(AssetRate::create(root.clone(), Box::new(asset_kind.clone()), 2.into()));
- Dmp::make_parachain_reachable(1000);
-
// create and approve a treasury spend.
assert_ok!(Treasury::spend(
root,
diff --git a/cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-rococo/src/tests/send_xcm.rs b/cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-rococo/src/tests/send_xcm.rs
index cfcb581238e6d2415b3b3aedefd0719da8a7a7be..116ec4dc0e55d0931c9288660b78ab019b3f9a27 100644
--- a/cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-rococo/src/tests/send_xcm.rs
+++ b/cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-rococo/src/tests/send_xcm.rs
@@ -13,8 +13,6 @@
// See the License for the specific language governing permissions and
// limitations under the License.
-use rococo_system_emulated_network::rococo_emulated_chain::rococo_runtime::Dmp;
-
use crate::tests::*;
#[test]
@@ -40,8 +38,6 @@ fn send_xcm_from_rococo_relay_to_westend_asset_hub_should_fail_on_not_applicable
// Rococo Global Consensus
// Send XCM message from Relay Chain to Bridge Hub source Parachain
Rococo::execute_with(|| {
- Dmp::make_parachain_reachable(BridgeHubRococo::para_id());
-
assert_ok!(<Rococo as RococoPallet>::XcmPallet::send(
sudo_origin,
bx!(destination),
diff --git a/cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-westend/src/tests/send_xcm.rs b/cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-westend/src/tests/send_xcm.rs
index 60f8af2242f96c504d1836dbb7c6996ca0cb40f2..acce60b4fa76d50962ab5c06362c6666a0bf9ec7 100644
--- a/cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-westend/src/tests/send_xcm.rs
+++ b/cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-westend/src/tests/send_xcm.rs
@@ -13,8 +13,6 @@
// See the License for the specific language governing permissions and
// limitations under the License.
-use rococo_westend_system_emulated_network::westend_emulated_chain::westend_runtime::Dmp;
-
use crate::tests::*;
#[test]
@@ -40,8 +38,6 @@ fn send_xcm_from_westend_relay_to_rococo_asset_hub_should_fail_on_not_applicable
// Westend Global Consensus
// Send XCM message from Relay Chain to Bridge Hub source Parachain
Westend::execute_with(|| {
- Dmp::make_parachain_reachable(BridgeHubWestend::para_id());
-
assert_ok!(<Westend as WestendPallet>::XcmPallet::send(
sudo_origin,
bx!(destination),
diff --git a/cumulus/parachains/integration-tests/emulated/tests/collectives/collectives-westend/src/tests/fellowship_treasury.rs b/cumulus/parachains/integration-tests/emulated/tests/collectives/collectives-westend/src/tests/fellowship_treasury.rs
index ed7c9bafc607f0f39bad26ff6f939fb5c97ee243..8418e3da3bba0731a56c6b9dfbeb378d089adf28 100644
--- a/cumulus/parachains/integration-tests/emulated/tests/collectives/collectives-westend/src/tests/fellowship_treasury.rs
+++ b/cumulus/parachains/integration-tests/emulated/tests/collectives/collectives-westend/src/tests/fellowship_treasury.rs
@@ -20,7 +20,6 @@ use frame_support::{
};
use polkadot_runtime_common::impls::VersionedLocatableAsset;
use westend_runtime_constants::currency::UNITS;
-use westend_system_emulated_network::westend_emulated_chain::westend_runtime::Dmp;
use xcm_executor::traits::ConvertLocation;
// Fund Fellowship Treasury from Westend Treasury and spend from Fellowship Treasury.
@@ -58,8 +57,6 @@ fn fellowship_treasury_spend() {
treasury_balance * 2,
));
- Dmp::make_parachain_reachable(1000);
-
let native_asset = Location::here();
let asset_hub_location: Location = [Parachain(1000)].into();
let treasury_location: Location = (Parent, PalletInstance(37)).into();
diff --git a/cumulus/parachains/integration-tests/emulated/tests/coretime/coretime-rococo/src/tests/coretime_interface.rs b/cumulus/parachains/integration-tests/emulated/tests/coretime/coretime-rococo/src/tests/coretime_interface.rs
index 554025e1ecfed070a60bfc44c76b5a06ebc9818e..9915b1753ef6d2af0e5d4c49fa7b166ee68c959b 100644
--- a/cumulus/parachains/integration-tests/emulated/tests/coretime/coretime-rococo/src/tests/coretime_interface.rs
+++ b/cumulus/parachains/integration-tests/emulated/tests/coretime/coretime-rococo/src/tests/coretime_interface.rs
@@ -17,7 +17,6 @@ use crate::imports::*;
use frame_support::traits::OnInitialize;
use pallet_broker::{ConfigRecord, Configuration, CoreAssignment, CoreMask, ScheduleItem};
use rococo_runtime_constants::system_parachain::coretime::TIMESLICE_PERIOD;
-use rococo_system_emulated_network::rococo_emulated_chain::rococo_runtime::Dmp;
use sp_runtime::Perbill;
#[test]
@@ -35,10 +34,6 @@ fn transact_hardcoded_weights_are_sane() {
type CoretimeEvent = <CoretimeRococo as Chain>::RuntimeEvent;
type RelayEvent = <Rococo as Chain>::RuntimeEvent;
- Rococo::execute_with(|| {
- Dmp::make_parachain_reachable(CoretimeRococo::para_id());
- });
-
// Reserve a workload, configure broker and start sales.
CoretimeRococo::execute_with(|| {
// Hooks don't run in emulated tests - workaround as we need `on_initialize` to tick things
diff --git a/cumulus/parachains/integration-tests/emulated/tests/coretime/coretime-westend/src/tests/coretime_interface.rs b/cumulus/parachains/integration-tests/emulated/tests/coretime/coretime-westend/src/tests/coretime_interface.rs
index 900994b1afc1f62f40b55aad3f7a9c694c2219a9..00530f80b958d25f3af36ea17a63059d239831e7 100644
--- a/cumulus/parachains/integration-tests/emulated/tests/coretime/coretime-westend/src/tests/coretime_interface.rs
+++ b/cumulus/parachains/integration-tests/emulated/tests/coretime/coretime-westend/src/tests/coretime_interface.rs
@@ -18,7 +18,6 @@ use frame_support::traits::OnInitialize;
use pallet_broker::{ConfigRecord, Configuration, CoreAssignment, CoreMask, ScheduleItem};
use sp_runtime::Perbill;
use westend_runtime_constants::system_parachain::coretime::TIMESLICE_PERIOD;
-use westend_system_emulated_network::westend_emulated_chain::westend_runtime::Dmp;
#[test]
fn transact_hardcoded_weights_are_sane() {
@@ -35,10 +34,6 @@ fn transact_hardcoded_weights_are_sane() {
type CoretimeEvent = <CoretimeWestend as Chain>::RuntimeEvent;
type RelayEvent = <Westend as Chain>::RuntimeEvent;
- Westend::execute_with(|| {
- Dmp::make_parachain_reachable(CoretimeWestend::para_id());
- });
-
// Reserve a workload, configure broker and start sales.
CoretimeWestend::execute_with(|| {
// Hooks don't run in emulated tests - workaround as we need `on_initialize` to tick things
diff --git a/cumulus/parachains/integration-tests/emulated/tests/people/people-westend/src/tests/governance.rs b/cumulus/parachains/integration-tests/emulated/tests/people/people-westend/src/tests/governance.rs
index ea438f80552e283cc0f9c1501d2d204e8bb2e709..1ba787aaec52893ab78f29b9db60fcaffd03f6b7 100644
--- a/cumulus/parachains/integration-tests/emulated/tests/people/people-westend/src/tests/governance.rs
+++ b/cumulus/parachains/integration-tests/emulated/tests/people/people-westend/src/tests/governance.rs
@@ -20,9 +20,7 @@ use codec::Encode;
use frame_support::sp_runtime::traits::Dispatchable;
use parachains_common::AccountId;
use people_westend_runtime::people::IdentityInfo;
-use westend_runtime::{
- governance::pallet_custom_origins::Origin::GeneralAdmin as GeneralAdminOrigin, Dmp,
-};
+use westend_runtime::governance::pallet_custom_origins::Origin::GeneralAdmin as GeneralAdminOrigin;
use westend_system_emulated_network::people_westend_emulated_chain::people_westend_runtime;
use pallet_identity::Data;
@@ -41,8 +39,6 @@ fn relay_commands_add_registrar() {
type PeopleCall = <PeopleWestend as Chain>::RuntimeCall;
type PeopleRuntime = <PeopleWestend as Chain>::Runtime;
- Dmp::make_parachain_reachable(1004);
-
let add_registrar_call =
PeopleCall::Identity(pallet_identity::Call::<PeopleRuntime>::add_registrar {
account: registrar.into(),
@@ -106,8 +102,6 @@ fn relay_commands_add_registrar_wrong_origin() {
type PeopleCall = <PeopleWestend as Chain>::RuntimeCall;
type PeopleRuntime = <PeopleWestend as Chain>::Runtime;
- Dmp::make_parachain_reachable(1004);
-
let add_registrar_call =
PeopleCall::Identity(pallet_identity::Call::<PeopleRuntime>::add_registrar {
account: registrar.into(),
@@ -197,8 +191,6 @@ fn relay_commands_kill_identity() {
type RuntimeEvent = <Westend as Chain>::RuntimeEvent;
type PeopleRuntime = <PeopleWestend as Chain>::Runtime;
- Dmp::make_parachain_reachable(1004);
-
let kill_identity_call =
PeopleCall::Identity(pallet_identity::Call::<PeopleRuntime>::kill_identity {
target: people_westend_runtime::MultiAddress::Id(PeopleWestend::account_id_of(
@@ -261,8 +253,6 @@ fn relay_commands_kill_identity_wrong_origin() {
type RuntimeEvent = <Westend as Chain>::RuntimeEvent;
type PeopleRuntime = <PeopleWestend as Chain>::Runtime;
- Dmp::make_parachain_reachable(1004);
-
let kill_identity_call =
PeopleCall::Identity(pallet_identity::Call::<PeopleRuntime>::kill_identity {
target: people_westend_runtime::MultiAddress::Id(PeopleWestend::account_id_of(
@@ -313,8 +303,6 @@ fn relay_commands_add_remove_username_authority() {
type PeopleCall = <PeopleWestend as Chain>::RuntimeCall;
type PeopleRuntime = <PeopleWestend as Chain>::Runtime;
- Dmp::make_parachain_reachable(1004);
-
let add_username_authority =
PeopleCall::Identity(pallet_identity::Call::<PeopleRuntime>::add_username_authority {
authority: people_westend_runtime::MultiAddress::Id(people_westend_alice.clone()),
@@ -404,8 +392,6 @@ fn relay_commands_add_remove_username_authority() {
type PeopleCall = <PeopleWestend as Chain>::RuntimeCall;
type PeopleRuntime = <PeopleWestend as Chain>::Runtime;
- Dmp::make_parachain_reachable(1004);
-
let remove_username_authority = PeopleCall::Identity(pallet_identity::Call::<
PeopleRuntime,
>::remove_username_authority {
@@ -469,8 +455,6 @@ fn relay_commands_add_remove_username_authority_wrong_origin() {
type PeopleCall = <PeopleWestend as Chain>::RuntimeCall;
type PeopleRuntime = <PeopleWestend as Chain>::Runtime;
- Dmp::make_parachain_reachable(1004);
-
let add_username_authority = PeopleCall::Identity(pallet_identity::Call::<
PeopleRuntime,
>::add_username_authority {
@@ -519,8 +503,6 @@ fn relay_commands_add_remove_username_authority_wrong_origin() {
suffix: b"suffix1".into(),
});
- Dmp::make_parachain_reachable(1004);
-
let remove_authority_xcm_msg =
RuntimeCall::XcmPallet(pallet_xcm::Call::<Runtime>::send {
dest: bx!(VersionedLocation::from(Location::new(0, [Parachain(1004)]))),
diff --git a/cumulus/parachains/runtimes/assets/asset-hub-rococo/Cargo.toml b/cumulus/parachains/runtimes/assets/asset-hub-rococo/Cargo.toml
index f6e01096dc7ae9138908b2dcde82bd6c45170262..96c1e7b08c7b2a4239ddedf379d47cd65132ccbd 100644
--- a/cumulus/parachains/runtimes/assets/asset-hub-rococo/Cargo.toml
+++ b/cumulus/parachains/runtimes/assets/asset-hub-rococo/Cargo.toml
@@ -135,7 +135,6 @@ runtime-benchmarks = [
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
"xcm-runtime-apis/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
"cumulus-pallet-aura-ext/try-runtime",
diff --git a/cumulus/parachains/runtimes/assets/asset-hub-westend/Cargo.toml b/cumulus/parachains/runtimes/assets/asset-hub-westend/Cargo.toml
index 821a9479e729e1e5734a34b6dc8236b3f14e8193..8c3097ea5d70c949a40b644bdad64a498864f94d 100644
--- a/cumulus/parachains/runtimes/assets/asset-hub-westend/Cargo.toml
+++ b/cumulus/parachains/runtimes/assets/asset-hub-westend/Cargo.toml
@@ -140,7 +140,6 @@ runtime-benchmarks = [
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
"xcm-runtime-apis/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
"cumulus-pallet-aura-ext/try-runtime",
diff --git a/cumulus/parachains/runtimes/assets/common/Cargo.toml b/cumulus/parachains/runtimes/assets/common/Cargo.toml
index 2f7a1e2545ec2f7fcea7f46bc72274ee288b0b82..6bf4a5d34a9b039a97b6fb6cadb8d5097fc09e3f 100644
--- a/cumulus/parachains/runtimes/assets/common/Cargo.toml
+++ b/cumulus/parachains/runtimes/assets/common/Cargo.toml
@@ -59,5 +59,4 @@ runtime-benchmarks = [
"sp-runtime/runtime-benchmarks",
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
diff --git a/cumulus/parachains/runtimes/bridge-hubs/bridge-hub-rococo/Cargo.toml b/cumulus/parachains/runtimes/bridge-hubs/bridge-hub-rococo/Cargo.toml
index ced5f4064c2956f8bab4e0faa0838130adf89a33..d15881de3df904f7d1f5bbf0bc93f25f403a9d69 100644
--- a/cumulus/parachains/runtimes/bridge-hubs/bridge-hub-rococo/Cargo.toml
+++ b/cumulus/parachains/runtimes/bridge-hubs/bridge-hub-rococo/Cargo.toml
@@ -251,7 +251,6 @@ runtime-benchmarks = [
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
"xcm-runtime-apis/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
diff --git a/cumulus/parachains/runtimes/bridge-hubs/bridge-hub-westend/Cargo.toml b/cumulus/parachains/runtimes/bridge-hubs/bridge-hub-westend/Cargo.toml
index e978ae891d6e8db3ebce3c6b2fd858699432f4d6..243ff718f9e675c8c0406f9ccbd8607b302388bb 100644
--- a/cumulus/parachains/runtimes/bridge-hubs/bridge-hub-westend/Cargo.toml
+++ b/cumulus/parachains/runtimes/bridge-hubs/bridge-hub-westend/Cargo.toml
@@ -248,7 +248,6 @@ runtime-benchmarks = [
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
"xcm-runtime-apis/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
diff --git a/cumulus/parachains/runtimes/bridge-hubs/common/Cargo.toml b/cumulus/parachains/runtimes/bridge-hubs/common/Cargo.toml
index a619693ba7a983666a39858c7663938c3425ed86..53a8cd707e7546e28cf4a57fa888b9004025e2bd 100644
--- a/cumulus/parachains/runtimes/bridge-hubs/common/Cargo.toml
+++ b/cumulus/parachains/runtimes/bridge-hubs/common/Cargo.toml
@@ -39,5 +39,4 @@ runtime-benchmarks = [
"pallet-message-queue/runtime-benchmarks",
"snowbridge-core/runtime-benchmarks",
"sp-runtime/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
diff --git a/cumulus/parachains/runtimes/collectives/collectives-westend/Cargo.toml b/cumulus/parachains/runtimes/collectives/collectives-westend/Cargo.toml
index 12f1998d7758ccce891637cbbc60e897b0c1a2c1..e1ecfb723f6b5cbbc96bc6d869c09610e7d8105b 100644
--- a/cumulus/parachains/runtimes/collectives/collectives-westend/Cargo.toml
+++ b/cumulus/parachains/runtimes/collectives/collectives-westend/Cargo.toml
@@ -128,7 +128,6 @@ runtime-benchmarks = [
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
"xcm-runtime-apis/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
"cumulus-pallet-aura-ext/try-runtime",
diff --git a/cumulus/parachains/runtimes/contracts/contracts-rococo/Cargo.toml b/cumulus/parachains/runtimes/contracts/contracts-rococo/Cargo.toml
index 2c2e21bf985da5d34102b2e784ea7320ff6115a7..b123e846504fd9ec33fa282f6c45f1ffff41978f 100644
--- a/cumulus/parachains/runtimes/contracts/contracts-rococo/Cargo.toml
+++ b/cumulus/parachains/runtimes/contracts/contracts-rococo/Cargo.toml
@@ -164,7 +164,6 @@ runtime-benchmarks = [
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
"xcm-runtime-apis/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
diff --git a/cumulus/parachains/runtimes/coretime/coretime-rococo/Cargo.toml b/cumulus/parachains/runtimes/coretime/coretime-rococo/Cargo.toml
index cf9825d2824690c80f39f17721f55b1641f61cd5..e74de363d26ea09bfea73e19cefc0e3d28922393 100644
--- a/cumulus/parachains/runtimes/coretime/coretime-rococo/Cargo.toml
+++ b/cumulus/parachains/runtimes/coretime/coretime-rococo/Cargo.toml
@@ -168,7 +168,6 @@ runtime-benchmarks = [
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
"xcm-runtime-apis/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
diff --git a/cumulus/parachains/runtimes/coretime/coretime-westend/Cargo.toml b/cumulus/parachains/runtimes/coretime/coretime-westend/Cargo.toml
index d9bc7ea2b23550dd37600e10a300e6a159dac253..8220b67b002c77e94f57a71b5434f41b573f5d14 100644
--- a/cumulus/parachains/runtimes/coretime/coretime-westend/Cargo.toml
+++ b/cumulus/parachains/runtimes/coretime/coretime-westend/Cargo.toml
@@ -165,7 +165,6 @@ runtime-benchmarks = [
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
"xcm-runtime-apis/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
diff --git a/cumulus/parachains/runtimes/glutton/glutton-westend/Cargo.toml b/cumulus/parachains/runtimes/glutton/glutton-westend/Cargo.toml
index ed9b51b5ee0c596b9fd94c8ad1abc4ca4a1a3ad4..4196f16df37dbf9dd2b117b092365ab763f27610 100644
--- a/cumulus/parachains/runtimes/glutton/glutton-westend/Cargo.toml
+++ b/cumulus/parachains/runtimes/glutton/glutton-westend/Cargo.toml
@@ -70,7 +70,6 @@ runtime-benchmarks = [
"sp-runtime/runtime-benchmarks",
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
std = [
"codec/std",
diff --git a/cumulus/parachains/runtimes/people/people-rococo/Cargo.toml b/cumulus/parachains/runtimes/people/people-rococo/Cargo.toml
index 20c531a3abc7b822745f9e7ed90c74a11649bd75..9bf627af2abe705641ec6249a3027b673ebfd9d7 100644
--- a/cumulus/parachains/runtimes/people/people-rococo/Cargo.toml
+++ b/cumulus/parachains/runtimes/people/people-rococo/Cargo.toml
@@ -165,7 +165,6 @@ runtime-benchmarks = [
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
"xcm-runtime-apis/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
diff --git a/cumulus/parachains/runtimes/people/people-westend/Cargo.toml b/cumulus/parachains/runtimes/people/people-westend/Cargo.toml
index 1766180be294f5dad898c240210f5d4724061569..85e26c9688e0a3c7eb6e8d42b94738e8b1c3db42 100644
--- a/cumulus/parachains/runtimes/people/people-westend/Cargo.toml
+++ b/cumulus/parachains/runtimes/people/people-westend/Cargo.toml
@@ -165,7 +165,6 @@ runtime-benchmarks = [
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
"xcm-runtime-apis/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
diff --git a/cumulus/parachains/runtimes/testing/penpal/Cargo.toml b/cumulus/parachains/runtimes/testing/penpal/Cargo.toml
index ed3a7cdf936ca69be65221cd3e78571a672b6d39..91606ffa19e4e14cc5eae21e29b99e6076afe4f3 100644
--- a/cumulus/parachains/runtimes/testing/penpal/Cargo.toml
+++ b/cumulus/parachains/runtimes/testing/penpal/Cargo.toml
@@ -169,7 +169,6 @@ runtime-benchmarks = [
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
"xcm-runtime-apis/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
diff --git a/cumulus/parachains/runtimes/testing/rococo-parachain/Cargo.toml b/cumulus/parachains/runtimes/testing/rococo-parachain/Cargo.toml
index 725896f10dded1f1fb273523d6f7a8b624cf36db..692973989b15b65cc3a2d5c8aff96b5e164f72b9 100644
--- a/cumulus/parachains/runtimes/testing/rococo-parachain/Cargo.toml
+++ b/cumulus/parachains/runtimes/testing/rococo-parachain/Cargo.toml
@@ -128,7 +128,6 @@ runtime-benchmarks = [
"sp-runtime/runtime-benchmarks",
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
# A feature that should be enabled when the runtime should be built for on-chain
diff --git a/cumulus/polkadot-parachain/Cargo.toml b/cumulus/polkadot-parachain/Cargo.toml
index fd6fa3a8c22fd9e5e784ce8c3d1ab966cfed7746..394b730e1c6da3381961fdb51d8fc2393240f5e4 100644
--- a/cumulus/polkadot-parachain/Cargo.toml
+++ b/cumulus/polkadot-parachain/Cargo.toml
@@ -82,7 +82,6 @@ runtime-benchmarks = [
"people-rococo-runtime/runtime-benchmarks",
"people-westend-runtime/runtime-benchmarks",
"rococo-parachain-runtime/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
"polkadot-omni-node-lib/try-runtime",
diff --git a/cumulus/primitives/core/Cargo.toml b/cumulus/primitives/core/Cargo.toml
index 52f523698d0a3959742d48f0b238b34849d15c8c..2c3e38e17598792510d71fc7e6f9a94144a33e5c 100644
--- a/cumulus/primitives/core/Cargo.toml
+++ b/cumulus/primitives/core/Cargo.toml
@@ -37,5 +37,4 @@ runtime-benchmarks = [
"polkadot-parachain-primitives/runtime-benchmarks",
"polkadot-primitives/runtime-benchmarks",
"sp-runtime/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
diff --git a/cumulus/primitives/utility/Cargo.toml b/cumulus/primitives/utility/Cargo.toml
index b422f1f50e63ba9bcd3a1c71ae332f6ec136e9c8..afb5be632954158dcc2d4deeacaf99b6d036b255 100644
--- a/cumulus/primitives/utility/Cargo.toml
+++ b/cumulus/primitives/utility/Cargo.toml
@@ -44,5 +44,4 @@ runtime-benchmarks = [
"sp-runtime/runtime-benchmarks",
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
diff --git a/polkadot/node/service/Cargo.toml b/polkadot/node/service/Cargo.toml
index 1bbe70fe5bbab3157c06b08f8cb676e31f7b3a13..8d60b7ce2a08b3a7126d09e138a14858a07ffbfe 100644
--- a/polkadot/node/service/Cargo.toml
+++ b/polkadot/node/service/Cargo.toml
@@ -253,7 +253,6 @@ runtime-benchmarks = [
"sp-runtime/runtime-benchmarks",
"westend-runtime?/runtime-benchmarks",
"xcm-runtime-apis/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
"frame-system/try-runtime",
diff --git a/polkadot/parachain/src/primitives.rs b/polkadot/parachain/src/primitives.rs
index 1f2f9e2e9cdc7068a623db6469c935833dbedab0..c5757928c3fc2f86fdd6ddb779856f0f39db70f7 100644
--- a/polkadot/parachain/src/primitives.rs
+++ b/polkadot/parachain/src/primitives.rs
@@ -57,8 +57,6 @@ impl HeadData {
}
}
-impl codec::EncodeLike<HeadData> for alloc::vec::Vec<u8> {}
-
/// Parachain validation code.
#[derive(
PartialEq,
@@ -156,9 +154,6 @@ pub struct BlockData(#[cfg_attr(feature = "std", serde(with = "bytes"))] pub Vec
#[cfg_attr(feature = "std", derive(derive_more::Display))]
pub struct Id(u32);
-impl codec::EncodeLike<u32> for Id {}
-impl codec::EncodeLike<Id> for u32 {}
-
impl TypeId for Id {
const TYPE_ID: [u8; 4] = *b"para";
}
diff --git a/polkadot/runtime/common/Cargo.toml b/polkadot/runtime/common/Cargo.toml
index 62b4f2dc0f3549147d72f0b2de17cab045d209e6..cb1bfa00460819943e47b03d8fe15e1e2cc889a6 100644
--- a/polkadot/runtime/common/Cargo.toml
+++ b/polkadot/runtime/common/Cargo.toml
@@ -135,7 +135,6 @@ runtime-benchmarks = [
"sp-staking/runtime-benchmarks",
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
"frame-election-provider-support/try-runtime",
diff --git a/polkadot/runtime/common/src/identity_migrator.rs b/polkadot/runtime/common/src/identity_migrator.rs
index e3835b692526e5cac1a8a541f91a85675e4fa5f1..126c886280e6ed713d81c93f5afedbb69b03a1b5 100644
--- a/polkadot/runtime/common/src/identity_migrator.rs
+++ b/polkadot/runtime/common/src/identity_migrator.rs
@@ -160,22 +160,12 @@ pub trait OnReapIdentity<AccountId> {
/// - `bytes`: The byte size of `IdentityInfo`.
/// - `subs`: The number of sub-accounts they had.
fn on_reap_identity(who: &AccountId, bytes: u32, subs: u32) -> DispatchResult;
-
- /// Ensure that identity reaping will be succesful in benchmarking.
- ///
- /// Should setup the state in a way that the same call ot `[Self::on_reap_identity]` will be
- /// successful.
- #[cfg(feature = "runtime-benchmarks")]
- fn ensure_successful_identity_reaping(who: &AccountId, bytes: u32, subs: u32);
}
impl<AccountId> OnReapIdentity<AccountId> for () {
fn on_reap_identity(_who: &AccountId, _bytes: u32, _subs: u32) -> DispatchResult {
Ok(())
}
-
- #[cfg(feature = "runtime-benchmarks")]
- fn ensure_successful_identity_reaping(_: &AccountId, _: u32, _: u32) {}
}
#[cfg(feature = "runtime-benchmarks")]
@@ -229,12 +219,6 @@ mod benchmarks {
}
Identity::<T>::set_subs(target_origin.clone(), subs.clone())?;
- T::ReapIdentityHandler::ensure_successful_identity_reaping(
- &target,
- info.encoded_size() as u32,
- subs.len() as u32,
- );
-
// add registrars and provide judgements
let registrar_origin = T::RegistrarOrigin::try_successful_origin()
.expect("RegistrarOrigin has no successful origin required for the benchmark");
diff --git a/polkadot/runtime/common/src/paras_sudo_wrapper.rs b/polkadot/runtime/common/src/paras_sudo_wrapper.rs
index 4309931fe815eb6286de4522aebd47be642837f8..af93c70b4783f717a2adddd0f0f34365ae27d954 100644
--- a/polkadot/runtime/common/src/paras_sudo_wrapper.rs
+++ b/polkadot/runtime/common/src/paras_sudo_wrapper.rs
@@ -48,8 +48,6 @@ pub mod pallet {
/// A DMP message couldn't be sent because it exceeds the maximum size allowed for a
/// downward message.
ExceedsMaxMessageSize,
- /// A DMP message couldn't be sent because the destination is unreachable.
- Unroutable,
/// Could not schedule para cleanup.
CouldntCleanup,
/// Not a parathread (on-demand parachain).
@@ -154,7 +152,6 @@ pub mod pallet {
{
dmp::QueueDownwardMessageError::ExceedsMaxMessageSize =>
Error::<T>::ExceedsMaxMessageSize.into(),
- dmp::QueueDownwardMessageError::Unroutable => Error::<T>::Unroutable.into(),
})
}
diff --git a/polkadot/runtime/common/src/xcm_sender.rs b/polkadot/runtime/common/src/xcm_sender.rs
index 32ea4fdd2f27090acc1ca7ccf890d5c6fdcb3624..7ff7f69faf148e9dde971f62d33c7e7a74609bf7 100644
--- a/polkadot/runtime/common/src/xcm_sender.rs
+++ b/polkadot/runtime/common/src/xcm_sender.rs
@@ -138,13 +138,6 @@ where
.map(|()| hash)
.map_err(|_| SendError::Transport(&"Error placing into DMP queue"))
}
-
- #[cfg(feature = "runtime-benchmarks")]
- fn ensure_successful_delivery(location: Option<Location>) {
- if let Some((0, [Parachain(id)])) = location.as_ref().map(|l| l.unpack()) {
- dmp::Pallet::<T>::make_parachain_reachable(*id);
- }
- }
}
impl<T: dmp::Config, W, P> InspectMessageQueues for ChildParachainRouter<T, W, P> {
@@ -197,7 +190,7 @@ impl<
ExistentialDeposit: Get<Option<Asset>>,
PriceForDelivery: PriceForMessageDelivery<Id = ParaId>,
Parachain: Get<ParaId>,
- ToParachainHelper: polkadot_runtime_parachains::EnsureForParachain,
+ ToParachainHelper: EnsureForParachain,
> xcm_builder::EnsureDelivery
for ToParachainDeliveryHelper<
XcmConfig,
@@ -226,9 +219,6 @@ impl<
return (None, None)
}
- // allow more initialization for target parachain
- ToParachainHelper::ensure(Parachain::get());
-
let mut fees_mode = None;
if !XcmConfig::FeeManager::is_waived(Some(origin_ref), fee_reason) {
// if not waived, we need to set up accounts for paying and receiving fees
@@ -248,6 +238,9 @@ impl<
XcmConfig::AssetTransactor::deposit_asset(&fee, &origin_ref, None).unwrap();
}
+ // allow more initialization for target parachain
+ ToParachainHelper::ensure(Parachain::get());
+
// expected worst case - direct withdraw
fees_mode = Some(FeesMode { jit_withdraw: true });
}
@@ -255,6 +248,18 @@ impl<
}
}
+/// Ensure more initialization for `ParaId`. (e.g. open HRMP channels, ...)
+#[cfg(feature = "runtime-benchmarks")]
+pub trait EnsureForParachain {
+ fn ensure(para_id: ParaId);
+}
+#[cfg(feature = "runtime-benchmarks")]
+impl EnsureForParachain for () {
+ fn ensure(_: ParaId) {
+ // doing nothing
+ }
+}
+
#[cfg(test)]
mod tests {
use super::*;
@@ -344,8 +349,6 @@ mod tests {
c.max_downward_message_size = u32::MAX;
});
- dmp::Pallet::<crate::integration_tests::Test>::make_parachain_reachable(5555);
-
// Check that the good message is validated:
assert_ok!(<Router as SendXcm>::validate(
&mut Some(dest.into()),
diff --git a/polkadot/runtime/parachains/Cargo.toml b/polkadot/runtime/parachains/Cargo.toml
index dd0788170b6fe5106219e6775255726e71922e25..31e786ae6baa3ed557efc36d88953843bf1680b7 100644
--- a/polkadot/runtime/parachains/Cargo.toml
+++ b/polkadot/runtime/parachains/Cargo.toml
@@ -135,7 +135,6 @@ runtime-benchmarks = [
"sp-std",
"static_assertions",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
"frame-support/try-runtime",
diff --git a/polkadot/runtime/parachains/src/coretime/benchmarking.rs b/polkadot/runtime/parachains/src/coretime/benchmarking.rs
index 49e3d8a88c0158f09aa3a4d8e98a969ca907be44..6d593f1954ff1500fd59ae5d178e038dd38e01b5 100644
--- a/polkadot/runtime/parachains/src/coretime/benchmarking.rs
+++ b/polkadot/runtime/parachains/src/coretime/benchmarking.rs
@@ -43,8 +43,6 @@ mod benchmarks {
.unwrap();
on_demand::Revenue::<T>::put(rev);
- crate::paras::Heads::<T>::insert(ParaId::from(T::BrokerId::get()), vec![1, 2, 3]);
-
<T as on_demand::Config>::Currency::make_free_balance_be(
&<on_demand::Pallet<T>>::account_id(),
minimum_balance * (mhr * (mhr + 1)).into(),
diff --git a/polkadot/runtime/parachains/src/dmp.rs b/polkadot/runtime/parachains/src/dmp.rs
index 3c9cf8004186527bc999f1eccc71325b81bf5af1..03580e11b8e9cbe7dce4c9e375741ec3677bd9fe 100644
--- a/polkadot/runtime/parachains/src/dmp.rs
+++ b/polkadot/runtime/parachains/src/dmp.rs
@@ -44,7 +44,7 @@
use crate::{
configuration::{self, HostConfiguration},
- initializer, paras, FeeTracker,
+ initializer, FeeTracker,
};
use alloc::vec::Vec;
use core::fmt;
@@ -72,15 +72,12 @@ const MESSAGE_SIZE_FEE_BASE: FixedU128 = FixedU128::from_rational(1, 1000); // 0
pub enum QueueDownwardMessageError {
/// The message being sent exceeds the configured max message size.
ExceedsMaxMessageSize,
- /// The destination is unknown.
- Unroutable,
}
impl From<QueueDownwardMessageError> for SendError {
fn from(err: QueueDownwardMessageError) -> Self {
match err {
QueueDownwardMessageError::ExceedsMaxMessageSize => SendError::ExceedsMaxMessageSize,
- QueueDownwardMessageError::Unroutable => SendError::Unroutable,
}
}
}
@@ -119,7 +116,7 @@ pub mod pallet {
pub struct Pallet<T>(_);
#[pallet::config]
- pub trait Config: frame_system::Config + configuration::Config + paras::Config {}
+ pub trait Config: frame_system::Config + configuration::Config {}
/// The downward messages addressed for a certain para.
#[pallet::storage]
@@ -203,11 +200,6 @@ impl<T: Config> Pallet<T> {
return Err(QueueDownwardMessageError::ExceedsMaxMessageSize)
}
- // If the head exists, we assume the parachain is legit and exists.
- if !paras::Heads::<T>::contains_key(para) {
- return Err(QueueDownwardMessageError::Unroutable)
- }
-
Ok(())
}
@@ -225,7 +217,14 @@ impl<T: Config> Pallet<T> {
msg: DownwardMessage,
) -> Result<(), QueueDownwardMessageError> {
let serialized_len = msg.len() as u32;
- Self::can_queue_downward_message(config, ¶, &msg)?;
+ if serialized_len > config.max_downward_message_size {
+ return Err(QueueDownwardMessageError::ExceedsMaxMessageSize)
+ }
+
+ // Hard limit on Queue size
+ if Self::dmq_length(para) > Self::dmq_max_length(config.max_downward_message_size) {
+ return Err(QueueDownwardMessageError::ExceedsMaxMessageSize)
+ }
let inbound =
InboundDownwardMessage { msg, sent_at: frame_system::Pallet::<T>::block_number() };
@@ -337,15 +336,6 @@ impl<T: Config> Pallet<T> {
) -> Vec<InboundDownwardMessage<BlockNumberFor<T>>> {
DownwardMessageQueues::<T>::get(&recipient)
}
-
- /// Make the parachain reachable for downward messages.
- ///
- /// Only useable in benchmarks or tests.
- #[cfg(any(feature = "runtime-benchmarks", feature = "std"))]
- pub fn make_parachain_reachable(para: impl Into<ParaId>) {
- let para = para.into();
- crate::paras::Heads::<T>::insert(para, para.encode());
- }
}
impl<T: Config> FeeTracker for Pallet<T> {
@@ -369,10 +359,3 @@ impl<T: Config> FeeTracker for Pallet<T> {
})
}
}
-
-#[cfg(feature = "runtime-benchmarks")]
-impl<T: Config> crate::EnsureForParachain for Pallet<T> {
- fn ensure(para: ParaId) {
- Self::make_parachain_reachable(para);
- }
-}
diff --git a/polkadot/runtime/parachains/src/dmp/tests.rs b/polkadot/runtime/parachains/src/dmp/tests.rs
index 617c9488bd2a22c713f188ff89993253374848f4..de15159581252aa29775addda7797d84abc4d115 100644
--- a/polkadot/runtime/parachains/src/dmp/tests.rs
+++ b/polkadot/runtime/parachains/src/dmp/tests.rs
@@ -61,12 +61,6 @@ fn queue_downward_message(
Dmp::queue_downward_message(&configuration::ActiveConfig::<Test>::get(), para_id, msg)
}
-fn register_paras(paras: &[ParaId]) {
- paras.iter().for_each(|p| {
- Dmp::make_parachain_reachable(*p);
- });
-}
-
#[test]
fn clean_dmp_works() {
let a = ParaId::from(1312);
@@ -74,8 +68,6 @@ fn clean_dmp_works() {
let c = ParaId::from(123);
new_test_ext(default_genesis_config()).execute_with(|| {
- register_paras(&[a, b, c]);
-
// enqueue downward messages to A, B and C.
queue_downward_message(a, vec![1, 2, 3]).unwrap();
queue_downward_message(b, vec![4, 5, 6]).unwrap();
@@ -97,8 +89,6 @@ fn dmq_length_and_head_updated_properly() {
let b = ParaId::from(228);
new_test_ext(default_genesis_config()).execute_with(|| {
- register_paras(&[a, b]);
-
assert_eq!(Dmp::dmq_length(a), 0);
assert_eq!(Dmp::dmq_length(b), 0);
@@ -111,30 +101,11 @@ fn dmq_length_and_head_updated_properly() {
});
}
-#[test]
-fn dmq_fail_if_para_does_not_exist() {
- let a = ParaId::from(1312);
-
- new_test_ext(default_genesis_config()).execute_with(|| {
- assert_eq!(Dmp::dmq_length(a), 0);
-
- assert!(matches!(
- queue_downward_message(a, vec![1, 2, 3]),
- Err(QueueDownwardMessageError::Unroutable)
- ));
-
- assert_eq!(Dmp::dmq_length(a), 0);
- assert!(Dmp::dmq_mqc_head(a).is_zero());
- });
-}
-
#[test]
fn dmp_mqc_head_fixture() {
let a = ParaId::from(2000);
new_test_ext(default_genesis_config()).execute_with(|| {
- register_paras(&[a]);
-
run_to_block(2, None);
assert!(Dmp::dmq_mqc_head(a).is_zero());
queue_downward_message(a, vec![1, 2, 3]).unwrap();
@@ -154,8 +125,6 @@ fn check_processed_downward_messages() {
let a = ParaId::from(1312);
new_test_ext(default_genesis_config()).execute_with(|| {
- register_paras(&[a]);
-
let block_number = System::block_number();
// processed_downward_messages=0 is allowed when the DMQ is empty.
@@ -181,8 +150,6 @@ fn check_processed_downward_messages_advancement_rule() {
let a = ParaId::from(1312);
new_test_ext(default_genesis_config()).execute_with(|| {
- register_paras(&[a]);
-
let block_number = System::block_number();
run_to_block(block_number + 1, None);
@@ -203,8 +170,6 @@ fn dmq_pruning() {
let a = ParaId::from(1312);
new_test_ext(default_genesis_config()).execute_with(|| {
- register_paras(&[a]);
-
assert_eq!(Dmp::dmq_length(a), 0);
queue_downward_message(a, vec![1, 2, 3]).unwrap();
@@ -229,8 +194,6 @@ fn queue_downward_message_critical() {
genesis.configuration.config.max_downward_message_size = 7;
new_test_ext(genesis).execute_with(|| {
- register_paras(&[a]);
-
let smol = [0; 3].to_vec();
let big = [0; 8].to_vec();
@@ -252,8 +215,6 @@ fn verify_dmq_mqc_head_is_externally_accessible() {
let a = ParaId::from(2020);
new_test_ext(default_genesis_config()).execute_with(|| {
- register_paras(&[a]);
-
let head = sp_io::storage::get(&well_known_keys::dmq_mqc_head(a));
assert_eq!(head, None);
@@ -274,12 +235,9 @@ fn verify_dmq_mqc_head_is_externally_accessible() {
#[test]
fn verify_fee_increase_and_decrease() {
let a = ParaId::from(123);
-
let mut genesis = default_genesis_config();
genesis.configuration.config.max_downward_message_size = 16777216;
new_test_ext(genesis).execute_with(|| {
- register_paras(&[a]);
-
let initial = InitialFactor::get();
assert_eq!(DeliveryFeeFactor::<Test>::get(a), initial);
@@ -329,8 +287,6 @@ fn verify_fee_factor_reaches_high_value() {
let mut genesis = default_genesis_config();
genesis.configuration.config.max_downward_message_size = 51200;
new_test_ext(genesis).execute_with(|| {
- register_paras(&[a]);
-
let max_messages =
Dmp::dmq_max_length(ActiveConfig::<Test>::get().max_downward_message_size);
let mut total_fee_factor = FixedU128::from_float(1.0);
diff --git a/polkadot/runtime/parachains/src/lib.rs b/polkadot/runtime/parachains/src/lib.rs
index b1ff5419470e18c16546397fb2ebbc2d5440f309..828c0b9bcef21410d15107191c1255d4439a1bc5 100644
--- a/polkadot/runtime/parachains/src/lib.rs
+++ b/polkadot/runtime/parachains/src/lib.rs
@@ -114,19 +114,3 @@ pub fn schedule_code_upgrade<T: paras::Config>(
pub fn set_current_head<T: paras::Config>(id: ParaId, new_head: HeadData) {
paras::Pallet::<T>::set_current_head(id, new_head)
}
-
-/// Ensure more initialization for `ParaId` when benchmarking. (e.g. open HRMP channels, ...)
-#[cfg(feature = "runtime-benchmarks")]
-pub trait EnsureForParachain {
- fn ensure(para_id: ParaId);
-}
-
-#[cfg(feature = "runtime-benchmarks")]
-#[impl_trait_for_tuples::impl_for_tuples(30)]
-impl EnsureForParachain for Tuple {
- fn ensure(para: ParaId) {
- for_tuples!( #(
- Tuple::ensure(para);
- )* );
- }
-}
diff --git a/polkadot/runtime/rococo/Cargo.toml b/polkadot/runtime/rococo/Cargo.toml
index d1202fc90935a8417ee5296884a41a3bea34d3bb..e321d973ba7de03b17cbe7e07ba78db6cbc8e75c 100644
--- a/polkadot/runtime/rococo/Cargo.toml
+++ b/polkadot/runtime/rococo/Cargo.toml
@@ -269,7 +269,6 @@ runtime-benchmarks = [
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
"xcm-runtime-apis/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
"frame-executive/try-runtime",
diff --git a/polkadot/runtime/rococo/src/impls.rs b/polkadot/runtime/rococo/src/impls.rs
index a5cb2eddfa0d7d23c649cab899680d0d9fb15339..7d7e9fa9f06c5f0ee3df502ca8cf0c4feee90c28 100644
--- a/polkadot/runtime/rococo/src/impls.rs
+++ b/polkadot/runtime/rococo/src/impls.rs
@@ -176,9 +176,4 @@ where
)?;
Ok(())
}
-
- #[cfg(feature = "runtime-benchmarks")]
- fn ensure_successful_identity_reaping(_: &AccountId, _: u32, _: u32) {
- crate::Dmp::make_parachain_reachable(1004);
- }
}
diff --git a/polkadot/runtime/rococo/src/lib.rs b/polkadot/runtime/rococo/src/lib.rs
index 0bf014a0c91feee1ea95219fce508a8fcd7edaa8..4a9d429a01ccb304048297521c61c297db2db170 100644
--- a/polkadot/runtime/rococo/src/lib.rs
+++ b/polkadot/runtime/rococo/src/lib.rs
@@ -2469,14 +2469,14 @@ sp_api::impl_runtime_apis! {
ExistentialDepositAsset,
xcm_config::PriceForChildParachainDelivery,
AssetHubParaId,
- Dmp,
+ (),
>,
polkadot_runtime_common::xcm_sender::ToParachainDeliveryHelper<
XcmConfig,
ExistentialDepositAsset,
xcm_config::PriceForChildParachainDelivery,
RandomParaId,
- Dmp,
+ (),
>
);
@@ -2535,7 +2535,7 @@ sp_api::impl_runtime_apis! {
ExistentialDepositAsset,
xcm_config::PriceForChildParachainDelivery,
AssetHubParaId,
- Dmp,
+ (),
>;
fn valid_destination() -> Result<Location, BenchmarkError> {
Ok(AssetHub::get())
diff --git a/polkadot/runtime/test-runtime/Cargo.toml b/polkadot/runtime/test-runtime/Cargo.toml
index 7775fee4ed0911eb77623a6ad80284aca0502b3e..f555fab5a71e82aab2cf709e180f215e3a67cbb9 100644
--- a/polkadot/runtime/test-runtime/Cargo.toml
+++ b/polkadot/runtime/test-runtime/Cargo.toml
@@ -153,5 +153,4 @@ runtime-benchmarks = [
"sp-staking/runtime-benchmarks",
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
diff --git a/polkadot/runtime/westend/Cargo.toml b/polkadot/runtime/westend/Cargo.toml
index 600d2c7fa993db13ab71b094185dc39e228ac4ef..cf4be6ebe441765ee315d254137436b8f5cac74f 100644
--- a/polkadot/runtime/westend/Cargo.toml
+++ b/polkadot/runtime/westend/Cargo.toml
@@ -291,7 +291,6 @@ runtime-benchmarks = [
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
"xcm-runtime-apis/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
"frame-election-provider-support/try-runtime",
diff --git a/polkadot/runtime/westend/src/impls.rs b/polkadot/runtime/westend/src/impls.rs
index 0e0d345a0ed44a35f6490f1586ef6e6721df0aa6..8cb597cbaa95e4cce65ff3fe8a16a84dff153384 100644
--- a/polkadot/runtime/westend/src/impls.rs
+++ b/polkadot/runtime/westend/src/impls.rs
@@ -176,9 +176,4 @@ where
)?;
Ok(())
}
-
- #[cfg(feature = "runtime-benchmarks")]
- fn ensure_successful_identity_reaping(_: &AccountId, _: u32, _: u32) {
- crate::Dmp::make_parachain_reachable(1004);
- }
}
diff --git a/polkadot/runtime/westend/src/lib.rs b/polkadot/runtime/westend/src/lib.rs
index 516b085d2914491ba5b82b7739aee82e034c5a6d..25e7e653d220abe3dd0d06e8cf0198b35a88c3ae 100644
--- a/polkadot/runtime/westend/src/lib.rs
+++ b/polkadot/runtime/westend/src/lib.rs
@@ -2636,14 +2636,14 @@ sp_api::impl_runtime_apis! {
ExistentialDepositAsset,
xcm_config::PriceForChildParachainDelivery,
AssetHubParaId,
- Dmp,
+ (),
>,
polkadot_runtime_common::xcm_sender::ToParachainDeliveryHelper<
xcm_config::XcmConfig,
ExistentialDepositAsset,
xcm_config::PriceForChildParachainDelivery,
RandomParaId,
- Dmp,
+ (),
>
);
@@ -2709,7 +2709,7 @@ sp_api::impl_runtime_apis! {
ExistentialDepositAsset,
xcm_config::PriceForChildParachainDelivery,
AssetHubParaId,
- Dmp,
+ (),
>;
fn valid_destination() -> Result<Location, BenchmarkError> {
Ok(AssetHub::get())
diff --git a/polkadot/xcm/Cargo.toml b/polkadot/xcm/Cargo.toml
index 524b3f3a18dbe70daf69d02e212e87ecba2adeac..87910cd4a456b1b47a68153ffccafe2a42ba7697 100644
--- a/polkadot/xcm/Cargo.toml
+++ b/polkadot/xcm/Cargo.toml
@@ -50,7 +50,3 @@ json-schema = [
"dep:schemars",
"sp-weights/json-schema",
]
-runtime-benchmarks = [
- "frame-support/runtime-benchmarks",
- "sp-runtime/runtime-benchmarks",
-]
diff --git a/polkadot/xcm/pallet-xcm-benchmarks/Cargo.toml b/polkadot/xcm/pallet-xcm-benchmarks/Cargo.toml
index 9e2a33607d9b7800335c1613c7ea8dadf1e8400c..72e96aa3e1d0848e3bc5316661c8487bb8caa81d 100644
--- a/polkadot/xcm/pallet-xcm-benchmarks/Cargo.toml
+++ b/polkadot/xcm/pallet-xcm-benchmarks/Cargo.toml
@@ -57,5 +57,4 @@ runtime-benchmarks = [
"sp-runtime/runtime-benchmarks",
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
diff --git a/polkadot/xcm/pallet-xcm-benchmarks/src/fungible/benchmarking.rs b/polkadot/xcm/pallet-xcm-benchmarks/src/fungible/benchmarking.rs
index 4428076aa07715083bf62876d5c899d07228d249..303ff9493f71b6a6411427146d93774209850e5c 100644
--- a/polkadot/xcm/pallet-xcm-benchmarks/src/fungible/benchmarking.rs
+++ b/polkadot/xcm/pallet-xcm-benchmarks/src/fungible/benchmarking.rs
@@ -231,13 +231,6 @@ benchmarks_instance_pallet! {
let dest_account = T::AccountIdConverter::convert_location(&dest_location).unwrap();
assert!(T::TransactAsset::balance(&dest_account).is_zero());
- // Ensure that origin can send to destination (e.g. setup delivery fees, ensure router setup, ...)
- let (_, _) = T::DeliveryHelper::ensure_successful_delivery(
- &Default::default(),
- &dest_location,
- FeeReason::ChargeFees,
- );
-
let mut executor = new_executor::<T>(Default::default());
executor.set_holding(holding.into());
let instruction = Instruction::<XcmCallOf<T>>::DepositAsset {
@@ -264,13 +257,6 @@ benchmarks_instance_pallet! {
let dest_account = T::AccountIdConverter::convert_location(&dest_location).unwrap();
assert!(T::TransactAsset::balance(&dest_account).is_zero());
- // Ensure that origin can send to destination (e.g. setup delivery fees, ensure router setup, ...)
- let (_, _) = T::DeliveryHelper::ensure_successful_delivery(
- &Default::default(),
- &dest_location,
- FeeReason::ChargeFees,
- );
-
let mut executor = new_executor::<T>(Default::default());
executor.set_holding(holding.into());
let instruction = Instruction::<XcmCallOf<T>>::DepositReserveAsset {
@@ -295,20 +281,12 @@ benchmarks_instance_pallet! {
// Checked account starts at zero
assert!(T::CheckedAccount::get().map_or(true, |(c, _)| T::TransactAsset::balance(&c).is_zero()));
- let dest_location = T::valid_destination()?;
-
- // Ensure that origin can send to destination (e.g. setup delivery fees, ensure router setup, ...)
- let (_, _) = T::DeliveryHelper::ensure_successful_delivery(
- &Default::default(),
- &dest_location,
- FeeReason::ChargeFees,
- );
let mut executor = new_executor::<T>(Default::default());
executor.set_holding(holding.into());
let instruction = Instruction::<XcmCallOf<T>>::InitiateTeleport {
assets: asset.into(),
- dest: dest_location,
+ dest: T::valid_destination()?,
xcm: Xcm::new(),
};
let xcm = Xcm(vec![instruction]);
@@ -325,15 +303,6 @@ benchmarks_instance_pallet! {
let (sender_account, sender_location) = account_and_location::<T>(1);
let asset = T::get_asset();
let mut holding = T::worst_case_holding(1);
- let dest_location = T::valid_destination()?;
-
- // Ensure that origin can send to destination (e.g. setup delivery fees, ensure router setup, ...)
- let (_, _) = T::DeliveryHelper::ensure_successful_delivery(
- &sender_location,
- &dest_location,
- FeeReason::ChargeFees,
- );
-
let sender_account_balance_before = T::TransactAsset::balance(&sender_account);
// Add our asset to the holding.
@@ -342,7 +311,7 @@ benchmarks_instance_pallet! {
let mut executor = new_executor::<T>(sender_location);
executor.set_holding(holding.into());
let instruction = Instruction::<XcmCallOf<T>>::InitiateTransfer {
- destination: dest_location,
+ destination: T::valid_destination()?,
// ReserveDeposit is the most expensive filter.
remote_fees: Some(AssetTransferFilter::ReserveDeposit(asset.clone().into())),
// It's more expensive if we reanchor the origin.
diff --git a/polkadot/xcm/pallet-xcm-benchmarks/src/generic/benchmarking.rs b/polkadot/xcm/pallet-xcm-benchmarks/src/generic/benchmarking.rs
index 4d15dcfb936f0d0dd792908edf0310b3744f065b..ca2715555f7ca17a1d45378980d8b6a084720382 100644
--- a/polkadot/xcm/pallet-xcm-benchmarks/src/generic/benchmarking.rs
+++ b/polkadot/xcm/pallet-xcm-benchmarks/src/generic/benchmarking.rs
@@ -344,7 +344,6 @@ benchmarks! {
let mut executor = new_executor::<T>(origin.clone());
let instruction = Instruction::SubscribeVersion { query_id, max_response_weight };
let xcm = Xcm(vec![instruction]);
- T::DeliveryHelper::ensure_successful_delivery(&origin, &origin, FeeReason::QueryPallet);
} : {
executor.bench_process(xcm)?;
} verify {
@@ -355,9 +354,6 @@ benchmarks! {
use xcm_executor::traits::VersionChangeNotifier;
// First we need to subscribe to notifications.
let (origin, _) = T::transact_origin_and_runtime_call()?;
-
- T::DeliveryHelper::ensure_successful_delivery(&origin, &origin, FeeReason::QueryPallet);
-
let query_id = Default::default();
let max_response_weight = Default::default();
<T::XcmConfig as xcm_executor::Config>::SubscriptionService::start(
diff --git a/polkadot/xcm/pallet-xcm/Cargo.toml b/polkadot/xcm/pallet-xcm/Cargo.toml
index 2b961985cb82dc38da8c3df97fab71a24dcbf080..25b9c0a20b5e49168e2ef43d437264164d98ea68 100644
--- a/polkadot/xcm/pallet-xcm/Cargo.toml
+++ b/polkadot/xcm/pallet-xcm/Cargo.toml
@@ -61,7 +61,6 @@ runtime-benchmarks = [
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
"xcm-runtime-apis/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
"frame-support/try-runtime",
diff --git a/polkadot/xcm/pallet-xcm/src/benchmarking.rs b/polkadot/xcm/pallet-xcm/src/benchmarking.rs
index dd3c58c5dc7728e6ee4e7d256a81b15778f57823..e493d4838f5ca4a7a6bca0c8a5def791046f84ef 100644
--- a/polkadot/xcm/pallet-xcm/src/benchmarking.rs
+++ b/polkadot/xcm/pallet-xcm/src/benchmarking.rs
@@ -96,13 +96,6 @@ benchmarks! {
)?
.into();
let versioned_msg = VersionedXcm::from(msg);
-
- // Ensure that origin can send to destination (e.g. setup delivery fees, ensure router setup, ...)
- T::DeliveryHelper::ensure_successful_delivery(
- &Default::default(),
- &versioned_dest.clone().try_into().unwrap(),
- FeeReason::ChargeFees,
- );
}: _<RuntimeOrigin<T>>(send_origin, Box::new(versioned_dest), Box::new(versioned_msg))
teleport_assets {
@@ -171,7 +164,7 @@ benchmarks! {
}
// Ensure that origin can send to destination (e.g. setup delivery fees, ensure router setup, ...)
- T::DeliveryHelper::ensure_successful_delivery(
+ let (_, _) = T::DeliveryHelper::ensure_successful_delivery(
&origin_location,
&destination,
FeeReason::ChargeFees,
@@ -234,13 +227,6 @@ benchmarks! {
let versioned_beneficiary: VersionedLocation =
AccountId32 { network: None, id: recipient.into() }.into();
let versioned_assets: VersionedAssets = assets.into();
-
- // Ensure that origin can send to destination (e.g. setup delivery fees, ensure router setup, ...)
- T::DeliveryHelper::ensure_successful_delivery(
- &Default::default(),
- &versioned_dest.clone().try_into().unwrap(),
- FeeReason::ChargeFees,
- );
}: _<RuntimeOrigin<T>>(send_origin.into(), Box::new(versioned_dest), Box::new(versioned_beneficiary), Box::new(versioned_assets), 0, WeightLimit::Unlimited)
verify {
// run provided verification function
@@ -273,14 +259,6 @@ benchmarks! {
BenchmarkError::Override(BenchmarkResult::from_weight(Weight::MAX)),
)?
.into();
-
- // Ensure that origin can send to destination (e.g. setup delivery fees, ensure router setup, ...)
- T::DeliveryHelper::ensure_successful_delivery(
- &Default::default(),
- &versioned_loc.clone().try_into().unwrap(),
- FeeReason::ChargeFees,
- );
-
}: _(RawOrigin::Root, Box::new(versioned_loc))
force_unsubscribe_version_notify {
@@ -288,14 +266,6 @@ benchmarks! {
BenchmarkError::Override(BenchmarkResult::from_weight(Weight::MAX)),
)?;
let versioned_loc: VersionedLocation = loc.clone().into();
-
- // Ensure that origin can send to destination (e.g. setup delivery fees, ensure router setup, ...)
- T::DeliveryHelper::ensure_successful_delivery(
- &Default::default(),
- &versioned_loc.clone().try_into().unwrap(),
- FeeReason::ChargeFees,
- );
-
let _ = crate::Pallet::<T>::request_version_notify(loc);
}: _(RawOrigin::Root, Box::new(versioned_loc))
diff --git a/polkadot/xcm/src/v5/traits.rs b/polkadot/xcm/src/v5/traits.rs
index 79d3285614287fd98e7cbc8013dd2f45e209c4e5..71b67e97d5fecd0c67bd9f9c9f175ccd4b1f178f 100644
--- a/polkadot/xcm/src/v5/traits.rs
+++ b/polkadot/xcm/src/v5/traits.rs
@@ -460,10 +460,6 @@ pub trait SendXcm {
/// Actually carry out the delivery operation for a previously validated message sending.
fn deliver(ticket: Self::Ticket) -> result::Result<XcmHash, SendError>;
-
- /// Ensure `[Self::delivery]` is successful for the given `location` when called in benchmarks.
- #[cfg(feature = "runtime-benchmarks")]
- fn ensure_successful_delivery(_location: Option<Location>) {}
}
#[impl_trait_for_tuples::impl_for_tuples(30)]
@@ -504,13 +500,6 @@ impl SendXcm for Tuple {
)* );
Err(SendError::Unroutable)
}
-
- #[cfg(feature = "runtime-benchmarks")]
- fn ensure_successful_delivery(location: Option<Location>) {
- for_tuples!( #(
- return Tuple::ensure_successful_delivery(location.clone());
- )* );
- }
}
/// Convenience function for using a `SendXcm` implementation. Just interprets the `dest` and wraps
diff --git a/polkadot/xcm/xcm-builder/Cargo.toml b/polkadot/xcm/xcm-builder/Cargo.toml
index b806abbfefe8aec9b355d0522c18778f8278f031..cd23e691deaef61896bb696192b29dfe6c23234f 100644
--- a/polkadot/xcm/xcm-builder/Cargo.toml
+++ b/polkadot/xcm/xcm-builder/Cargo.toml
@@ -48,7 +48,6 @@ runtime-benchmarks = [
"polkadot-parachain-primitives/runtime-benchmarks",
"sp-runtime/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
std = [
"codec/std",
diff --git a/polkadot/xcm/xcm-builder/src/pay.rs b/polkadot/xcm/xcm-builder/src/pay.rs
index 0093051290b74bcda97721944361d2af562ed738..978c6870cdaf1e65ffc74adb175fd2e80ba53031 100644
--- a/polkadot/xcm/xcm-builder/src/pay.rs
+++ b/polkadot/xcm/xcm-builder/src/pay.rs
@@ -70,8 +70,8 @@ impl<
Router: SendXcm,
Querier: QueryHandler,
Timeout: Get<Querier::BlockNumber>,
- Beneficiary: Clone + core::fmt::Debug,
- AssetKind: core::fmt::Debug,
+ Beneficiary: Clone,
+ AssetKind,
AssetKindToLocatableAsset: TryConvert<AssetKind, LocatableAssetId>,
BeneficiaryRefToLocation: for<'a> TryConvert<&'a Beneficiary, Location>,
> Pay
@@ -144,9 +144,10 @@ impl<
}
#[cfg(feature = "runtime-benchmarks")]
- fn ensure_successful(_: &Self::Beneficiary, asset_kind: Self::AssetKind, _: Self::Balance) {
- let locatable = AssetKindToLocatableAsset::try_convert(asset_kind).unwrap();
- Router::ensure_successful_delivery(Some(locatable.location));
+ fn ensure_successful(_: &Self::Beneficiary, _: Self::AssetKind, _: Self::Balance) {
+ // We cannot generally guarantee this will go through successfully since we don't have any
+ // control over the XCM transport layers. We just assume that the benchmark environment
+ // will be sending it somewhere sensible.
}
#[cfg(feature = "runtime-benchmarks")]
diff --git a/polkadot/xcm/xcm-builder/src/routing.rs b/polkadot/xcm/xcm-builder/src/routing.rs
index 5b0d0a5f983503b570ccc41259981a7bd1d549a2..fc2de89d21285076fa2db281108ae9c98bacc7bb 100644
--- a/polkadot/xcm/xcm-builder/src/routing.rs
+++ b/polkadot/xcm/xcm-builder/src/routing.rs
@@ -60,11 +60,6 @@ impl<Inner: SendXcm> SendXcm for WithUniqueTopic<Inner> {
Inner::deliver(ticket)?;
Ok(unique_id)
}
-
- #[cfg(feature = "runtime-benchmarks")]
- fn ensure_successful_delivery(location: Option<Location>) {
- Inner::ensure_successful_delivery(location);
- }
}
impl<Inner: InspectMessageQueues> InspectMessageQueues for WithUniqueTopic<Inner> {
fn clear_messages() {
@@ -119,11 +114,6 @@ impl<Inner: SendXcm, TopicSource: SourceTopic> SendXcm for WithTopicSource<Inner
Inner::deliver(ticket)?;
Ok(unique_id)
}
-
- #[cfg(feature = "runtime-benchmarks")]
- fn ensure_successful_delivery(location: Option<Location>) {
- Inner::ensure_successful_delivery(location);
- }
}
/// Trait for a type which ensures all requirements for successful delivery with XCM transport
@@ -221,9 +211,4 @@ impl<Inner: SendXcm> SendXcm for EnsureDecodableXcm<Inner> {
fn deliver(ticket: Self::Ticket) -> Result<XcmHash, SendError> {
Inner::deliver(ticket)
}
-
- #[cfg(feature = "runtime-benchmarks")]
- fn ensure_successful_delivery(location: Option<Location>) {
- Inner::ensure_successful_delivery(location);
- }
}
diff --git a/polkadot/xcm/xcm-builder/src/tests/pay/pay.rs b/polkadot/xcm/xcm-builder/src/tests/pay/pay.rs
index b4718edc6c9889cf8c5b248a2a97dd19c3b5eac5..062faee2abd96a07afda49883adf86e9b2107e5a 100644
--- a/polkadot/xcm/xcm-builder/src/tests/pay/pay.rs
+++ b/polkadot/xcm/xcm-builder/src/tests/pay/pay.rs
@@ -22,7 +22,7 @@ use frame_support::{assert_ok, traits::tokens::Pay};
/// Type representing both a location and an asset that is held at that location.
/// The id of the held asset is relative to the location where it is being held.
-#[derive(Encode, Decode, Clone, PartialEq, Eq, Debug)]
+#[derive(Encode, Decode, Clone, PartialEq, Eq)]
pub struct AssetKind {
destination: Location,
asset_id: AssetId,
diff --git a/polkadot/xcm/xcm-builder/src/universal_exports.rs b/polkadot/xcm/xcm-builder/src/universal_exports.rs
index 6b3c3adf737dbb6e938d59383c5100041c493d52..aae8438c78d2e1685c323faf38d89b0a52578cad 100644
--- a/polkadot/xcm/xcm-builder/src/universal_exports.rs
+++ b/polkadot/xcm/xcm-builder/src/universal_exports.rs
@@ -95,9 +95,6 @@ impl<Exporter: ExportXcm, UniversalLocation: Get<InteriorLocation>> SendXcm
fn deliver(ticket: Exporter::Ticket) -> Result<XcmHash, SendError> {
Exporter::deliver(ticket)
}
-
- #[cfg(feature = "runtime-benchmarks")]
- fn ensure_successful_delivery(_: Option<Location>) {}
}
pub trait ExporterFor {
@@ -264,11 +261,6 @@ impl<Bridges: ExporterFor, Router: SendXcm, UniversalLocation: Get<InteriorLocat
fn deliver(validation: Self::Ticket) -> Result<XcmHash, SendError> {
Router::deliver(validation)
}
-
- #[cfg(feature = "runtime-benchmarks")]
- fn ensure_successful_delivery(location: Option<Location>) {
- Router::ensure_successful_delivery(location);
- }
}
/// Implementation of `SendXcm` which wraps the message inside an `ExportMessage` instruction
@@ -369,11 +361,6 @@ impl<Bridges: ExporterFor, Router: SendXcm, UniversalLocation: Get<InteriorLocat
fn deliver(ticket: Router::Ticket) -> Result<XcmHash, SendError> {
Router::deliver(ticket)
}
-
- #[cfg(feature = "runtime-benchmarks")]
- fn ensure_successful_delivery(location: Option<Location>) {
- Router::ensure_successful_delivery(location);
- }
}
impl<Bridges, Router, UniversalLocation> InspectMessageQueues
@@ -626,9 +613,6 @@ mod tests {
fn deliver(_ticket: Self::Ticket) -> Result<XcmHash, SendError> {
Ok([0; 32])
}
-
- #[cfg(feature = "runtime-benchmarks")]
- fn ensure_successful_delivery(_: Option<Location>) {}
}
impl<Filter: Contains<(NetworkId, InteriorLocation)>> ExportXcm for OkFor<Filter> {
type Ticket = ();
diff --git a/polkadot/xcm/xcm-executor/Cargo.toml b/polkadot/xcm/xcm-executor/Cargo.toml
index ac3cda4d5320e756454cc5b5bacaeef74cff2e2d..f2bc31fee2e9737a7375ab9796135f7c1b955b7e 100644
--- a/polkadot/xcm/xcm-executor/Cargo.toml
+++ b/polkadot/xcm/xcm-executor/Cargo.toml
@@ -30,7 +30,6 @@ runtime-benchmarks = [
"frame-benchmarking/runtime-benchmarks",
"frame-support/runtime-benchmarks",
"sp-runtime/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
std = [
"codec/std",
diff --git a/polkadot/xcm/xcm-executor/integration-tests/Cargo.toml b/polkadot/xcm/xcm-executor/integration-tests/Cargo.toml
index a89dd74a44faf6140f228d15748b3c3ca0c5510f..0045295deb0f59228d8418d6f31fb6ae0c756688 100644
--- a/polkadot/xcm/xcm-executor/integration-tests/Cargo.toml
+++ b/polkadot/xcm/xcm-executor/integration-tests/Cargo.toml
@@ -12,25 +12,30 @@ workspace = true
[dependencies]
codec = { workspace = true, default-features = true }
-frame-support = { workspace = true }
-frame-system = { workspace = true, default-features = true }
+frame-support.workspace = true
futures = { workspace = true }
-pallet-transaction-payment = { workspace = true, default-features = true }
-pallet-sudo = { workspace = true, default-features = true }
-pallet-xcm = { workspace = true, default-features = true }
-polkadot-runtime-parachains = { workspace = true, default-features = true }
-polkadot-test-client = { workspace = true }
-polkadot-test-runtime = { workspace = true }
-polkadot-test-service = { workspace = true }
-sp-consensus = { workspace = true, default-features = true }
-sp-keyring = { workspace = true, default-features = true }
-sp-runtime = { workspace = true }
-sp-state-machine = { workspace = true, default-features = true }
-xcm = { workspace = true }
-xcm-executor = { workspace = true, default-features = true }
-sp-tracing = { workspace = true, default-features = true }
-sp-core = { workspace = true, default-features = true }
+pallet-transaction-payment.workspace = true
+pallet-transaction-payment.default-features = true
+pallet-xcm.workspace = true
+pallet-xcm.default-features = true
+polkadot-test-client.workspace = true
+polkadot-test-runtime.workspace = true
+polkadot-test-service.workspace = true
+sp-consensus.workspace = true
+sp-consensus.default-features = true
+sp-keyring.workspace = true
+sp-keyring.default-features = true
+sp-runtime.workspace = true
+sp-state-machine.workspace = true
+sp-state-machine.default-features = true
+xcm.workspace = true
+xcm-executor.workspace = true
+xcm-executor.default-features = true
+sp-tracing.workspace = true
+sp-tracing.default-features = true
+sp-core.workspace = true
+sp-core.default-features = true
[features]
default = ["std"]
-std = ["frame-support/std", "frame-system/std", "pallet-sudo/std", "polkadot-runtime-parachains/std", "sp-runtime/std", "xcm/std"]
+std = ["frame-support/std", "sp-runtime/std", "xcm/std"]
diff --git a/polkadot/xcm/xcm-executor/integration-tests/src/lib.rs b/polkadot/xcm/xcm-executor/integration-tests/src/lib.rs
index dfcc3fc4187fa751682c11f2c765ff305549135b..699a081e4f2245de481957272ea31c1caabd3ffe 100644
--- a/polkadot/xcm/xcm-executor/integration-tests/src/lib.rs
+++ b/polkadot/xcm/xcm-executor/integration-tests/src/lib.rs
@@ -375,26 +375,6 @@ fn deposit_reserve_asset_works_for_any_xcm_sender() {
let mut block_builder = client.init_polkadot_block_builder();
- // Make the para available, so that `DMP` doesn't reject the XCM because the para is unknown.
- let make_para_available =
- construct_extrinsic(
- &client,
- polkadot_test_runtime::RuntimeCall::Sudo(pallet_sudo::Call::sudo {
- call: Box::new(polkadot_test_runtime::RuntimeCall::System(
- frame_system::Call::set_storage {
- items: vec![(
- polkadot_runtime_parachains::paras::Heads::<
- polkadot_test_runtime::Runtime,
- >::hashed_key_for(2000u32),
- vec![1, 2, 3],
- )],
- },
- )),
- }),
- sp_keyring::Sr25519Keyring::Alice,
- 0,
- );
-
// Simulate execution of an incoming XCM message at the reserve chain
let execute = construct_extrinsic(
&client,
@@ -403,12 +383,9 @@ fn deposit_reserve_asset_works_for_any_xcm_sender() {
max_weight: Weight::from_parts(1_000_000_000, 1024 * 1024),
}),
sp_keyring::Sr25519Keyring::Alice,
- 1,
+ 0,
);
- block_builder
- .push_polkadot_extrinsic(make_para_available)
- .expect("pushes extrinsic");
block_builder.push_polkadot_extrinsic(execute).expect("pushes extrinsic");
let block = block_builder.build().expect("Finalizes the block").block;
diff --git a/polkadot/xcm/xcm-runtime-apis/Cargo.toml b/polkadot/xcm/xcm-runtime-apis/Cargo.toml
index 0d944c368d66cab09022a02384c0cde6ac15437a..c00b8ee0b853ba2f28bb395b7b5f5a0469279f37 100644
--- a/polkadot/xcm/xcm-runtime-apis/Cargo.toml
+++ b/polkadot/xcm/xcm-runtime-apis/Cargo.toml
@@ -46,5 +46,4 @@ std = [
runtime-benchmarks = [
"frame-support/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
diff --git a/polkadot/xcm/xcm-simulator/example/Cargo.toml b/polkadot/xcm/xcm-simulator/example/Cargo.toml
index a25ddd63d1629c086a85f84c36f0febc48cba11a..9870944f088638edadbd82a6f591e190a44d262a 100644
--- a/polkadot/xcm/xcm-simulator/example/Cargo.toml
+++ b/polkadot/xcm/xcm-simulator/example/Cargo.toml
@@ -64,5 +64,4 @@ runtime-benchmarks = [
"sp-runtime/runtime-benchmarks",
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
diff --git a/polkadot/xcm/xcm-simulator/fuzzer/Cargo.toml b/polkadot/xcm/xcm-simulator/fuzzer/Cargo.toml
index 2e3e9af5e9947729673b4779998d34f5db574777..2168af5dbf2a976f4b4d68f0a0915c0d7691ed30 100644
--- a/polkadot/xcm/xcm-simulator/fuzzer/Cargo.toml
+++ b/polkadot/xcm/xcm-simulator/fuzzer/Cargo.toml
@@ -75,7 +75,6 @@ runtime-benchmarks = [
"sp-runtime/runtime-benchmarks",
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
[[bin]]
diff --git a/prdoc/pr_6604.prdoc b/prdoc/pr_6604.prdoc
deleted file mode 100644
index dc198287ff679ac7b03d2239eae1f4dd80f71c9e..0000000000000000000000000000000000000000
--- a/prdoc/pr_6604.prdoc
+++ /dev/null
@@ -1,106 +0,0 @@
-title: 'dmp: Check that the para exist before delivering a message'
-doc:
-- audience: Runtime Dev
- description: |
- Ensure that a para exists before trying to deliver a message to it.
- Besides that `ensure_successful_delivery` function is added to `SendXcm`. This function
- should be used by benchmarks to ensure that the delivery of a Xcm will work in the benchmark.
-crates:
-- name: polkadot-runtime-parachains
- bump: major
-- name: polkadot-runtime-common
- bump: major
-- name: polkadot-parachain-primitives
- bump: major
-- name: rococo-runtime
- bump: major
-- name: westend-runtime
- bump: major
-- name: pallet-xcm-benchmarks
- bump: major
-- name: pallet-xcm
- bump: major
-- name: cumulus-pallet-parachain-system
- bump: major
-- name: staging-xcm
- bump: major
-- name: staging-xcm-builder
- bump: major
-- name: bridge-runtime-common
- bump: major
-- name: pallet-xcm-bridge-hub-router
- bump: major
-- name: pallet-xcm-bridge-hub
- bump: major
-- name: snowbridge-pallet-inbound-queue
- bump: major
-- name: snowbridge-pallet-system
- bump: major
-- name: snowbridge-core
- bump: major
-- name: snowbridge-router-primitives
- bump: major
-- name: snowbridge-runtime-common
- bump: major
-- name: snowbridge-runtime-test-common
- bump: major
-- name: cumulus-pallet-dmp-queue
- bump: major
-- name: cumulus-pallet-xcmp-queue
- bump: major
-- name: parachains-common
- bump: major
-- name: asset-hub-rococo-runtime
- bump: major
-- name: asset-hub-westend-runtime
- bump: major
-- name: assets-common
- bump: major
-- name: bridge-hub-rococo-runtime
- bump: major
-- name: bridge-hub-westend-runtime
- bump: major
-- name: bridge-hub-common
- bump: major
-- name: collectives-westend-runtime
- bump: major
-- name: contracts-rococo-runtime
- bump: major
-- name: coretime-rococo-runtime
- bump: major
-- name: coretime-westend-runtime
- bump: major
-- name: glutton-westend-runtime
- bump: major
-- name: people-rococo-runtime
- bump: major
-- name: people-westend-runtime
- bump: major
-- name: penpal-runtime
- bump: major
-- name: rococo-parachain-runtime
- bump: major
-- name: polkadot-parachain-bin
- bump: major
-- name: cumulus-primitives-core
- bump: major
-- name: cumulus-primitives-utility
- bump: major
-- name: polkadot-service
- bump: major
-- name: staging-xcm-executor
- bump: major
-- name: xcm-runtime-apis
- bump: major
-- name: xcm-simulator-example
- bump: major
-- name: pallet-contracts
- bump: major
-- name: pallet-contracts-mock-network
- bump: major
-- name: pallet-revive
- bump: major
-- name: pallet-revive-mock-network
- bump: major
-- name: polkadot-sdk
- bump: major
diff --git a/substrate/frame/contracts/Cargo.toml b/substrate/frame/contracts/Cargo.toml
index a18c9d66ca40914ae9d63f76754b18303920eeaf..8265f666fe3155005d44c8450aea21b8a646f6ed 100644
--- a/substrate/frame/contracts/Cargo.toml
+++ b/substrate/frame/contracts/Cargo.toml
@@ -110,7 +110,6 @@ runtime-benchmarks = [
"sp-runtime/runtime-benchmarks",
"wasm-instrument",
"xcm-builder/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
"frame-support/try-runtime",
diff --git a/substrate/frame/contracts/mock-network/Cargo.toml b/substrate/frame/contracts/mock-network/Cargo.toml
index c46a02b6d9d27e75f5387996acda1d02d7003f25..aa715d93596c2d29d0aad2c6590ef80cb625be4a 100644
--- a/substrate/frame/contracts/mock-network/Cargo.toml
+++ b/substrate/frame/contracts/mock-network/Cargo.toml
@@ -102,5 +102,4 @@ runtime-benchmarks = [
"sp-runtime/runtime-benchmarks",
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
diff --git a/substrate/frame/revive/Cargo.toml b/substrate/frame/revive/Cargo.toml
index 5a87344f920271e02e49da0d2b5c630e875ba251..8b845af147a1e2a22b7710f5cf5faf8e22092589 100644
--- a/substrate/frame/revive/Cargo.toml
+++ b/substrate/frame/revive/Cargo.toml
@@ -117,7 +117,6 @@ runtime-benchmarks = [
"pallet-transaction-payment/runtime-benchmarks",
"sp-runtime/runtime-benchmarks",
"xcm-builder/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
"frame-support/try-runtime",
diff --git a/substrate/frame/revive/mock-network/Cargo.toml b/substrate/frame/revive/mock-network/Cargo.toml
index cc866012ec4c96d7889587ba282a2859cef3a1b9..76a8be8f2cabb6ebea9100b8a62d549e5d79e5c4 100644
--- a/substrate/frame/revive/mock-network/Cargo.toml
+++ b/substrate/frame/revive/mock-network/Cargo.toml
@@ -99,7 +99,6 @@ runtime-benchmarks = [
"sp-runtime/runtime-benchmarks",
"xcm-builder/runtime-benchmarks",
"xcm-executor/runtime-benchmarks",
- "xcm/runtime-benchmarks",
]
try-runtime = [
"frame-support/try-runtime",
diff --git a/umbrella/Cargo.toml b/umbrella/Cargo.toml
index 4bcd4b4b4a618dd499c8e2b2315e292321fe145e..76bcb4d1c5270b01a2f9c58b8adc705dbf764ba1 100644
--- a/umbrella/Cargo.toml
+++ b/umbrella/Cargo.toml
@@ -364,7 +364,6 @@ runtime-benchmarks = [
"staging-node-inspect?/runtime-benchmarks",
"staging-xcm-builder?/runtime-benchmarks",
"staging-xcm-executor?/runtime-benchmarks",
- "staging-xcm?/runtime-benchmarks",
"xcm-runtime-apis?/runtime-benchmarks",
]
try-runtime = [