Skip to content

Add docs for OOM-handling in Wasmtime#13224

Merged
fitzgen merged 3 commits intobytecodealliance:mainfrom
fitzgen:wasmtime-oom-docs
Apr 28, 2026
Merged

Add docs for OOM-handling in Wasmtime#13224
fitzgen merged 3 commits intobytecodealliance:mainfrom
fitzgen:wasmtime-oom-docs

Conversation

@fitzgen
Copy link
Copy Markdown
Member

@fitzgen fitzgen commented Apr 28, 2026

Part of #12069

fitzgen added 2 commits April 28, 2026 13:34
And use this as a place to document which portions of Wasmtime attempt to handle
OOM, how to enable `cfg(arc_try_new)` to handle more OOMs with a nightly rust
toolchain, and clarify that OOM handling is not tier 1.
@fitzgen fitzgen requested a review from a team as a code owner April 28, 2026 21:23
@fitzgen fitzgen requested review from cfallin and removed request for a team April 28, 2026 21:23
Copy link
Copy Markdown
Member

@cfallin cfallin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally looks great -- thanks for documenting all of this!

(I didn't check the exact correspondence of where you added the # Errors subsections to which functions should have then but I spot-checked, and trust that you carefully lined those up)

Only a few thoughts on the table below...

Comment thread crates/core/src/error/oom.rs Outdated
Comment thread crates/core/src/error/oom.rs Outdated
@fitzgen fitzgen enabled auto-merge April 28, 2026 22:02
@fitzgen fitzgen added this pull request to the merge queue Apr 28, 2026
Merged via the queue into bytecodealliance:main with commit 42e4093 Apr 28, 2026
48 checks passed
@fitzgen fitzgen deleted the wasmtime-oom-docs branch April 28, 2026 22:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants