Stream: git-wasmtime

Topic: wasmtime / PR #14388 Refactor graceful shutdown in `wasmt...


view this post on Zulip Wasmtime GitHub notifications bot (Sep 23 2026 at 16:30):

alexcrichton opened PR #14388 from alexcrichton:refactor-serve-shutdown to bytecodealliance:main:

This commit fixes an issue with the synchronization used in shutting down wasmtime serve with a ctrl-c or a --shutdown-addr option. Notably the previously used method .notify_waiters() did not work if the acceptor loops weren't actually blocked in waiting. Using tokio's Notify abstraction is a bit finnicky, so this encapsulates the shutdown state in a local mod to avoid reaching into private internals outside of this small region of code, and operations are now all under the Mutex for the shutdown entirely which should provide the synchronization necessary.

<!--
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 review the Bytecode Alliance's AI tool usage policy at
https://github.com/bytecodealliance/governance/blob/main/AI_TOOL_POLICY.md

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 (Sep 23 2026 at 16:30):

alexcrichton requested dicej for a review on PR #14388.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 23 2026 at 16:30):

alexcrichton requested wasmtime-core-reviewers for a review on PR #14388.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 23 2026 at 16:53):

:thumbs_up: dicej submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 23 2026 at 16:53):

:speech_balloon: dicej created PR review comment:

        /// Flags this state as done spawning tasks and returns a future which
        /// will resolve when all active tasks have completed, if any.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 23 2026 at 17:08):

alexcrichton updated PR #14388.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 23 2026 at 17:08):

alexcrichton has enabled auto merge for PR #14388.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 23 2026 at 17:18):

alexcrichton added PR #14388 Refactor graceful shutdown in wasmtime serve to the merge queue

view this post on Zulip Wasmtime GitHub notifications bot (Sep 23 2026 at 17:43):

:check: alexcrichton merged PR #14388.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 23 2026 at 17:43):

alexcrichton removed PR #14388 Refactor graceful shutdown in wasmtime serve from the merge queue


Last updated: Oct 11 2026 at 04:10 UTC