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

Make krun_start_enter error when root directory does not exist #150

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mtjhrc
Copy link
Collaborator

@mtjhrc mtjhrc commented Oct 25, 2023

Previously krun_start_enter would succeed and the guest kernel would just panic. The root filesystem directory was opened lazily when the guest kernel used fuse init opcode.
This commit changes it so the root directory is opened when creating the fs device.

This only fixes it on Linux, but not in the macOS implementation.

Previously krun_start_enter would succeed and the guest kernel would
just panic. The root filesystem directory was opened lazily when
the guest kernel used fuse init opcode.
This commit changes it so the root directory is opened when
creating the fs device.

This only fixes it on Linux, but not in the macOS implementation.

Signed-off-by: Matej Hrica <mhrica@redhat.com>
@mtjhrc
Copy link
Collaborator Author

mtjhrc commented Nov 1, 2023

@slp I fixed it in the macOS implementation too. I don't know if it works, but it compiles with cargo build --target aarch64-apple-darwin.

…xist

Signed-off-by: Matej Hrica <mhrica@redhat.com>
@mtjhrc
Copy link
Collaborator Author

mtjhrc commented Nov 29, 2023

Pipeline fails because of clippy issue fixed (separately) both in #148 and #149.

@tylerfanelli
Copy link
Collaborator

Is this still in progress?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants