io_uring: remove io_register_submitter
authorDylan Yudaken <dylany@fb.com>
Mon, 26 Sep 2022 17:09:27 +0000 (10:09 -0700)
committerJens Axboe <axboe@kernel.dk>
Fri, 7 Oct 2022 18:25:30 +0000 (12:25 -0600)
this is no longer needed, as submitter_task is set at creation time.

Signed-off-by: Dylan Yudaken <dylany@fb.com>
Fixes: 97bbdc06a444 ("io_uring: add IORING_SETUP_SINGLE_ISSUER")
Signed-off-by: Jens Axboe <axboe@kernel.dk>
io_uring/tctx.c

index dd0205f..4324b1c 100644 (file)
@@ -91,20 +91,6 @@ __cold int io_uring_alloc_task_context(struct task_struct *task,
        return 0;
 }
 
-static int io_register_submitter(struct io_ring_ctx *ctx)
-{
-       int ret = 0;
-
-       mutex_lock(&ctx->uring_lock);
-       if (!ctx->submitter_task)
-               ctx->submitter_task = get_task_struct(current);
-       else if (ctx->submitter_task != current)
-               ret = -EEXIST;
-       mutex_unlock(&ctx->uring_lock);
-
-       return ret;
-}
-
 int __io_uring_add_tctx_node(struct io_ring_ctx *ctx)
 {
        struct io_uring_task *tctx = current->io_uring;
@@ -151,11 +137,9 @@ int __io_uring_add_tctx_node_from_submit(struct io_ring_ctx *ctx)
 {
        int ret;
 
-       if (ctx->flags & IORING_SETUP_SINGLE_ISSUER) {
-               ret = io_register_submitter(ctx);
-               if (ret)
-                       return ret;
-       }
+       if (ctx->flags & IORING_SETUP_SINGLE_ISSUER
+           && ctx->submitter_task != current)
+               return -EEXIST;
 
        ret = __io_uring_add_tctx_node(ctx);
        if (ret)