Stream: git-wasmtime

Topic: wasmtime / PR #8849 Allow 4G memories by default in the p...


view this post on Zulip Wasmtime GitHub notifications bot (Jun 20 2024 at 15:22):

alexcrichton opened PR #8849 from alexcrichton:increase-pooling-allocator-defaults to bytecodealliance:main:

This commit raises the default setting of max_memory_size in the pooling allocator from 10M to 4G. This won't actually impact the virtual memory reserved in the pooling allocator because we already reserved 6G of virtual memory for each linear memory this instead allows access to all of it by default. This matches the default behavior of Wasmtime for the non-pooling allocator which is to not artificially limit memory by default.

The main impact of this setting is that the memory-protection-keys feature, which is disabled by default, will have no effect by default unless max_memory_size is also configured to something smaller than 4G. The documentation has been updated to this effect.

Closes #8846

<!--
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 (Jun 20 2024 at 15:22):

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

view this post on Zulip Wasmtime GitHub notifications bot (Jun 20 2024 at 15:22):

alexcrichton requested elliottt for a review on PR #8849.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 20 2024 at 15:31):

dicej submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 20 2024 at 16:14):

elliottt submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 20 2024 at 16:35):

alexcrichton merged PR #8849.


Last updated: Oct 23 2024 at 20:03 UTC