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

Setup vsock devices for APPVMs and use it for Waypipe #256

Merged
merged 1 commit into from
Sep 29, 2023

Conversation

nesteroff
Copy link
Contributor

This PR makes Waypipe work over VSOCK instead of UNIX domain sockets. It increases performance by getting rid of the unnecessary overhead caused by forwarding UNIX sockets over SSH. Testing with iperf show that UNIX sockets over SSH can transfer about 4 Gbits/sec on Lenovo X1 Carbon but with VSOCK it can handle about 15 Gbits/sec between GUIVM and APPVMs.

@nesteroff nesteroff force-pushed the waypipe-vsock branch 2 times, most recently from 40bfb97 to 1dff954 Compare September 7, 2023 14:56
@vadika
Copy link
Contributor

vadika commented Sep 8, 2023 via email

@nesteroff
Copy link
Contributor Author

In this particular case APPVMs connect to GUIVM but not the other way around. It means that they only need to know the CID of the GUIVM which is always 3 (first available number).

@nesteroff nesteroff force-pushed the waypipe-vsock branch 6 times, most recently from f6ea378 to 278c33e Compare September 21, 2023 09:45
@nesteroff nesteroff marked this pull request as ready for review September 21, 2023 09:58
Copy link
Contributor

@mikatammi mikatammi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please create repo under tiiuae for vsockproxy. Otherwise LGTM

@mikatammi
Copy link
Contributor

Needs rebase now

Signed-off-by: Yuri Nesterov <[email protected]>
Copy link
Contributor

@mikatammi mikatammi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I will test this first

@mikatammi mikatammi merged commit aaeafbc into tiiuae:main Sep 29, 2023
3 checks passed
@nesteroff nesteroff deleted the waypipe-vsock branch October 5, 2023 12:30
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.

4 participants