]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
io_uring: ensure we clear io_kiocb->result before each issue
authorJens Axboe <axboe@kernel.dk>
Wed, 30 Oct 2019 19:53:09 +0000 (13:53 -0600)
committerJens Axboe <axboe@kernel.dk>
Wed, 30 Oct 2019 20:45:22 +0000 (14:45 -0600)
We use io_kiocb->result == -EAGAIN as a way to know if we need to
re-submit a polled request, as -EAGAIN reporting happens out-of-line
for IO submission failures. This field is cleared when we originally
allocate the request, but it isn't reset when we retry the submission
from async context. This can cause issues where we think something
needs a re-issue, but we're really just reading stale data.

Reset ->result whenever we re-prep a request for polled submission.

Cc: stable@vger.kernel.org
Fixes: 9e645e1105ca ("io_uring: add support for sqe links")
Reported-by: Bijan Mottahedeh <bijan.mottahedeh@oracle.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
fs/io_uring.c

index c11c4157a4c22c6c0d718749fd0e68beefda409b..f9a38998f2fc515c94c4cc292e00450a55aaa912 100644 (file)
@@ -1124,6 +1124,7 @@ static int io_prep_rw(struct io_kiocb *req, const struct sqe_submit *s,
 
                kiocb->ki_flags |= IOCB_HIPRI;
                kiocb->ki_complete = io_complete_rw_iopoll;
+               req->result = 0;
        } else {
                if (kiocb->ki_flags & IOCB_HIPRI)
                        return -EINVAL;