diff options
| author | Pavel Begunkov <asml.silence@gmail.com> | 2025-07-16 17:20:17 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2025-07-24 08:51:48 +0200 |
| commit | 283bace0409504e3f60df72eeaef53d014f573ee (patch) | |
| tree | f5d687db18f5e3c338da68cdb6e14a0ce3bf911c /io_uring/poll.c | |
| parent | 7803b28c9aa8d8bd4e19ebcf5f0db9612b0f333b (diff) | |
| download | linux-283bace0409504e3f60df72eeaef53d014f573ee.tar.gz linux-283bace0409504e3f60df72eeaef53d014f573ee.tar.bz2 linux-283bace0409504e3f60df72eeaef53d014f573ee.zip | |
io_uring/poll: fix POLLERR handling
commit c7cafd5b81cc07fb402e3068d134c21e60ea688c upstream.
8c8492ca64e7 ("io_uring/net: don't retry connect operation on EPOLLERR")
is a little dirty hack that
1) wrongfully assumes that POLLERR equals to a failed request, which
breaks all POLLERR users, e.g. all error queue recv interfaces.
2) deviates the connection request behaviour from connect(2), and
3) racy and solved at a wrong level.
Nothing can be done with 2) now, and 3) is beyond the scope of the
patch. At least solve 1) by moving the hack out of generic poll handling
into io_connect().
Cc: stable@vger.kernel.org
Fixes: 8c8492ca64e79 ("io_uring/net: don't retry connect operation on EPOLLERR")
Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/3dc89036388d602ebd84c28e5042e457bdfc952b.1752682444.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'io_uring/poll.c')
| -rw-r--r-- | io_uring/poll.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/io_uring/poll.c b/io_uring/poll.c index bbdc9a7624a1..ab27a627fd4c 100644 --- a/io_uring/poll.c +++ b/io_uring/poll.c @@ -288,8 +288,6 @@ static int io_poll_check_events(struct io_kiocb *req, bool *locked) return IOU_POLL_REISSUE; } } - if (unlikely(req->cqe.res & EPOLLERR)) - req_set_fail(req); if (req->apoll_events & EPOLLONESHOT) return IOU_POLL_DONE; if (io_is_uring_fops(req->file)) |