Re: [PATCH 0/7] further ring init cleanups

[Jens Axboe <axboe@kernel.dk>]

On 11/12/25 5:45 AM, Pavel Begunkov wrote:
> There are several goals for this patch set. It deduplicates ring size and
> offset calculations between ring setup and resizing. It moves most of
> verification earlier before any allocations, which usually means simpler
> error handling. And it keeps the logic localised instead of spreading it
> across the file.
> 
> Pavel Begunkov (7):
>   io_uring: refactor rings_size nosqarray handling
>   io_uring: use size_add helpers for ring offsets
>   io_uring: convert params to pointer in ring reisze
>   io_uring: introduce struct io_ctx_config
>   io_uring: keep ring laoyut in a structure
>   io_uring: pre-calculate scq layout
>   io_uring: move cq/sq user offset init around
> 
>  io_uring/io_uring.c | 137 ++++++++++++++++++++++++--------------------
>  io_uring/io_uring.h |  19 +++++-
>  io_uring/register.c |  65 +++++++++------------
>  3 files changed, 119 insertions(+), 102 deletions(-)

All looks pretty straightforward to me. I'm going to let it simmer for a
day or so to give other folks a chance to take a look too.

-- 
Jens Axboe