]> git.baikalelectronics.ru Git - kernel.git/commitdiff
io_uring: cache __io_free_req()'d requests
authorPavel Begunkov <asml.silence@gmail.com>
Mon, 9 Aug 2021 19:18:08 +0000 (20:18 +0100)
committerJens Axboe <axboe@kernel.dk>
Mon, 23 Aug 2021 19:09:43 +0000 (13:09 -0600)
Don't kfree requests in __io_free_req() but put them back into the
internal request cache. That makes allocations more sustainable and will
be used for refcounting optimisations.

Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Link: https://lore.kernel.org/r/9f4950fbe7771c8d41799366d0a3a08ac3040236.1628536684.git.asml.silence@gmail.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
fs/io_uring.c

index 86162ebd6baec560a9b49e3f04201e3b7eb810fd..cc2d3de164234a55c6bc36bfdb585cac37a4721f 100644 (file)
@@ -1831,11 +1831,16 @@ static void io_dismantle_req(struct io_kiocb *req)
 static void __io_free_req(struct io_kiocb *req)
 {
        struct io_ring_ctx *ctx = req->ctx;
+       unsigned long flags;
 
        io_dismantle_req(req);
        io_put_task(req->task, 1);
 
-       kmem_cache_free(req_cachep, req);
+       spin_lock_irqsave(&ctx->completion_lock, flags);
+       list_add(&req->compl.list, &ctx->locked_free_list);
+       ctx->locked_free_nr++;
+       spin_unlock_irqrestore(&ctx->completion_lock, flags);
+
        percpu_ref_put(&ctx->refs);
 }