alexcrichton opened PR #13092 from alexcrichton:fix-drop-deadlock to bytecodealliance:main:
This commit removes a now-older assertion when disposing of a guest task that all threads are empty. This assertion is hit in a situation where
TypedFunc::call_asyncis used where the guest can't be entered due to backpressure. The assertion is a bit dated now and originally this helper function handled reparenting, but that's all gone now too. This commit removes the assertion entirely meaning that in this situation the guest thread will "leak" into the host, but the store is locked down at that point anyway, so this is in theory reasonable.<!--
Please make sure you include the following information:
If this work has been discussed elsewhere, please include a link to that
conversation. If it was discussed in an issue, just mention "issue #...".Explain why this change is needed. If the details are in an issue already,
this can be brief.Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.htmlPlease ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
alexcrichton requested wasmtime-core-reviewers for a review on PR #13092.
alexcrichton requested pchickey for a review on PR #13092.
pchickey submitted PR review.
alexcrichton added PR #13092 cm-async: Remove assertion when disposing a task to the merge queue.
github-merge-queue[bot] removed PR #13092 cm-async: Remove assertion when disposing a task from the merge queue.
github-actions[bot] added the label wasmtime:api on PR #13092.
alexcrichton updated PR #13092.
alexcrichton has enabled auto merge for PR #13092.
alexcrichton added PR #13092 cm-async: Remove assertion when disposing a task to the merge queue.
alexcrichton merged PR #13092.
alexcrichton removed PR #13092 cm-async: Remove assertion when disposing a task from the merge queue.
Last updated: May 03 2026 at 22:13 UTC