mbox series

[RFC,v2,liburing,0/2] multishot recvmsg

Message ID 20220714115428.1569612-1-dylany@fb.com (mailing list archive)
Headers show
Series multishot recvmsg | expand

Message

Dylan Yudaken July 14, 2022, 11:54 a.m. UTC
This series adds an API (patch 1) and a test (#2) for multishot recvmsg.

I have not included docs yet, but I want to get feedback on the API for handling
the result (if there is any).

There was no feedback on v1, but in v2 I added more API to make it easier to deal
with truncated payloads. Additionally there is extra testing for this.

Dylan Yudaken (2):
  add multishot recvmsg API
  add tests for multishot recvmsg

 src/include/liburing.h          |  66 ++++++++++++
 src/include/liburing/io_uring.h |   7 ++
 test/recv-multishot.c           | 180 ++++++++++++++++++++++++++++----
 3 files changed, 234 insertions(+), 19 deletions(-)


base-commit: a705a6b307adfa52ba6315df0b8d5964a3b898b2

Comments

Jens Axboe July 14, 2022, 6:01 p.m. UTC | #1
On 7/14/22 5:54 AM, Dylan Yudaken wrote:
> This series adds an API (patch 1) and a test (#2) for multishot
> recvmsg.
> 
> I have not included docs yet, but I want to get feedback on the API
> for handling the result (if there is any).
> 
> There was no feedback on v1, but in v2 I added more API to make it
> easier to deal with truncated payloads. Additionally there is extra
> testing for this.

I'm going to shove this in for some testing. We have a bit of time to
play with before the next release, might make sense to attempt to mature
it in tree.
Jens Axboe July 14, 2022, 6:02 p.m. UTC | #2
On Thu, 14 Jul 2022 04:54:26 -0700, Dylan Yudaken wrote:
> This series adds an API (patch 1) and a test (#2) for multishot recvmsg.
> 
> I have not included docs yet, but I want to get feedback on the API for handling
> the result (if there is any).
> 
> There was no feedback on v1, but in v2 I added more API to make it easier to deal
> with truncated payloads. Additionally there is extra testing for this.
> 
> [...]

Applied, thanks!

[1/2] add multishot recvmsg API
      commit: 874406f7fb0942238a77041d84f07a480400ed81
[2/2] add tests for multishot recvmsg
      commit: 95dc4789e9dbacb7b3913b1a6e0dfef80fabb74d

Best regards,