This patch uniforms the returned error (EBADFD) when the ring state (enabled/disabled) is not the expected one.
The changes affect io_uring_enter() and io_uring_register() syscalls. Signed-off-by: Stefano Garzarella <[email protected]> --- Hi Jens, I also updated the test/register-restrictions in liburing here: https://github.com/stefano-garzarella/liburing (branch: fix-disabled-ring-error) Thanks, Stefano --- fs/io_uring.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/fs/io_uring.c b/fs/io_uring.c index 0490edfcdd88..cf5992e79d88 100644 --- a/fs/io_uring.c +++ b/fs/io_uring.c @@ -8642,6 +8642,7 @@ SYSCALL_DEFINE6(io_uring_enter, unsigned int, fd, u32, to_submit, if (!percpu_ref_tryget(&ctx->refs)) goto out_fput; + ret = -EBADFD; if (ctx->flags & IORING_SETUP_R_DISABLED) goto out_fput; @@ -9137,7 +9138,7 @@ static int io_register_restrictions(struct io_ring_ctx *ctx, void __user *arg, /* Restrictions allowed only if rings started disabled */ if (!(ctx->flags & IORING_SETUP_R_DISABLED)) - return -EINVAL; + return -EBADFD; /* We allow only a single restrictions registration */ if (ctx->restrictions.registered) @@ -9201,7 +9202,7 @@ static int io_register_restrictions(struct io_ring_ctx *ctx, void __user *arg, static int io_register_enable_rings(struct io_ring_ctx *ctx) { if (!(ctx->flags & IORING_SETUP_R_DISABLED)) - return -EINVAL; + return -EBADFD; if (ctx->restrictions.registered) ctx->restricted = 1; -- 2.26.2
