io_uring: clean up io_prep_rw_setup()

Remove unnecessary call to iov_iter_save_state() in io_prep_rw_setup()
as io_import_iovec() already does this. Then the result from
io_import_iovec() can be returned directly.

Signed-off-by: David Wei <dw@davidwei.uk>
Reviewed-by: Anuj Gupta <anuj20.g@samsung.com>
Tested-by: Li Zetao <lizetao1@huawei.com>
Link: https://lore.kernel.org/r/20241207004144.783631-1-dw@davidwei.uk
Signed-off-by: Jens Axboe <axboe@kernel.dk>

diff --git a/io_uring/rw.c b/io_uring/rw.c
index 04e4467ab0ee8317a52070411b34392bfff9fedd..5b24fd8b69f620a78391d37e2b4cc76188cd6754 100644 (file)
--- a/io_uring/rw.c
+++ b/io_uring/rw.c
@@ -240,7 +240,6 @@ done:
 static int io_prep_rw_setup(struct io_kiocb *req, int ddir, bool do_import)
 {
        struct io_async_rw *rw;
-       int ret;
 
        if (io_rw_alloc_async(req))
                return -ENOMEM;
@@ -249,12 +248,7 @@ static int io_prep_rw_setup(struct io_kiocb *req, int ddir, bool do_import)
                return 0;
 
        rw = req->async_data;
-       ret = io_import_iovec(ddir, req, rw, 0);
-       if (unlikely(ret < 0))
-               return ret;
-
-       iov_iter_save_state(&rw->iter, &rw->iter_state);
-       return 0;
+       return io_import_iovec(ddir, req, rw, 0);
 }
 
 static inline void io_meta_save_state(struct io_async_rw *io)