Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

maelstrom-linux should be IO-safe #160

Open
nfachan opened this issue Feb 3, 2024 · 0 comments
Open

maelstrom-linux should be IO-safe #160

nfachan opened this issue Feb 3, 2024 · 0 comments
Labels
Milestone

Comments

@nfachan
Copy link
Collaborator

nfachan commented Feb 3, 2024

See: https://rust-lang.github.io/rfcs/3128-io-safety.html
And: rust-lang/rust#87074
And: nix-rust/nix#1750

Our maelstrom-linux APIs are no IO-safe.

One problem we face is that the library is no_std, or at least can be no_std. The new file descriptor types introduced to help with IO-safety are only in std. We could probably create our own similar types. Or we could look to see if there's a generic way we could implement the appropriate functions that would work for both std and no_std.

@nfachan nfachan added the worker label Feb 3, 2024
@nfachan nfachan added this to the Someday milestone Feb 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant