io_uring/rw: get rid of using req->imu
authorJens Axboe <axboe@kernel.dk>
Wed, 16 Oct 2024 21:49:49 +0000 (15:49 -0600)
committerJens Axboe <axboe@kernel.dk>
Tue, 29 Oct 2024 19:43:27 +0000 (13:43 -0600)
It's assigned in the same function that it's being used, get rid of
it. A local variable will do just fine.

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

index 354c4e175654cf74d4e61933f5fd2bd4efd8efc8..d8b9e7a712f61410b0243218e6b9853737aefc36 100644 (file)
@@ -330,6 +330,7 @@ static int io_prep_rw_fixed(struct io_kiocb *req, const struct io_uring_sqe *sqe
 {
        struct io_rw *rw = io_kiocb_to_cmd(req, struct io_rw);
        struct io_ring_ctx *ctx = req->ctx;
+       struct io_mapped_ubuf *imu;
        struct io_async_rw *io;
        u16 index;
        int ret;
@@ -341,11 +342,11 @@ static int io_prep_rw_fixed(struct io_kiocb *req, const struct io_uring_sqe *sqe
        if (unlikely(req->buf_index >= ctx->nr_user_bufs))
                return -EFAULT;
        index = array_index_nospec(req->buf_index, ctx->nr_user_bufs);
-       req->imu = ctx->user_bufs[index];
+       imu = ctx->user_bufs[index];
        io_req_set_rsrc_node(req, ctx, 0);
 
        io = req->async_data;
-       ret = io_import_fixed(ddir, &io->iter, req->imu, rw->addr, rw->len);
+       ret = io_import_fixed(ddir, &io->iter, imu, rw->addr, rw->len);
        iov_iter_save_state(&io->iter, &io->iter_state);
        return ret;
 }