diff options
| author |  Jens Axboe <axboe@kernel.dk>
| 2024-04-17 09:23:55 -0600 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk>
| 2024-04-17 09:23:55 -0600 |
| commit | df604d2ad480fcf7b39767280c9093e13b1de952 (patch) | |
| tree | 10fa5144873d57d4cfd6f56af02e40e3585b31f6 /io_uring/io_uring.h | |
| parent | io-wq: Drop intermediate step between pending list and active work (diff) | |
| download | linux-df604d2ad480fcf7b39767280c9093e13b1de952.tar.gz linux-df604d2ad480fcf7b39767280c9093e13b1de952.tar.bz2 linux-df604d2ad480fcf7b39767280c9093e13b1de952.zip | |
io_uring/rw: ensure retry condition isn't lost
A previous commit removed the checking on whether or not it was possible
to retry a request, since it's now possible to retry any of them. This
would previously have caused the request to have been ended with an error,
but now the retry condition can simply get lost instead.
Cleanup the retry handling and always just punt it to task_work, which
will queue it with io-wq appropriately.
Reported-by: Changhui Zhong <czhong@redhat.com>
Tested-by: Ming Lei <ming.lei@redhat.com>
Fixes: cca6571381a0 ("io_uring/rw: cleanup retry path")
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'io_uring/io_uring.h')
| -rw-r--r-- | io_uring/io_uring.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/io_uring/io_uring.h b/io_uring/io_uring.h index 624ca9076a50..b83a719c5443 100644 --- a/io_uring/io_uring.h +++ b/io_uring/io_uring.h @@ -75,6 +75,7 @@ struct file *io_file_get_fixed(struct io_kiocb *req, int fd, void __io_req_task_work_add(struct io_kiocb *req, unsigned flags); bool io_alloc_async_data(struct io_kiocb *req); void io_req_task_queue(struct io_kiocb *req); +void io_tw_queue_iowq(struct io_kiocb *req); void io_req_task_complete(struct io_kiocb *req, struct io_tw_state *ts); void io_req_task_queue_fail(struct io_kiocb *req, int ret); void io_req_task_submit(struct io_kiocb *req, struct io_tw_state *ts); |