Commit 553deffd authored by Pavel Begunkov's avatar Pavel Begunkov Committed by Jens Axboe
Browse files

io_uring: don't pass state to io_submit_state_end 



Submission state and ctx and coupled together, no need to passs

Signed-off-by: default avatarPavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/e22d77a5786ef77e0c49b933ad74bae55cfb6ca6.1632516769.git.asml.silence@gmail.com


Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent 1cce17ac

fs/io_uring.c

+5 −3
Original line number Diff line number Diff line
@@ -7137,11 +7137,13 @@ static int io_submit_sqe(struct io_ring_ctx *ctx, struct io_kiocb *req, 
/*

 * Batched submission is done, ensure local IO is flushed out.

 */

static void io_submit_state_end(struct io_submit_state *state,

				struct io_ring_ctx *ctx)

static void io_submit_state_end(struct io_ring_ctx *ctx)

{

	struct io_submit_state *state = &ctx->submit_state;



	if (state->link.head)

		io_queue_sqe(state->link.head);

	/* flush only after queuing links as they can generate completions */

	io_submit_flush_completions(ctx);

	if (state->plug_started)

		blk_finish_plug(&state->plug);
@@ -7244,7 +7246,7 @@ static int io_submit_sqes(struct io_ring_ctx *ctx, unsigned int nr) 
		percpu_ref_put_many(&ctx->refs, unused);

	}



	io_submit_state_end(&ctx->submit_state, ctx);

	io_submit_state_end(ctx);

	 /* Commit SQ ring head once we've consumed and submitted all SQEs */

	io_commit_sqring(ctx);