Skip to content

Conversation

@alexcrichton
Copy link
Member

This commit adds PoolingAlloationConfig::pagemap_scan and additionally adds -Opooling-pagemap-scan to configure on the CLI. This is the same tri-state configuration option as MpkEnable so that enum was renamed to just Enabled and repurposed for both options.

This then additionally turns the option off-by-default instead of the previous on-by-default-if-able-to to enable more slowly rolling out this feature.

@alexcrichton alexcrichton requested a review from a team as a code owner August 14, 2025 20:30
@alexcrichton alexcrichton requested review from dicej and removed request for a team August 14, 2025 20:30
@alexcrichton alexcrichton requested review from a team as code owners August 14, 2025 20:40
@alexcrichton alexcrichton requested review from fitzgen and removed request for a team August 14, 2025 20:40
@alexcrichton alexcrichton added this pull request to the merge queue Aug 14, 2025
@alexcrichton alexcrichton removed this pull request from the merge queue due to a manual request Aug 14, 2025
This commit adds `PoolingAlloationConfig::pagemap_scan` and additionally
adds `-Opooling-pagemap-scan` to configure on the CLI. This is the same
tri-state configuration option as `MpkEnable` so that enum was renamed
to just `Enabled` and repurposed for both options.

This then additionally turns the option off-by-default instead of the
previous on-by-default-if-able-to to enable more slowly rolling out this
feature.
@alexcrichton alexcrichton enabled auto-merge August 14, 2025 21:01
@alexcrichton alexcrichton added this pull request to the merge queue Aug 14, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 14, 2025
@alexcrichton alexcrichton enabled auto-merge August 14, 2025 21:55
@alexcrichton alexcrichton added this pull request to the merge queue Aug 14, 2025
Merged via the queue into bytecodealliance:main with commit 698028c Aug 14, 2025
44 checks passed
@alexcrichton alexcrichton deleted the pagemap-scan-option branch August 14, 2025 22:29
bongjunj pushed a commit to prosyslab/wasmtime that referenced this pull request Oct 20, 2025
* Add a configuration knob for `PAGEMAP_SCAN`

This commit adds `PoolingAlloationConfig::pagemap_scan` and additionally
adds `-Opooling-pagemap-scan` to configure on the CLI. This is the same
tri-state configuration option as `MpkEnable` so that enum was renamed
to just `Enabled` and repurposed for both options.

This then additionally turns the option off-by-default instead of the
previous on-by-default-if-able-to to enable more slowly rolling out this
feature.

* Fix broken test
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