io_uring: break iopolling on signal
authorPavel Begunkov <asml.silence@gmail.com>
Wed, 9 Aug 2023 15:20:21 +0000 (16:20 +0100)
committerJens Axboe <axboe@kernel.dk>
Wed, 9 Aug 2023 16:46:46 +0000 (10:46 -0600)
Don't keep spinning iopoll with a signal set. It'll eventually return
back, e.g. by virtue of need_resched(), but it's not a nice user
experience.

Cc: stable@vger.kernel.org
Fixes: def596e9557c9 ("io_uring: support for IO polling")
Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/eeba551e82cad12af30c3220125eb6cb244cc94c.1691594339.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
io_uring/io_uring.c

index d088890..ad4ffd3 100644 (file)
@@ -1673,6 +1673,9 @@ static int io_iopoll_check(struct io_ring_ctx *ctx, long min)
                        break;
                nr_events += ret;
                ret = 0;
+
+               if (task_sigpending(current))
+                       return -EINTR;
        } while (nr_events < min && !need_resched());
 
        return ret;