-
Notifications
You must be signed in to change notification settings - Fork 88
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
Use hypercall definitions from uhyve-interface #1393
Conversation
Depends on #1442 |
9720732
to
9a16274
Compare
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.
Overall, this looks good to me, but I have a few remarks. :)
12eebc9
to
0f6b24b
Compare
I hope the latest version has addressed all your comments 🙂 |
0f6b24b
to
d98ff02
Compare
d98ff02
to
954098f
Compare
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.
Looks good to me! :)
Sorry for the wait!
This depends on hermit-os/uhyve#757 (and a publication of uhyve interface V0.1.2)
This is the final step of putting the hypercall definitions in a central crate.
This PR should just deduplicate code without changing functionality.