rustix icon indicating copy to clipboard operation
rustix copied to clipboard

Add support for sendmmsg(2) on linux

Open colinmarc opened this issue 1 year ago • 2 comments

https://man7.org/linux/man-pages/man2/sendmmsg.2.html

Partially addresses #1156. I would've liked to add recvmmsg in the same PR, but it's actually much more complicated.

colinmarc avatar Sep 18 '24 21:09 colinmarc

My test is unfortunately flaky, because of course the kernel can consolidate stream messages. Will fix tomorrow.

colinmarc avatar Sep 18 '24 21:09 colinmarc

I think I addressed all your comments.

colinmarc avatar Oct 03 '24 11:10 colinmarc

Hi, anything needed from me on this?

colinmarc avatar Jan 29 '25 11:01 colinmarc

Sorry for pinging and then disappearing - was really sick :)

I rebased and addressed your comments. Please let me know how you'd like to resolve the overlap with #1004. It seems like I could use the updated SockAddrAny from that instead of RawSocketAddr.

colinmarc avatar Feb 10 '25 15:02 colinmarc

Rebased on top of #1004 now that it's in main. The test failures look unrelated.

This patch is a lot smaller now!

colinmarc avatar Feb 11 '25 10:02 colinmarc

The FreeBSD CI failure is fixed on main in https://github.com/bytecodealliance/rustix/pull/1322.

sunfishcode avatar Feb 11 '25 14:02 sunfishcode

Thanks!

sunfishcode avatar Feb 11 '25 19:02 sunfishcode

This is now released in rustix 1.0.0.

sunfishcode avatar Mar 07 '25 06:03 sunfishcode