On 1/16/23 18:43, Jens Axboe wrote:
On 1/16/23 9:48 AM, Pavel Begunkov wrote:
io_submit_flush_completions() may queue new requests for tw execution,
especially true for linked requests. Recheck the tw list for emptiness
after flushing completions.
Did you check when it got lost? Would be nice to add a Fixes link?
fwiw, not fan of putting a "Fixes" tag on sth that is not a fix.
Looks like the optimisation was there for normal task_work, then
disappeared in f88262e60bb9c ("io_uring: lockless task list").
DEFERRED_TASKRUN came later and this patch handles exclusively
deferred tw. I probably need to send a patch for normal tw as well.
--
Pavel Begunkov
