block: only allocate/free mq_usage_counter in blk-mq
authorMing Lei <tom.leiming@gmail.com>
Tue, 27 May 2014 15:35:14 +0000 (23:35 +0800)
committerJens Axboe <axboe@fb.com>
Tue, 27 May 2014 15:37:08 +0000 (09:37 -0600)
The percpu counter is only used for blk-mq, so move
its allocation and free inside blk-mq, and don't
allocate it for legacy queue device.

Signed-off-by: Ming Lei <tom.leiming@gmail.com>
Signed-off-by: Jens Axboe <axboe@fb.com>
block/blk-core.c
block/blk-mq.c
block/blk-sysfs.c

index 5b6f768a7c01ae5ab87c44302dddeb326b20b775..29d5fbafd94a78e8f150cf47dbd30fdad8af85fe 100644 (file)
@@ -576,12 +576,9 @@ struct request_queue *blk_alloc_queue_node(gfp_t gfp_mask, int node_id)
        if (!q)
                return NULL;
 
-       if (percpu_counter_init(&q->mq_usage_counter, 0))
-               goto fail_q;
-
        q->id = ida_simple_get(&blk_queue_ida, 0, 0, gfp_mask);
        if (q->id < 0)
-               goto fail_c;
+               goto fail_q;
 
        q->backing_dev_info.ra_pages =
                        (VM_MAX_READAHEAD * 1024) / PAGE_CACHE_SIZE;
@@ -639,8 +636,6 @@ fail_bdi:
        bdi_destroy(&q->backing_dev_info);
 fail_id:
        ida_simple_remove(&blk_queue_ida, q->id);
-fail_c:
-       percpu_counter_destroy(&q->mq_usage_counter);
 fail_q:
        kmem_cache_free(blk_requestq_cachep, q);
        return NULL;
index 07851753a0490d1185df21721f55385f12c3caee..e8b5f74dc1a177a5dd4dce3a79881a171b2b2c5c 100644 (file)
@@ -1781,6 +1781,9 @@ struct request_queue *blk_mq_init_queue(struct blk_mq_tag_set *set)
        if (!q)
                goto err_hctxs;
 
+       if (percpu_counter_init(&q->mq_usage_counter, 0))
+               goto err_map;
+
        q->mq_map = blk_mq_make_queue_map(set);
        if (!q->mq_map)
                goto err_map;
@@ -1867,6 +1870,8 @@ void blk_mq_free_queue(struct request_queue *q)
        blk_mq_exit_hw_queues(q, set, set->nr_hw_queues);
        blk_mq_free_hw_queues(q, set);
 
+       percpu_counter_destroy(&q->mq_usage_counter);
+
        free_percpu(q->queue_ctx);
        kfree(q->queue_hw_ctx);
        kfree(q->mq_map);
index 4d6811ac13fddc9bfeba6e81afaef923ee6b4295..23321fbab29318ae5b550216c66eb9ae2d026c52 100644 (file)
@@ -517,8 +517,6 @@ static void blk_release_queue(struct kobject *kobj)
        if (q->queue_tags)
                __blk_queue_free_tags(q);
 
-       percpu_counter_destroy(&q->mq_usage_counter);
-
        if (q->mq_ops)
                blk_mq_free_queue(q);