]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
io_uring: use the right type for work_llist empty check
authorJens Axboe <axboe@kernel.dk>
Tue, 26 Mar 2024 00:53:33 +0000 (18:53 -0600)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 30 May 2024 07:48:58 +0000 (09:48 +0200)
[ Upstream commit 22537c9f79417fed70b352d54d01d2586fee9521 ]

io_task_work_pending() uses wq_list_empty() on ctx->work_llist, but it's
not an io_wq_work_list, it's a struct llist_head. They both have
->first as head-of-list, and it turns out the checks are identical. But
be proper and use the right helper.

Fixes: dac6a0eae793 ("io_uring: ensure iopoll runs local task work as well")
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Signed-off-by: Sasha Levin <sashal@kernel.org>
io_uring/io_uring.h

index d5495710c17877624c75d8fa36b71af9535336a3..c0cff024db7b443988db224477aff4006b59d5ed 100644 (file)
@@ -301,7 +301,7 @@ static inline int io_run_task_work(void)
 
 static inline bool io_task_work_pending(struct io_ring_ctx *ctx)
 {
-       return task_work_pending(current) || !wq_list_empty(&ctx->work_llist);
+       return task_work_pending(current) || !llist_empty(&ctx->work_llist);
 }
 
 static inline void io_tw_lock(struct io_ring_ctx *ctx, struct io_tw_state *ts)