diff options
author | Jens Axboe <axboe@kernel.dk> | 2021-08-10 15:15:25 -0600 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2021-08-23 13:10:32 -0600 |
commit | 8ef12efe26c8e44323011e57753b8c0e87af1582 (patch) | |
tree | dc950ff582e6b1ede6a7331d0860b407af3ee0da /fs/io_uring.c | |
parent | 89b263f6d56e683ddcf7643140271ef6e36c72b9 (diff) | |
download | lwn-8ef12efe26c8e44323011e57753b8c0e87af1582.tar.gz lwn-8ef12efe26c8e44323011e57753b8c0e87af1582.zip |
io_uring: run regular file completions from task_work
This is in preparation to making the completion lock work outside of
hard/soft IRQ context.
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'fs/io_uring.c')
-rw-r--r-- | fs/io_uring.c | 31 |
1 files changed, 24 insertions, 7 deletions
diff --git a/fs/io_uring.c b/fs/io_uring.c index 8a1c461559ac..6477b0d52c19 100644 --- a/fs/io_uring.c +++ b/fs/io_uring.c @@ -2482,31 +2482,48 @@ static bool io_rw_should_reissue(struct io_kiocb *req) } #endif -static void __io_complete_rw(struct io_kiocb *req, long res, long res2, - unsigned int issue_flags) +static bool __io_complete_rw_common(struct io_kiocb *req, long res) { - int cflags = 0; - if (req->rw.kiocb.ki_flags & IOCB_WRITE) kiocb_end_write(req); if (res != req->result) { if ((res == -EAGAIN || res == -EOPNOTSUPP) && io_rw_should_reissue(req)) { req->flags |= REQ_F_REISSUE; - return; + return true; } req_set_fail(req); + req->result = res; } + return false; +} + +static void io_req_task_complete(struct io_kiocb *req) +{ + int cflags = 0; + if (req->flags & REQ_F_BUFFER_SELECTED) cflags = io_put_rw_kbuf(req); - __io_req_complete(req, issue_flags, res, cflags); + __io_req_complete(req, 0, req->result, cflags); +} + +static void __io_complete_rw(struct io_kiocb *req, long res, long res2, + unsigned int issue_flags) +{ + if (__io_complete_rw_common(req, res)) + return; + io_req_task_complete(req); } static void io_complete_rw(struct kiocb *kiocb, long res, long res2) { struct io_kiocb *req = container_of(kiocb, struct io_kiocb, rw.kiocb); - __io_complete_rw(req, res, res2, 0); + if (__io_complete_rw_common(req, res)) + return; + req->result = res; + req->io_task_work.func = io_req_task_complete; + io_req_task_work_add(req); } static void io_complete_rw_iopoll(struct kiocb *kiocb, long res, long res2) |