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

Add support for Vhost-user-frontend #122

Closed
wants to merge 4 commits into from

Conversation

vireshk
Copy link
Contributor

@vireshk vireshk commented Oct 18, 2022

Hello everyone,

This is just a draft review request to show what I have copied over from cloud-hypervisor and I am looking for help to get this merged in vhost workspace.

There is some code here which we aren't using right now for the Xen frontend side, but I copied over anyway to modify as less as possible.

There are no tests available, some help on that from people who wrote this would be great. More feedback welcome

Would be great to get more feedback on this.

Initial discussion around adding it can be found here: rust-vmm/community#133.

@andreeaflorescu @slp @sboeuf @jiangliu @eryugey @stefano-garzarella

This adds a new crate for vhost-user-frontend.

Signed-off-by: Viresh Kumar <[email protected]>
This copies few files and definitions related to frontend side of
vhost-user protocol from cloud hypervisor.

Signed-off-by: Viresh Kumar <[email protected]>
Some of the helpers are not required for every user, and can be dropped.
Remove them and do minor cleanups.

Signed-off-by: Viresh Kumar <[email protected]>
Provide a generic implementation of VirtioDevice, which can be reused by
all sort of devices.

Signed-off-by: Viresh Kumar <[email protected]>
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