io_uring/poll: attempt request issue after racy poll wakeup
authorJens Axboe <axboe@kernel.dk>
Tue, 10 Jan 2023 17:44:37 +0000 (10:44 -0700)
committerJens Axboe <axboe@kernel.dk>
Thu, 12 Jan 2023 17:35:51 +0000 (10:35 -0700)
commit6e5aedb9324aab1c14a23fae3d8eeb64a679c20e
tree06e7fe8c6c4aaaaf2a3059e1d2cc6d4956a20794
parentea97cbebaf861d99c3e892275147e6fca6d2c1ca
io_uring/poll: attempt request issue after racy poll wakeup

If we have multiple requests waiting on the same target poll waitqueue,
then it's quite possible to get a request triggered and get disappointed
in not being able to make any progress with it. If we race in doing so,
we'll potentially leave the poll request on the internal tables, but
removed from the waitqueue. That means that any subsequent trigger of
the poll waitqueue will not kick that request into action, causing an
application to potentially wait for completion of a request that will
never happen.

Fix this by adding a new poll return state, IOU_POLL_REISSUE. Rather
than have complicated logic for how to re-arm a given type of request,
just punt it for a reissue.

While in there, move the 'ret' variable to the only section where it
gets used. This avoids confusion the scope of it.

Cc: stable@vger.kernel.org
Fixes: eb0089d629ba ("io_uring: single shot poll removal optimisation")
Signed-off-by: Jens Axboe <axboe@kernel.dk>
io_uring/poll.c