Commit caf582c6 authored by Pavel Begunkov's avatar Pavel Begunkov Committed by Jens Axboe

io_uring: optimise commit_sqring() for common case

It should be pretty rare to not submitting anything when there is
something in the ring. No need to keep heuristics for this case.
Signed-off-by: default avatarPavel Begunkov <asml.silence@gmail.com>
Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent ee7d46d9
...@@ -4501,14 +4501,12 @@ static void io_commit_sqring(struct io_ring_ctx *ctx) ...@@ -4501,14 +4501,12 @@ static void io_commit_sqring(struct io_ring_ctx *ctx)
{ {
struct io_rings *rings = ctx->rings; struct io_rings *rings = ctx->rings;
if (ctx->cached_sq_head != READ_ONCE(rings->sq.head)) { /*
/* * Ensure any loads from the SQEs are done at this point,
* Ensure any loads from the SQEs are done at this point, * since once we write the new head, the application could
* since once we write the new head, the application could * write new data to them.
* write new data to them. */
*/ smp_store_release(&rings->sq.head, ctx->cached_sq_head);
smp_store_release(&rings->sq.head, ctx->cached_sq_head);
}
} }
/* /*
......
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