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
Add mount syscall #494
Add mount syscall #494
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was a little uncertain at first at providing higher level APIs instead of just exposing the syscall, but I think you have captured all the use cases I can think of. (I looked through man 8 mount
for the high level operations)
So a somewhat superficial LGTM.
Cool beans, as long as we agree on the API surface we can muck around with the flags later if they become an issue. |
Every nightly check is currently broken because of a missing fuschia build. |
The fuchsia fix is now on main, so rebasing on main should fix the CI here. I also don't have time today to review this, but at a superficial glance this looks good. |
Signed-off-by: Alex Saveau <saveau.alexandre@gmail.com>
Yeep, fixed. |
Looks good, thanks! |
Woohoo! Unmount: #512 |
Signed-off-by: Alex Saveau <saveau.alexandre@gmail.com> Signed-off-by: Alex Saveau <saveau.alexandre@gmail.com>
Signed-off-by: Alex Saveau <saveau.alexandre@gmail.com> Signed-off-by: Alex Saveau <saveau.alexandre@gmail.com>
My flag management is absolutely terrible, any suggestions would be appreciated. :) The goal is to try and only provide applicable flags (which is maybe a hopeless cause), but that means having partially overlapping sets thereby complicating everything. Also,
StatVfsMountFlags
are related, not sure if we should somehow combine things.In another PR, I'll add umount.
Closes #491