Skip to content
This repository was archived by the owner on Mar 24, 2022. It is now read-only.

bump wasmtime: guestpointer conversions, wasi-common pipes, wiggle borrow checker factored out#553

Merged
pchickey merged 12 commits intomainfrom
pch/wasmtime_has_wiggle_ptr_stuff
Jul 9, 2020
Merged

bump wasmtime: guestpointer conversions, wasi-common pipes, wiggle borrow checker factored out#553
pchickey merged 12 commits intomainfrom
pch/wasmtime_has_wiggle_ptr_stuff

Conversation

@pchickey
Copy link
Contributor

@pchickey pchickey commented Jun 24, 2020

Several package version and minor interface bumps to pull in the latest wasmtime main branch

Adam did the majority of the work in #564 to get wasi-common to have a virtual pipe implementation upstream. Virtual pipes are now used throughout the Lucet tests instead of unix pipes.

Wiggle's borrow checker was factored out upstream, this PR copies the previous implementation to live in the Lucet crate and fixes up the implementation of wiggle's GuestMemory trait to use our own version.

@pchickey pchickey closed this Jun 26, 2020
@pchickey pchickey mentioned this pull request Jun 26, 2020
@acfoltzer acfoltzer reopened this Jun 26, 2020
@acfoltzer acfoltzer changed the base branch from master to main June 26, 2020 00:53
@pchickey pchickey force-pushed the pch/wasmtime_has_wiggle_ptr_stuff branch from 8b3d89e to 1bdff6c Compare July 1, 2020 20:06
@pchickey pchickey changed the title bump wasmtime bump wasmtime: guestpointer conversions, wasi-common pipes Jul 1, 2020
@pchickey pchickey marked this pull request as ready for review July 1, 2020 20:45
@pchickey pchickey requested a review from iximeow July 1, 2020 20:45
@valpackett valpackett mentioned this pull request Jul 8, 2020
@pchickey
Copy link
Contributor Author

pchickey commented Jul 8, 2020

The virtual pipe abstraction is now passing tests downstream (in our private code), so I'm now satisfied that this PR is ready to go!

@pchickey pchickey changed the title bump wasmtime: guestpointer conversions, wasi-common pipes bump wasmtime: guestpointer conversions, wasi-common pipes, wiggle borrow checker factored out Jul 8, 2020
@pchickey pchickey mentioned this pull request Jul 8, 2020
@pchickey pchickey requested a review from acfoltzer July 9, 2020 13:28
Copy link
Contributor

@jedisct1 jedisct1 left a comment

Choose a reason for hiding this comment

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

Works totally fine here, including on macOS.

@pchickey
Copy link
Contributor Author

pchickey commented Jul 9, 2020

Thank you!

@pchickey pchickey merged commit 83ad8b3 into main Jul 9, 2020
@pchickey pchickey deleted the pch/wasmtime_has_wiggle_ptr_stuff branch July 9, 2020 18:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants