Commit 4bc4494e authored by Pavel Begunkov's avatar Pavel Begunkov Committed by Jens Axboe

io_uring: remove extra nxt check after punt

After __io_queue_sqe() ended up in io_queue_async_work(), it's already
known that there is no @nxt req, so skip the check and return from the
function.

Also, @nxt initialisation now can be done just before
io_put_req_find_next(), as there is no jumping until it's checked.
Signed-off-by: default avatarPavel Begunkov <asml.silence@gmail.com>
Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent d7718a9d
...@@ -4936,7 +4936,7 @@ static struct io_kiocb *io_prep_linked_timeout(struct io_kiocb *req) ...@@ -4936,7 +4936,7 @@ static struct io_kiocb *io_prep_linked_timeout(struct io_kiocb *req)
static void __io_queue_sqe(struct io_kiocb *req, const struct io_uring_sqe *sqe) static void __io_queue_sqe(struct io_kiocb *req, const struct io_uring_sqe *sqe)
{ {
struct io_kiocb *linked_timeout; struct io_kiocb *linked_timeout;
struct io_kiocb *nxt = NULL; struct io_kiocb *nxt;
const struct cred *old_creds = NULL; const struct cred *old_creds = NULL;
int ret; int ret;
...@@ -4963,7 +4963,7 @@ static void __io_queue_sqe(struct io_kiocb *req, const struct io_uring_sqe *sqe) ...@@ -4963,7 +4963,7 @@ static void __io_queue_sqe(struct io_kiocb *req, const struct io_uring_sqe *sqe)
if (io_arm_poll_handler(req)) { if (io_arm_poll_handler(req)) {
if (linked_timeout) if (linked_timeout)
io_queue_linked_timeout(linked_timeout); io_queue_linked_timeout(linked_timeout);
goto done_req; goto exit;
} }
punt: punt:
if (io_op_defs[req->opcode].file_table) { if (io_op_defs[req->opcode].file_table) {
...@@ -4977,10 +4977,11 @@ static void __io_queue_sqe(struct io_kiocb *req, const struct io_uring_sqe *sqe) ...@@ -4977,10 +4977,11 @@ static void __io_queue_sqe(struct io_kiocb *req, const struct io_uring_sqe *sqe)
* submit reference when the iocb is actually submitted. * submit reference when the iocb is actually submitted.
*/ */
io_queue_async_work(req); io_queue_async_work(req);
goto done_req; goto exit;
} }
err: err:
nxt = NULL;
/* drop submission reference */ /* drop submission reference */
io_put_req_find_next(req, &nxt); io_put_req_find_next(req, &nxt);
...@@ -4997,15 +4998,14 @@ static void __io_queue_sqe(struct io_kiocb *req, const struct io_uring_sqe *sqe) ...@@ -4997,15 +4998,14 @@ static void __io_queue_sqe(struct io_kiocb *req, const struct io_uring_sqe *sqe)
req_set_fail_links(req); req_set_fail_links(req);
io_put_req(req); io_put_req(req);
} }
done_req:
if (nxt) { if (nxt) {
req = nxt; req = nxt;
nxt = NULL;
if (req->flags & REQ_F_FORCE_ASYNC) if (req->flags & REQ_F_FORCE_ASYNC)
goto punt; goto punt;
goto again; goto again;
} }
exit:
if (old_creds) if (old_creds)
revert_creds(old_creds); revert_creds(old_creds);
} }
......
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