Stream: git-wasmtime

Topic: wasmtime / PR #8587 Handle all `decommit`s/`madvise(DONTN...

view this post on Zulip Wasmtime GitHub notifications bot (May 08 2024 at 23:52):

fitzgen opened PR #8587 from fitzgen:just-decommit to bytecodealliance:main:

We previously had different functions for tables vs memories vs stacks vs etc...

Now we use the same system virtual memory interface function for all of them. This enables follow up PRs where we will start batching these calls together (and potentially merging regions together in the further future, although that isn't a clear win as we've seen larger madvises take longer than small ones in the past). Those future batching PRs will allow us to in turn start prototyping new syscalls that can take advantage of that batching, like madvisev.

view this post on Zulip Wasmtime GitHub notifications bot (May 08 2024 at 23:52):

fitzgen requested alexcrichton for a review on PR #8587.

view this post on Zulip Wasmtime GitHub notifications bot (May 08 2024 at 23:52):

fitzgen requested wasmtime-core-reviewers for a review on PR #8587.

view this post on Zulip Wasmtime GitHub notifications bot (May 08 2024 at 23:53):

fitzgen commented on PR #8587:

cc https://github.com/bytecodealliance/wasmtime/issues/4637

view this post on Zulip Wasmtime GitHub notifications bot (May 09 2024 at 00:18):

alexcrichton submitted PR review:

I like it :+1:

view this post on Zulip Wasmtime GitHub notifications bot (May 09 2024 at 00:49):

fitzgen updated PR #8587.

view this post on Zulip Wasmtime GitHub notifications bot (May 09 2024 at 00:49):

fitzgen has enabled auto merge for PR #8587.

view this post on Zulip Wasmtime GitHub notifications bot (May 09 2024 at 01:28):

fitzgen merged PR #8587.

Last updated: Nov 22 2024 at 16:03 UTC