Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Limit the maximum number of wasm memory pages a runtime can have #9308

Merged
5 commits merged into from
Aug 2, 2021

Conversation

pepyakin
Copy link
Contributor

@pepyakin pepyakin commented Jul 8, 2021

Introduce a knob into the wasmtime configuration that allows for limiting the maximum number of wasm pages that a Substrate Runtime can allocate.

This is going to be used by polkadot (in a companion) to limit the maximum number of wasm memory pages a PVF can use.

polkadot companion: paritytech/polkadot#3449

@pepyakin pepyakin added A0-please_review Pull request needs code review. B5-clientnoteworthy C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels Jul 8, 2021
@pepyakin pepyakin requested review from athei and bkchr July 8, 2021 11:24
@pepyakin pepyakin added this to To Do in Substrate Runtime Wasm Execution via automation Jul 8, 2021
client/executor/wasmtime/src/tests.rs Outdated Show resolved Hide resolved
@pepyakin
Copy link
Contributor Author

pepyakin commented Aug 2, 2021

bot merge

@ghost
Copy link

ghost commented Aug 2, 2021

Trying merge.

@ghost ghost merged commit be72e5d into master Aug 2, 2021
Substrate Runtime Wasm Execution automation moved this from To Do to Done Aug 2, 2021
@ghost ghost deleted the ser-memory-pages branch August 2, 2021 19:15
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Development

Successfully merging this pull request may close these issues.

None yet

3 participants