diff options
| author | Jens Axboe <axboe@kernel.dk> | 2021-02-23 19:17:35 -0700 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2021-03-20 10:51:11 +0100 |
| commit | 62c8121d230a849670d98f13af0bb1e74e8b9ea8 (patch) | |
| tree | eeaa8e143279e5cc513b513d5e48d4771588353d /fs | |
| parent | dd5ae3523018ed0d6e58591910cd61bc596fc89c (diff) | |
| download | linux-rt-62c8121d230a849670d98f13af0bb1e74e8b9ea8.tar.gz | |
io_uring: don't attempt IO reissue from the ring exit path
[ Upstream commit 7c977a58dc83366e488c217fd88b1469d242bee5 ]
If we're exiting the ring, just let the IO fail with -EAGAIN as nobody
will care anyway. It's not the right context to reissue from.
Cc: stable@vger.kernel.org
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/io_uring.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/fs/io_uring.c b/fs/io_uring.c index 00ef0b90d149..68508f010b90 100644 --- a/fs/io_uring.c +++ b/fs/io_uring.c @@ -2717,6 +2717,13 @@ static bool io_rw_reissue(struct io_kiocb *req, long res) return false; if ((res != -EAGAIN && res != -EOPNOTSUPP) || io_wq_current_is_worker()) return false; + /* + * If ref is dying, we might be running poll reap from the exit work. + * Don't attempt to reissue from that path, just let it fail with + * -EAGAIN. + */ + if (percpu_ref_is_dying(&req->ctx->refs)) + return false; lockdep_assert_held(&req->ctx->uring_lock); |