diff options
author | Pavel Begunkov <asml.silence@gmail.com> | 2021-06-17 18:14:10 +0100 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2021-06-18 09:22:02 -0600 |
commit | 7a778f9dc32deae4f748903f6f9169dc01cbcd28 (patch) | |
tree | aad8d51857fdf98156e74af21b1c2c2e02fc0e32 /lib/test_fortify/write_overflow-memcpy.c | |
parent | 16f72070386fca59312bde696cff917bb04b183e (diff) |
io_uring: improve in tctx_task_work() resubmission
If task_state is cleared, io_req_task_work_add() will go the slow path
adding a task_work, setting the task_state, waking up the task and so
on. Not to mention it's expensive. tctx_task_work() first clears the
state and then executes all the work items queued, so if any of them
resubmits or adds new task_work items, it would unnecessarily go through
the slow path of io_req_task_work_add().
Let's clear the ->task_state at the end. We still have to check
->task_list for emptiness afterward to synchronise with
io_req_task_work_add(), do that, and set the state back if we're going
to retry, because clearing not-ours task_state on the next iteration
would be buggy.
Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/1ef72cdac7022adf0cd7ce4bfe3bb5c82a62eb93.1623949695.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'lib/test_fortify/write_overflow-memcpy.c')
0 files changed, 0 insertions, 0 deletions