Commit 13a99017 authored by Pavel Begunkov's avatar Pavel Begunkov Committed by Jens Axboe

io_uring: remove events caching atavisms

Remove events argument from *io_poll_execute(), it's not needed and not
used.
Signed-off-by: default avatarPavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/12efd4e15c6a90cf9e5b59807cfcb57852b51dc7.1655990418.git.asml.silence@gmail.comSigned-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent 0638cd7b
...@@ -289,8 +289,7 @@ static void io_apoll_task_func(struct io_kiocb *req, bool *locked) ...@@ -289,8 +289,7 @@ static void io_apoll_task_func(struct io_kiocb *req, bool *locked)
io_req_complete_failed(req, ret); io_req_complete_failed(req, ret);
} }
static void __io_poll_execute(struct io_kiocb *req, int mask, static void __io_poll_execute(struct io_kiocb *req, int mask)
__poll_t __maybe_unused events)
{ {
io_req_set_res(req, mask, 0); io_req_set_res(req, mask, 0);
/* /*
...@@ -308,18 +307,17 @@ static void __io_poll_execute(struct io_kiocb *req, int mask, ...@@ -308,18 +307,17 @@ static void __io_poll_execute(struct io_kiocb *req, int mask,
io_req_task_work_add(req); io_req_task_work_add(req);
} }
static inline void io_poll_execute(struct io_kiocb *req, int res, static inline void io_poll_execute(struct io_kiocb *req, int res)
__poll_t events)
{ {
if (io_poll_get_ownership(req)) if (io_poll_get_ownership(req))
__io_poll_execute(req, res, events); __io_poll_execute(req, res);
} }
static void io_poll_cancel_req(struct io_kiocb *req) static void io_poll_cancel_req(struct io_kiocb *req)
{ {
io_poll_mark_cancelled(req); io_poll_mark_cancelled(req);
/* kick tw, which should complete the request */ /* kick tw, which should complete the request */
io_poll_execute(req, 0, 0); io_poll_execute(req, 0);
} }
#define IO_ASYNC_POLL_COMMON (EPOLLONESHOT | EPOLLPRI) #define IO_ASYNC_POLL_COMMON (EPOLLONESHOT | EPOLLPRI)
...@@ -334,7 +332,7 @@ static int io_poll_wake(struct wait_queue_entry *wait, unsigned mode, int sync, ...@@ -334,7 +332,7 @@ static int io_poll_wake(struct wait_queue_entry *wait, unsigned mode, int sync,
if (unlikely(mask & POLLFREE)) { if (unlikely(mask & POLLFREE)) {
io_poll_mark_cancelled(req); io_poll_mark_cancelled(req);
/* we have to kick tw in case it's not already */ /* we have to kick tw in case it's not already */
io_poll_execute(req, 0, poll->events); io_poll_execute(req, 0);
/* /*
* If the waitqueue is being freed early but someone is already * If the waitqueue is being freed early but someone is already
...@@ -369,7 +367,7 @@ static int io_poll_wake(struct wait_queue_entry *wait, unsigned mode, int sync, ...@@ -369,7 +367,7 @@ static int io_poll_wake(struct wait_queue_entry *wait, unsigned mode, int sync,
else else
req->flags &= ~REQ_F_SINGLE_POLL; req->flags &= ~REQ_F_SINGLE_POLL;
} }
__io_poll_execute(req, mask, poll->events); __io_poll_execute(req, mask);
} }
return 1; return 1;
} }
...@@ -487,7 +485,7 @@ static int __io_arm_poll_handler(struct io_kiocb *req, ...@@ -487,7 +485,7 @@ static int __io_arm_poll_handler(struct io_kiocb *req,
req->apoll_events |= EPOLLONESHOT; req->apoll_events |= EPOLLONESHOT;
ipt->error = 0; ipt->error = 0;
} }
__io_poll_execute(req, mask, poll->events); __io_poll_execute(req, mask);
return 0; return 0;
} }
...@@ -497,7 +495,7 @@ static int __io_arm_poll_handler(struct io_kiocb *req, ...@@ -497,7 +495,7 @@ static int __io_arm_poll_handler(struct io_kiocb *req,
*/ */
v = atomic_dec_return(&req->poll_refs); v = atomic_dec_return(&req->poll_refs);
if (unlikely(v & IO_POLL_REF_MASK)) if (unlikely(v & IO_POLL_REF_MASK))
__io_poll_execute(req, 0, poll->events); __io_poll_execute(req, 0);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment