diff options
author | Jens Axboe <axboe@kernel.dk> | 2020-07-04 16:55:50 +0200 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2020-07-04 21:44:45 +0200 |
commit | b7db41c9e03b5189bc94993bd50e4506ac9e34c1 (patch) | |
tree | 66de3253898e9fc1128e7c7e7ed326bfd378ff9b /scripts/sorttable.h | |
parent | io_uring: use signal based task_work running (diff) | |
download | linux-b7db41c9e03b5189bc94993bd50e4506ac9e34c1.tar.xz linux-b7db41c9e03b5189bc94993bd50e4506ac9e34c1.zip |
io_uring: fix regression with always ignoring signals in io_cqring_wait()
When switching to TWA_SIGNAL for task_work notifications, we also made
any signal based condition in io_cqring_wait() return -ERESTARTSYS.
This breaks applications that rely on using signals to abort someone
waiting for events.
Check if we have a signal pending because of queued task_work, and
repeat the signal check once we've run the task_work. This provides a
reliable way of telling the two apart.
Additionally, only use TWA_SIGNAL if we are using an eventfd. If not,
we don't have the dependency situation described in the original commit,
and we can get by with just using TWA_RESUME like we previously did.
Fixes: ce593a6c480a ("io_uring: use signal based task_work running")
Cc: stable@vger.kernel.org # v5.7
Reported-by: Andres Freund <andres@anarazel.de>
Tested-by: Andres Freund <andres@anarazel.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'scripts/sorttable.h')
0 files changed, 0 insertions, 0 deletions