io_uring: don't setup async context for read/write fixed
authorJens Axboe <axboe@kernel.dk>
Tue, 14 Jan 2020 02:23:24 +0000 (19:23 -0700)
committerJens Axboe <axboe@kernel.dk>
Tue, 14 Jan 2020 02:25:29 +0000 (19:25 -0700)
We don't need it, and if we have it, then the retry handler will attempt
to copy the non-existent iovec with the inline iovec, with a segment
count that doesn't make sense.

Fixes: f67676d160c6 ("io_uring: ensure async punted read/write requests copy iovec")
Reported-by: Jonathan Lemon <jonathan.lemon@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
fs/io_uring.c

index 38b5405..8321c2f 100644 (file)
@@ -1786,6 +1786,9 @@ static int io_setup_async_rw(struct io_kiocb *req, ssize_t io_size,
                             struct iovec *iovec, struct iovec *fast_iov,
                             struct iov_iter *iter)
 {
+       if (req->opcode == IORING_OP_READ_FIXED ||
+           req->opcode == IORING_OP_WRITE_FIXED)
+               return 0;
        if (!req->io && io_alloc_async_ctx(req))
                return -ENOMEM;