diff options
author | Pavel Begunkov <asml.silence@gmail.com> | 2021-12-15 22:08:44 +0000 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2021-12-28 09:51:14 -0800 |
commit | e840b4baf3cfb37e2ead4f649a45bb78178677ff (patch) | |
tree | 9b10995b9927df1ab03a1fcdc2e1f575ec2f1678 /fs/io_uring.c | |
parent | 33ce2aff7d340bf48875ccd80628c884cf8017ae (diff) | |
download | lwn-e840b4baf3cfb37e2ead4f649a45bb78178677ff.tar.gz lwn-e840b4baf3cfb37e2ead4f649a45bb78178677ff.zip |
io_uring: remove double poll on poll update
Before updating a poll request we should remove it from poll queues,
including the double poll entry.
Fixes: b69de288e913 ("io_uring: allow events and user_data update of running poll requests")
Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/ac39e7f80152613603b8a6cc29a2b6063ac2434f.1639605189.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'fs/io_uring.c')
-rw-r--r-- | fs/io_uring.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/io_uring.c b/fs/io_uring.c index 1f2341d87588..39d50124bdea 100644 --- a/fs/io_uring.c +++ b/fs/io_uring.c @@ -5980,6 +5980,7 @@ static int io_poll_update(struct io_kiocb *req, unsigned int issue_flags) * update those. For multishot, if we're racing with completion, just * let completion re-add it. */ + io_poll_remove_double(preq); completing = !__io_poll_remove_one(preq, &preq->poll, false); if (completing && (preq->poll.events & EPOLLONESHOT)) { ret = -EALREADY; |