null_blk: Delete nullb.{queue_depth, nr_queues}
authorJohn Garry <john.g.garry@oracle.com>
Thu, 22 Feb 2024 08:34:20 +0000 (08:34 +0000)
committerJens Axboe <axboe@kernel.dk>
Thu, 22 Feb 2024 17:08:47 +0000 (10:08 -0700)
Since commit 8b631f9cf0b8 ("null_blk: remove the bio based I/O path"),
struct nullb members queue_depth and nr_queues are only ever written, so
delete them.

With that, null_exit_hctx() can also be deleted.

Signed-off-by: John Garry <john.g.garry@oracle.com>
Link: https://lore.kernel.org/r/20240222083420.6026-1-john.g.garry@oracle.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
drivers/block/null_blk/main.c
drivers/block/null_blk/null_blk.h

index a0b726c8366cc4248a9d6a9efeea1c03665b4783..71c39bcd872c7ecaabc67e91f35aa2fb267d6826 100644 (file)
@@ -1620,14 +1620,6 @@ static void null_queue_rqs(struct request **rqlist)
        *rqlist = requeue_list;
 }
 
-static void null_exit_hctx(struct blk_mq_hw_ctx *hctx, unsigned int hctx_idx)
-{
-       struct nullb_queue *nq = hctx->driver_data;
-       struct nullb *nullb = nq->dev->nullb;
-
-       nullb->nr_queues--;
-}
-
 static void null_init_queue(struct nullb *nullb, struct nullb_queue *nq)
 {
        nq->dev = nullb->dev;
@@ -1647,7 +1639,6 @@ static int null_init_hctx(struct blk_mq_hw_ctx *hctx, void *driver_data,
        nq = &nullb->queues[hctx_idx];
        hctx->driver_data = nq;
        null_init_queue(nullb, nq);
-       nullb->nr_queues++;
 
        return 0;
 }
@@ -1660,7 +1651,6 @@ static const struct blk_mq_ops null_mq_ops = {
        .poll           = null_poll,
        .map_queues     = null_map_queues,
        .init_hctx      = null_init_hctx,
-       .exit_hctx      = null_exit_hctx,
 };
 
 static void null_del_dev(struct nullb *nullb)
@@ -1731,7 +1721,6 @@ static int setup_queues(struct nullb *nullb)
        if (!nullb->queues)
                return -ENOMEM;
 
-       nullb->queue_depth = nullb->dev->hw_queue_depth;
        return 0;
 }
 
index 25320fe34bfe5e4f1a088af8d7fabcee4a4ae247..477b9774682346b95becee2afd9a7a1a5559af1b 100644 (file)
@@ -114,14 +114,12 @@ struct nullb {
        struct gendisk *disk;
        struct blk_mq_tag_set *tag_set;
        struct blk_mq_tag_set __tag_set;
-       unsigned int queue_depth;
        atomic_long_t cur_bytes;
        struct hrtimer bw_timer;
        unsigned long cache_flush_pos;
        spinlock_t lock;
 
        struct nullb_queue *queues;
-       unsigned int nr_queues;
        char disk_name[DISK_NAME_LEN];
 };