io_uring: have io_file_put() take an io_kiocb rather than the file
authorJens Axboe <axboe@kernel.dk>
Fri, 7 Jul 2023 17:14:40 +0000 (11:14 -0600)
committerJens Axboe <axboe@kernel.dk>
Thu, 10 Aug 2023 16:27:46 +0000 (10:27 -0600)
No functional changes in this patch, just a prep patch for needing the
request in io_file_put().

Signed-off-by: Jens Axboe <axboe@kernel.dk>
io_uring/io_uring.c
io_uring/io_uring.h

index dadd745..15697d8 100644 (file)
@@ -998,8 +998,7 @@ static void __io_req_complete_post(struct io_kiocb *req, unsigned issue_flags)
                io_put_kbuf_comp(req);
                if (unlikely(req->flags & IO_REQ_CLEAN_FLAGS))
                        io_clean_op(req);
-               if (!(req->flags & REQ_F_FIXED_FILE))
-                       io_put_file(req->file);
+               io_put_file(req);
 
                rsrc_node = req->rsrc_node;
                /*
@@ -1533,8 +1532,7 @@ void io_free_batch_list(struct io_ring_ctx *ctx, struct io_wq_work_node *node)
                        if (unlikely(req->flags & IO_REQ_CLEAN_FLAGS))
                                io_clean_op(req);
                }
-               if (!(req->flags & REQ_F_FIXED_FILE))
-                       io_put_file(req->file);
+               io_put_file(req);
 
                io_req_put_rsrc_locked(req, ctx);
 
index 12769ba..ff153af 100644 (file)
@@ -196,10 +196,10 @@ static inline bool req_has_async_data(struct io_kiocb *req)
        return req->flags & REQ_F_ASYNC_DATA;
 }
 
-static inline void io_put_file(struct file *file)
+static inline void io_put_file(struct io_kiocb *req)
 {
-       if (file)
-               fput(file);
+       if (!(req->flags & REQ_F_FIXED_FILE) && req->file)
+               fput(req->file);
 }
 
 static inline void io_ring_submit_unlock(struct io_ring_ctx *ctx,