summaryrefslogtreecommitdiffstats
path: root/scripts
diff options
context:
space:
mode:
authorJens Axboe <axboe@kernel.dk>2024-07-02 17:37:30 +0200
committerJens Axboe <axboe@kernel.dk>2024-07-02 17:42:10 +0200
commit6e92c646f5a4230d939a0882f879fc50dfa116c5 (patch)
tree89e3e0cdb4b395d7c2c9fa5ac240e29a14b4fef5 /scripts
parentio_uring: signal SQPOLL task_work with TWA_SIGNAL_NO_IPI (diff)
downloadlinux-6e92c646f5a4230d939a0882f879fc50dfa116c5.tar.xz
linux-6e92c646f5a4230d939a0882f879fc50dfa116c5.zip
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>
Diffstat (limited to 'scripts')
0 files changed, 0 insertions, 0 deletions