]> Gentwo Git Trees - linux/.git/commitdiff
io_uring: remove extra args from io_register_free_rings
authorPavel Begunkov <asml.silence@gmail.com>
Thu, 16 Oct 2025 13:23:20 +0000 (14:23 +0100)
committerJens Axboe <axboe@kernel.dk>
Mon, 20 Oct 2025 16:37:56 +0000 (10:37 -0600)
io_register_free_rings() doesn't use its "struct io_uring_params"
parameter, remove it.

Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Reviewed-by: Gabriel Krisman Bertazi <krisman@suse.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
io_uring/register.c

index a809d95153e4a2a59e4d4ee2d6765e32d68c8f2f..f7f71f035b0dc8244b2aab471043db078b291c51 100644 (file)
@@ -379,7 +379,6 @@ struct io_ring_ctx_rings {
 };
 
 static void io_register_free_rings(struct io_ring_ctx *ctx,
-                                  struct io_uring_params *p,
                                   struct io_ring_ctx_rings *r)
 {
        io_free_region(ctx, &r->sq_region);
@@ -434,7 +433,7 @@ static int io_register_resize_rings(struct io_ring_ctx *ctx, void __user *arg)
        }
        ret = io_create_region(ctx, &n.ring_region, &rd, IORING_OFF_CQ_RING);
        if (ret) {
-               io_register_free_rings(ctx, &p, &n);
+               io_register_free_rings(ctx, &n);
                return ret;
        }
        n.rings = io_region_get_ptr(&n.ring_region);
@@ -453,7 +452,7 @@ static int io_register_resize_rings(struct io_ring_ctx *ctx, void __user *arg)
        WRITE_ONCE(n.rings->cq_ring_entries, p.cq_entries);
 
        if (copy_to_user(arg, &p, sizeof(p))) {
-               io_register_free_rings(ctx, &p, &n);
+               io_register_free_rings(ctx, &n);
                return -EFAULT;
        }
 
@@ -462,7 +461,7 @@ static int io_register_resize_rings(struct io_ring_ctx *ctx, void __user *arg)
        else
                size = array_size(sizeof(struct io_uring_sqe), p.sq_entries);
        if (size == SIZE_MAX) {
-               io_register_free_rings(ctx, &p, &n);
+               io_register_free_rings(ctx, &n);
                return -EOVERFLOW;
        }
 
@@ -474,7 +473,7 @@ static int io_register_resize_rings(struct io_ring_ctx *ctx, void __user *arg)
        }
        ret = io_create_region(ctx, &n.sq_region, &rd, IORING_OFF_SQES);
        if (ret) {
-               io_register_free_rings(ctx, &p, &n);
+               io_register_free_rings(ctx, &n);
                return ret;
        }
        n.sq_sqes = io_region_get_ptr(&n.sq_region);
@@ -564,7 +563,7 @@ static int io_register_resize_rings(struct io_ring_ctx *ctx, void __user *arg)
 out:
        spin_unlock(&ctx->completion_lock);
        mutex_unlock(&ctx->mmap_lock);
-       io_register_free_rings(ctx, &p, to_free);
+       io_register_free_rings(ctx, to_free);
 
        if (ctx->sq_data)
                io_sq_thread_unpark(ctx->sq_data);