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

Passt support #56

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Passt support #56

wants to merge 8 commits into from

Conversation

mtjhrc
Copy link
Collaborator

@mtjhrc mtjhrc commented Nov 8, 2023

No description provided.

This seems to catch more problems, such as the u8/i8 mismatch on
aarch64 platforms.

Signed-off-by: Matej Hrica <[email protected]>
We need #[allow(unused_mut)], to disable warnings on other targets.

Signed-off-by: Matej Hrica <[email protected]>
You could have a trait and uniform interface for the run method,
but it isn't necessary for now. It also allows passing more arguments
from main.

Signed-off-by: Matej Hrica <[email protected]>
Signed-off-by: Matej Hrica <[email protected]>
@mtjhrc mtjhrc force-pushed the passt-support branch 5 times, most recently from 46bc7c6 to 2755c06 Compare November 9, 2023 10:43
This required adding another field to the config. This is done by
migrating the old config to a newer version.
We are backwards compatible on configuration but not forwards compatible
(older versions of krunvm will not be able to use the config from
this version)
If we want forward compatibility, I feel like we need to ditch the confy
crate.

Signed-off-by: Matej Hrica <[email protected]>
@mtjhrc mtjhrc marked this pull request as ready for review November 22, 2023 09:07
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.

1 participant