Stream: git-wasmtime

Topic: wasmtime / PR #12910 mpk: Fix index used when purging a m...


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

alexcrichton opened PR #12910 from alexcrichton:fix-mpk-purge-index to bytecodealliance:main:

This commit fixes an issue with the pooling allocator when MPK is enabled, which is off-by-default at compile time. When a module is dropped all remaining images are purged from the pooling allocator, but the purging logic mistakenly used the wrong kind of index during purging which led to corruption of the pooling allocator itself. This fixes the logic and adds regression tests showcasing the issue as well.

<!--
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 (Mar 31 2026 at 17:08):

alexcrichton requested cfallin for a review on PR #12910.

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

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

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2026 at 19:04):

github-actions[bot] added the label wasmtime:api on PR #12910.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2026 at 22:50):

cfallin submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2026 at 22:57):

cfallin added PR #12910 mpk: Fix index used when purging a module in the pooling allocator to the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2026 at 23:27):

cfallin removed PR #12910 mpk: Fix index used when purging a module in the pooling allocator from the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2026 at 23:27):

cfallin merged PR #12910.


Last updated: Apr 12 2026 at 23:10 UTC