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

Narrow the set of available host functions for PVF #3005

Merged
merged 1 commit into from
May 13, 2021
Merged

Conversation

pepyakin
Copy link
Contributor

The original definition is

pub type SubstrateHostFunctions = (
	storage::HostFunctions,
	default_child_storage::HostFunctions,
	misc::HostFunctions,
	wasm_tracing::HostFunctions,
	offchain::HostFunctions,
	crypto::HostFunctions,
	hashing::HostFunctions,
	allocator::HostFunctions,
	logging::HostFunctions,
	sandbox::HostFunctions,
	crate::trie::HostFunctions,
	offchain_index::HostFunctions,
	runtime_tasks::HostFunctions,
);

we leave out:

  • storage related stuff (PVF doesn't have a notion of a persistent storage/trie),
  • tracing
  • off chain workers (PVFs do not have such a notion)
  • runtime tasks (it is experimental)
  • sandbox (still is being worked on)

@pepyakin pepyakin added the A0-please_review Pull request needs code review. label May 11, 2021
@pepyakin pepyakin requested a review from bkchr May 11, 2021 13:54
@pepyakin pepyakin added B0-silent Changes should not be mentioned in any release notes 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 May 11, 2021
@gavofyork gavofyork closed this May 13, 2021
@gavofyork gavofyork reopened this May 13, 2021
@gavofyork gavofyork merged commit 8746ca2 into master May 13, 2021
@gavofyork gavofyork deleted the ser-narrow-hfs branch May 13, 2021 10:15
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. B0-silent Changes should not be mentioned in any release notes 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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants