badeend edited PR #10429.
jsturtevant submitted PR review.
jsturtevant created PR review comment:
should we be return some information about the stream being closed with this error?
jsturtevant created PR review comment:
do we need logic to clean up these table entries?
badeend submitted PR review.
badeend created PR review comment:
badeend submitted PR review.
badeend created PR review comment:
You mean the table entries for the newly created
wasi:io/error
resources? The ownership of these resource is transferred into the guest one line below it. It's then the responsibility of the guest to drop it properly like any other resource. When the guest drops that resource, wasmtime removes it from the table in https://github.com/bytecodealliance/wasmtime/blob/main/crates/wasi-io/src/impls.rs#L114-L116
jsturtevant submitted PR review.
jsturtevant created PR review comment:
thanks for the explanation
badeend updated PR #10429.
badeend edited PR #10429:
Implements https://github.com/WebAssembly/wasi-tls/pull/10
And added a test case to verify the error information is indeed reaching the guest.
~PR is still draft, because above issue needs to be merged first. But feel free to review the design & implementation already.~
CC @jsturtevant
badeend has marked PR #10429 as ready for review.
badeend requested wasmtime-default-reviewers for a review on PR #10429.
Last updated: Apr 22 2025 at 16:04 UTC