]> www.infradead.org Git - nvme.git/commit
io_uring/net: don't clear msg_inq before io_recv_buf_select() needs it
authorJens Axboe <axboe@kernel.dk>
Tue, 2 Jul 2024 15:37:30 +0000 (09:37 -0600)
committerJens Axboe <axboe@kernel.dk>
Tue, 2 Jul 2024 15:42:10 +0000 (09:42 -0600)
commit6e92c646f5a4230d939a0882f879fc50dfa116c5
tree89e3e0cdb4b395d7c2c9fa5ac240e29a14b4fef5
parentdbcabac138fdfc730ba458ed2199ff1f29a271fc
io_uring/net: don't clear msg_inq before io_recv_buf_select() needs it

For bundle receives to function properly, the previous iteration msg_inq
value is needed to make a judgement call on how much data there is to
receive. A previous fix ended up clearing it earlier as an error case
would potentially errantly set IORING_CQE_F_SOCK_NONEMPTY if the request
got failed.

Move the assignment to post assigning buffers for the receive, but
ensure that it's cleared for the buffer selection error case. With that,
buffer selection has the right msg_inq value and can correctly bundle
receives as designed.

Noticed while testing where it was apparent than more than 1 buffer was
never received. After fix was in place, multiple buffers are correctly
picked for receive. This provides a 10x speedup for the test case, as
the buffer size used was 64b.

Fixes: 18414a4a2eab ("io_uring/net: assign kmsg inq/flags before buffer selection")
Signed-off-by: Jens Axboe <axboe@kernel.dk>
io_uring/net.c