From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from bombadil.infradead.org ([198.137.202.9]:37531 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932244AbcH2Kx7 (ORCPT ); Mon, 29 Aug 2016 06:53:59 -0400 From: Christoph Hellwig To: axboe@fb.com Cc: keith.busch@intel.com, linux-block@vger.kernel.org, linux-nvme@lists.infradead.org Subject: [PATCH 7/7] blk-mq: get rid of the cpumask in struct blk_mq_tags Date: Mon, 29 Aug 2016 12:53:33 +0200 Message-Id: <1472468013-29936-8-git-send-email-hch@lst.de> In-Reply-To: <1472468013-29936-1-git-send-email-hch@lst.de> References: <1472468013-29936-1-git-send-email-hch@lst.de> Sender: linux-block-owner@vger.kernel.org List-Id: linux-block@vger.kernel.org Unused now that NVMe sets up irq affinity before calling into blk-mq. Signed-off-by: Christoph Hellwig --- block/blk-mq-tag.c | 6 ------ block/blk-mq-tag.h | 1 - block/blk-mq.c | 7 ------- include/linux/blk-mq.h | 1 - 4 files changed, 15 deletions(-) diff --git a/block/blk-mq-tag.c b/block/blk-mq-tag.c index 1602813..2eae3d5 100644 --- a/block/blk-mq-tag.c +++ b/block/blk-mq-tag.c @@ -665,11 +665,6 @@ struct blk_mq_tags *blk_mq_init_tags(unsigned int total_tags, if (!tags) return NULL; - if (!zalloc_cpumask_var(&tags->cpumask, GFP_KERNEL)) { - kfree(tags); - return NULL; - } - tags->nr_tags = total_tags; tags->nr_reserved_tags = reserved_tags; @@ -680,7 +675,6 @@ void blk_mq_free_tags(struct blk_mq_tags *tags) { bt_free(&tags->bitmap_tags); bt_free(&tags->breserved_tags); - free_cpumask_var(tags->cpumask); kfree(tags); } diff --git a/block/blk-mq-tag.h b/block/blk-mq-tag.h index d468a79..5569641 100644 --- a/block/blk-mq-tag.h +++ b/block/blk-mq-tag.h @@ -44,7 +44,6 @@ struct blk_mq_tags { struct list_head page_list; int alloc_policy; - cpumask_var_t cpumask; }; diff --git a/block/blk-mq.c b/block/blk-mq.c index a679562..1fc42a0 100644 --- a/block/blk-mq.c +++ b/block/blk-mq.c @@ -1852,7 +1852,6 @@ static void blk_mq_map_swqueue(struct request_queue *q, hctx->tags = set->tags[i]; WARN_ON(!hctx->tags); - cpumask_copy(hctx->tags->cpumask, hctx->cpumask); /* * Set the map size to the number of mapped software queues. * This is more accurate and more efficient than looping @@ -2279,12 +2278,6 @@ static int blk_mq_alloc_rq_maps(struct blk_mq_tag_set *set) return 0; } -struct cpumask *blk_mq_tags_cpumask(struct blk_mq_tags *tags) -{ - return tags->cpumask; -} -EXPORT_SYMBOL_GPL(blk_mq_tags_cpumask); - static int blk_mq_create_mq_map(struct blk_mq_tag_set *set, const struct cpumask *affinity_mask) { diff --git a/include/linux/blk-mq.h b/include/linux/blk-mq.h index 29e227b..79a421e 100644 --- a/include/linux/blk-mq.h +++ b/include/linux/blk-mq.h @@ -197,7 +197,6 @@ struct request *blk_mq_alloc_request(struct request_queue *q, int rw, struct request *blk_mq_alloc_request_hctx(struct request_queue *q, int op, unsigned int flags, unsigned int hctx_idx); struct request *blk_mq_tag_to_rq(struct blk_mq_tags *tags, unsigned int tag); -struct cpumask *blk_mq_tags_cpumask(struct blk_mq_tags *tags); enum { BLK_MQ_UNIQUE_TAG_BITS = 16, -- 2.1.4 From mboxrd@z Thu Jan 1 00:00:00 1970 From: hch@lst.de (Christoph Hellwig) Date: Mon, 29 Aug 2016 12:53:33 +0200 Subject: [PATCH 7/7] blk-mq: get rid of the cpumask in struct blk_mq_tags In-Reply-To: <1472468013-29936-1-git-send-email-hch@lst.de> References: <1472468013-29936-1-git-send-email-hch@lst.de> Message-ID: <1472468013-29936-8-git-send-email-hch@lst.de> Unused now that NVMe sets up irq affinity before calling into blk-mq. Signed-off-by: Christoph Hellwig --- block/blk-mq-tag.c | 6 ------ block/blk-mq-tag.h | 1 - block/blk-mq.c | 7 ------- include/linux/blk-mq.h | 1 - 4 files changed, 15 deletions(-) diff --git a/block/blk-mq-tag.c b/block/blk-mq-tag.c index 1602813..2eae3d5 100644 --- a/block/blk-mq-tag.c +++ b/block/blk-mq-tag.c @@ -665,11 +665,6 @@ struct blk_mq_tags *blk_mq_init_tags(unsigned int total_tags, if (!tags) return NULL; - if (!zalloc_cpumask_var(&tags->cpumask, GFP_KERNEL)) { - kfree(tags); - return NULL; - } - tags->nr_tags = total_tags; tags->nr_reserved_tags = reserved_tags; @@ -680,7 +675,6 @@ void blk_mq_free_tags(struct blk_mq_tags *tags) { bt_free(&tags->bitmap_tags); bt_free(&tags->breserved_tags); - free_cpumask_var(tags->cpumask); kfree(tags); } diff --git a/block/blk-mq-tag.h b/block/blk-mq-tag.h index d468a79..5569641 100644 --- a/block/blk-mq-tag.h +++ b/block/blk-mq-tag.h @@ -44,7 +44,6 @@ struct blk_mq_tags { struct list_head page_list; int alloc_policy; - cpumask_var_t cpumask; }; diff --git a/block/blk-mq.c b/block/blk-mq.c index a679562..1fc42a0 100644 --- a/block/blk-mq.c +++ b/block/blk-mq.c @@ -1852,7 +1852,6 @@ static void blk_mq_map_swqueue(struct request_queue *q, hctx->tags = set->tags[i]; WARN_ON(!hctx->tags); - cpumask_copy(hctx->tags->cpumask, hctx->cpumask); /* * Set the map size to the number of mapped software queues. * This is more accurate and more efficient than looping @@ -2279,12 +2278,6 @@ static int blk_mq_alloc_rq_maps(struct blk_mq_tag_set *set) return 0; } -struct cpumask *blk_mq_tags_cpumask(struct blk_mq_tags *tags) -{ - return tags->cpumask; -} -EXPORT_SYMBOL_GPL(blk_mq_tags_cpumask); - static int blk_mq_create_mq_map(struct blk_mq_tag_set *set, const struct cpumask *affinity_mask) { diff --git a/include/linux/blk-mq.h b/include/linux/blk-mq.h index 29e227b..79a421e 100644 --- a/include/linux/blk-mq.h +++ b/include/linux/blk-mq.h @@ -197,7 +197,6 @@ struct request *blk_mq_alloc_request(struct request_queue *q, int rw, struct request *blk_mq_alloc_request_hctx(struct request_queue *q, int op, unsigned int flags, unsigned int hctx_idx); struct request *blk_mq_tag_to_rq(struct blk_mq_tags *tags, unsigned int tag); -struct cpumask *blk_mq_tags_cpumask(struct blk_mq_tags *tags); enum { BLK_MQ_UNIQUE_TAG_BITS = 16, -- 2.1.4