diff options
author | David Wei <dw@davidwei.uk> | 2024-11-20 23:14:52 +0100 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2024-11-21 15:11:00 +0100 |
commit | f46b9cdb22f7a167c36b6bcddaef7e8aee2598fa (patch) | |
tree | 77367fc483f6da2618919fbb331a2964cc09324f /io_uring/io_uring.h | |
parent | io_uring: add io_local_work_pending() (diff) | |
download | linux-f46b9cdb22f7a167c36b6bcddaef7e8aee2598fa.tar.xz linux-f46b9cdb22f7a167c36b6bcddaef7e8aee2598fa.zip |
io_uring: limit local tw done
Instead of eagerly running all available local tw, limit the amount of
local tw done to the max of IO_LOCAL_TW_DEFAULT_MAX (20) or wait_nr. The
value of 20 is chosen as a reasonable heuristic to allow enough work
batching but also keep latency down.
Add a retry_llist that maintains a list of local tw that couldn't be
done in time. No synchronisation is needed since it is only modified
within the task context.
Signed-off-by: David Wei <dw@davidwei.uk>
Link: https://lore.kernel.org/r/20241120221452.3762588-3-dw@davidwei.uk
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'io_uring/io_uring.h')
-rw-r--r-- | io_uring/io_uring.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/io_uring/io_uring.h b/io_uring/io_uring.h index 69eb3b23a5a0..12abee607e4a 100644 --- a/io_uring/io_uring.h +++ b/io_uring/io_uring.h @@ -349,7 +349,7 @@ static inline int io_run_task_work(void) static inline bool io_local_work_pending(struct io_ring_ctx *ctx) { - return !llist_empty(&ctx->work_llist); + return !llist_empty(&ctx->work_llist) || !llist_empty(&ctx->retry_llist); } static inline bool io_task_work_pending(struct io_ring_ctx *ctx) |