-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pooling allocator should take virtual address space into account (on aarch64 and riscv64) #8607
Labels
bug
Incorrect behavior in the current implementation that needs fixing
Comments
matsbror
added
the
bug
Incorrect behavior in the current implementation that needs fixing
label
May 13, 2024
alexcrichton
added a commit
to alexcrichton/wasmtime
that referenced
this issue
May 13, 2024
This commit aims to address bytecodealliance#8607 by dynamically determining whether the pooling allocator should be used rather than unconditionally using it. It looks like some systems don't have enough virtual memory to support the default configuration settings so this should help `wasmtime serve` work on those systems. Closes bytecodealliance#8607
In lieu of directly probing address space limitations I've opted to do something slightly different in #8610 which probes for a large reservation to succeed before attempting another one for the pooling allocator. |
Thanks. Will this also help spin, which uses wasmtime as library? The same error occurs there. |
Not directly, no, but I can propose the same change on Spin to have the same effect |
github-merge-queue bot
pushed a commit
that referenced
this issue
May 14, 2024
This commit aims to address #8607 by dynamically determining whether the pooling allocator should be used rather than unconditionally using it. It looks like some systems don't have enough virtual memory to support the default configuration settings so this should help `wasmtime serve` work on those systems. Closes #8607
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Thanks for filing a bug report! Please fill out the TODOs below.
Test Case
Hello wasi http
Steps to Reproduce
These steps fail on Ubuntu 24.04/riscv64 on VisionFive 2 and on Ubuntu 18.04/aarch64 on Nvidia Jetson Nano.
wasmtime serve target/wasm32-wasi/debug/hello_wasi_http.wasm
Expected Results
I would have expected the tutorial to work as described.
Actual Results
When running without pooling allocator it works:
Versions and Environment
Wasmtime version or commit: 21.0
Operating system:
Architecture:
Extra Info
I guess there should be a way to detect the virtual address size as it seems to be around 39 bits in these architectures.
The text was updated successfully, but these errors were encountered: