Stream: git-wasmtime

Topic: wasmtime / PR #6789 Wasmtime: Introduce `{Module,Componen...


view this post on Zulip Wasmtime GitHub notifications bot (Jul 31 2023 at 17:52):

fitzgen requested wasmtime-core-reviewers for a review on PR #6789.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 31 2023 at 17:52):

fitzgen opened PR #6789 from fitzgen:resources-required to bytecodealliance:main:

Returns a summary of the resources required to instantiate this module or component.

Potential uses of the returned information:

Part of #6627.

<!--
Please make sure you include the following information:

Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.html

Please ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->

view this post on Zulip Wasmtime GitHub notifications bot (Jul 31 2023 at 17:52):

fitzgen requested pchickey for a review on PR #6789.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 31 2023 at 23:26):

pchickey submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 31 2023 at 23:26):

pchickey submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 31 2023 at 23:26):

pchickey created PR review comment:

I expect that this means, for all components using the wasi-preview1-component-adapter (which, for now, is approximately all "real world" components), we won't be able to use this estimator. How much effort would it be to detect and handle the simplest cases of modules importing a memory?

view this post on Zulip Wasmtime GitHub notifications bot (Jul 31 2023 at 23:36):

fitzgen submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 31 2023 at 23:36):

fitzgen created PR review comment:

If the adapter is still part of this component graph, then everything should be fine and this method will work today as-is.

If the adapter is a module that the component graph imports from the host, then there really isn't much we can do here. An imported module could define zero or five memories, we have no way of telling how many it will need without actually having the module.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 01 2023 at 00:04):

pchickey created PR review comment:

got it! thanks. I misunderstood the way in which Import was being used here.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 01 2023 at 00:04):

pchickey submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 01 2023 at 00:05):

pchickey edited PR review comment.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 01 2023 at 00:05):

pchickey edited PR review comment.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 01 2023 at 16:06):

fitzgen submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 01 2023 at 16:06):

fitzgen created PR review comment:

Ah, yeah this is after the component graph has been collapsed/inlined into a single thing, so imports at this layer are imports into the component graph, not between modules within the same component graph.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 01 2023 at 20:03):

fitzgen updated PR #6789.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 01 2023 at 20:03):

fitzgen has enabled auto merge for PR #6789.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 01 2023 at 21:19):

fitzgen merged PR #6789.


Last updated: Dec 23 2024 at 12:05 UTC