Skip to content

Add sendmmsg #2

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

Merged
merged 1 commit into from
Jun 13, 2025
Merged

Add sendmmsg #2

merged 1 commit into from
Jun 13, 2025

Conversation

Hasan6979
Copy link
Collaborator

@Hasan6979 Hasan6979 commented May 22, 2025

This PR adds support in socket2 crate for *mmsg syscalls which would allow to send/recv multiple messages in a single syscall.

Since there already is a draft PR on the same issue in socket2 repo of someone else, hence it doesn't make sense to add another one there. Not sure when that PR would be merged into main, hence for now keeping a separate repo to further neptun improvement optimizations.

Copy link

@jjanowsk jjanowsk left a comment

Choose a reason for hiding this comment

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

+1

Copy link

@LukasPukenis LukasPukenis left a comment

Choose a reason for hiding this comment

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

+1 The existing comments I added are no blockers so can be resolved optionally

@Hasan6979 Hasan6979 merged commit 9946af2 into master Jun 13, 2025
42 checks passed
@Hasan6979 Hasan6979 deleted the Add_sendmmsg branch June 13, 2025 09:19
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