Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Accept multi-descriptor TX packets in vsock - #7680

Merged
liuw merged 1 commit into
cloud-hypervisor:mainfrom
liuw:vsock-large-packets
Feb 11, 2026
Merged

liuw merged 1 commit into
cloud-hypervisor:mainfrom
liuw:vsock-large-packets

Conversation

@liuw

@liuw liuw commented Feb 10, 2026

Copy link
Copy Markdown
Member

It is reported that Kata UVM sends large TX packets which are broken into multiple data descriptors. The current implementation is broken.

This patch only handles the guest TX path. RX path is left to another day.

Fixes: #7672

@liuw
liuw requested a review from a team as a code owner February 10, 2026 00:55
@liuw
liuw force-pushed the vsock-large-packets branch 3 times, most recently from 9586743 to e0f1e6c Compare February 10, 2026 01:07

@phip1611 phip1611 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Code-wise LGTM. Unfortunately, I don't have sufficient domain background to review this aspect in depth.

Comment thread virtio-devices/src/vsock/packet.rs Outdated
Comment thread virtio-devices/src/vsock/packet.rs Outdated
Since kernel commit 6693731487a8 ("vsock/virtio: Allocate nonlinear SKBs
for handling large transmit buffers"), a large vsock packet can be split
into multiple descriptors.

If we encounter such TX packets, pull the content into an owned buffer.

Fixes: cloud-hypervisor#7672
Signed-off-by: Wei Liu <[email protected]>
@liuw
liuw force-pushed the vsock-large-packets branch from e0f1e6c to 13a5d9b Compare February 10, 2026 16:11
@liuw

liuw commented Feb 10, 2026

Copy link
Copy Markdown
Member Author

Thank you for the comments.

I will merge this myself. I need to ask my contact to test this upstream version.

@liuw
liuw added this pull request to the merge queue Feb 10, 2026
Merged via the queue into cloud-hypervisor:main with commit bf6f0f8 Feb 11, 2026
43 checks passed
@likebreath likebreath added the bug-fix Bug fix to include in release notes label Feb 19, 2026
@liuw
liuw deleted the vsock-large-packets branch February 25, 2026 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix Bug fix to include in release notes

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

Vsock breaks with newer kernel when it sends large packets

4 participants