-
Notifications
You must be signed in to change notification settings - Fork 770
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PolkaVM parachain runtimes MVP #6704
base: master
Are you sure you want to change the base?
Conversation
Note that we've recently updated the builder crate to use the newest PolkaVM and we're also updating the executor here, so you'll have to align to those changes. |
Questions:
|
No it doesn't make any difference for pallet-revive. Unless we also enable gas metering for the runtime itself. Then we could get rid of a lot of annoying benchmarks there.
I guess the relay chain runtime can stay wasm. But they still need to be updated to accept PVM PvFs. So yes they need to support this.
We currently build the following runtimes to PVM as part of CI: polkadot-sdk/.github/workflows/build-misc.yml Lines 42 to 45 in 65a4e5e
Yeah sure. It will just be like a runtime upgrade. You replace the wasm blob with a PVM blob that does the same thing. |
All GitHub workflows were cancelled due to failure one of the required jobs. |
Overview
This PR aims to enable parachain runtime authors to use PolkaVM as a compilation and execution target for PVFs. It brings in numerous advantages of PolkaVM over the Wasm executor:
Although PolkaVM potentially enables us also to use advanced gas metering for PVFs in the future, eliminating a lot of non-determinism issues, it's out of the scope of this MVP PR for now.
Current status
Fully working hacky PoC. Leaks host memory. Doesn't use execution workers and blocks the candidate validation subsystem on execution instead.
CC @eskimor @sandreim @koute @bkchr