]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
io_uring: fix manual setup of iov_iter for fixed buffers
authorAleix Roca Nonell <aleix.rocanonell@bsc.es>
Thu, 15 Aug 2019 12:03:22 +0000 (14:03 +0200)
committerJens Axboe <axboe@kernel.dk>
Thu, 15 Aug 2019 17:03:38 +0000 (11:03 -0600)
Commit bd11b3a391e3 ("io_uring: don't use iov_iter_advance() for fixed
buffers") introduced an optimization to avoid using the slow
iov_iter_advance by manually populating the iov_iter iterator in some
cases.

However, the computation of the iterator count field was erroneous: The
first bvec was always accounted for an extent of page size even if the
bvec length was smaller.

In consequence, some I/O operations on fixed buffers were unable to
operate on the full extent of the buffer, consistently skipping some
bytes at the end of it.

Fixes: bd11b3a391e3 ("io_uring: don't use iov_iter_advance() for fixed buffers")
Cc: stable@vger.kernel.org
Signed-off-by: Aleix Roca Nonell <aleix.rocanonell@bsc.es>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
fs/io_uring.c

index d542f1cf4428ed79af62c76273d32885490f72ff..aa25b5bbd4aee2b62d0e071b5650428113a34bda 100644 (file)
@@ -1097,10 +1097,8 @@ static int io_import_fixed(struct io_ring_ctx *ctx, int rw,
 
                        iter->bvec = bvec + seg_skip;
                        iter->nr_segs -= seg_skip;
-                       iter->count -= (seg_skip << PAGE_SHIFT);
+                       iter->count -= bvec->bv_len + offset;
                        iter->iov_offset = offset & ~PAGE_MASK;
-                       if (iter->iov_offset)
-                               iter->count -= iter->iov_offset;
                }
        }