Stream: git-wasmtime

Topic: wasmtime / Issue #1561 [WIP]: Introduce strongly-typed sy...

Wasmtime GitHub notifications bot (Apr 20 2020 at 19:02):

github-actions[bot] commented on Issue #1561:

Subscribe to Label Action

cc @kubkon

<details>
This issue or pull request has been labeled: "wasi"

Thus the following users have been cc'd because of the following labels:

• kubkon: wasi

To subscribe or unsubscribe from this label, edit the <code>.github/subscribe-to-label.json</code> configuration file.

Learn more.
</details>

Wasmtime GitHub notifications bot (Apr 22 2020 at 08:41):

kubkon commented on Issue #1561:

Hmm, so I reckon this PR is at a stage that's ready for review so I'll mark it as such. Besides the major changes to the way we treat OS handles, there's a couple more things I'm somewhat unclear on:

• Should OS-specific path, fd accept specific types such as OsFile, OsDir, and std::fs::File for syscalls that are OS handle-independent (e.g., fdstat_get)? This approach is currently presented in this PR, and it makes our intention a bit clearer what handle is expected for this OS-specific syscall to be called on. But then again, should we ever expand on different handle types and make some syscalls that are currently _only_ used by OsFile say, it would be more ergnonomic for the syscalls to simply accept std::fs::File.
• On the topic of using std::fs::File, the reason we currently have our _own_ type that resembles that of std::fs::File (OsHandle and OsDirHandle) is because this way we can wrap the underlying OS handle (RawFd or RawHandle) in a std::cell::Cell. Interior mutability _is_ required in fd_fdstat_set_flags call on Windows, where adjusting the flags can only be done by reopening the said file (am I correct here @peterhuene?). We could in principle wrap std::fs::File in a std::cell::RefCell but then we are risking a panic somewhere down the line. Perhaps though this risk is worth accepting at the benefit of cleaner codebase?

Anyhow, all thoughts, suggestions are more than welcome!

Wasmtime GitHub notifications bot (Apr 22 2020 at 08:44):

kubkon commented on Issue #1561:

@alexcrichton even though you might not know all the internals very well, I'd appreciate your opinion on this even if high level!

Last updated: Dec 23 2024 at 13:07 UTC