WIP: Rework gas mapping

[athei]

Replacement of #9740.

Build on top of the new tx payment changes: #9780

Started a new PR because due to the rebase on top of the new tx payment changes this PR is substantially different and I don't want to invalidate the existing comments on #9740 which are not implemented, yet.

TODO

• Decode gas supplied by the tx into Weight inside dry_run_eth_transact
• Prevent users from submitting signed eth_* transactions. Otherwise they can set an arbitrary effective_gas_price. Probably using custom origins.
• Prevent pallet_revive from consuming too much balance from the tx payment hold
• Divide the storage deposit by the NextFeeMultiplier inside the dry run
• Move everything dependent on pallet_transaction payment into a sealed trait to remove all the new trait bounds all over the place

[TorstenStueber]

I tried to find the logic where in new frames the weight limit needs to be converted to a gas limit and then back into a weigh limit.

For EVM it seems like both conversions are not implemented yet:

• from weight to gas
• from gas to weight

For PVM I can't find the code for either conversion. Is this actually implemented in the compiler?

[TorstenStueber]

All storage deposits deducted in a frame should be returned when the frame reverts. Is this the case?

[athei]

I tried to find the logic where in new frames the weight limit needs to be converted to a gas limit and then back into a weigh limit.

For EVM it seems like both conversions are not implemented yet:

• from weight to gas
• from gas to weight

For PVM I can't find the code for either conversion. Is this actually implemented in the compiler?

Correct. This will be a follow up PR with the "on chain gas"/

All storage deposits deducted in a frame should be returned when the frame reverts. Is this the case?

You mean they should be reverted? Yeah. This PR doesn't touch this logic.

[athei]

@pgherveou @TorstenStueber

Sorry for the force push. This was the last one. I will now address the comments from #9740 with individual commits.

[TorstenStueber]

In line 1554 we must not call T::Currency::transfer but charge_deposit as every deposit should come from the hold/imbalance created in the preparation step of pallet-transaction-payment.

[athei]

Good catch. Fixed.

[paritytech-workflow-stopper]

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/18039439323
Failed job name: build-rustdoc