[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <259f9d8c-e552-7565-5e05-88a1c8430317@grimberg.me>
Date: Thu, 6 Oct 2016 00:47:26 +0300
From: Sagi Grimberg <sagi@...mberg.me>
To: Alexander Gordeev <agordeev@...hat.com>,
linux-kernel@...r.kernel.org
Cc: linux-block@...r.kernel.org
Subject: Re: [PATCH v2 6/8] blk-mq: Rework blk_mq_realloc_hw_ctxs()
> @@ -1908,33 +1909,36 @@ static void blk_mq_realloc_hw_ctxs(struct blk_mq_tag_set *set,
> if (node == NUMA_NO_NODE)
> node = set->numa_node;
>
> - hctxs[i] = kzalloc_node(sizeof(struct blk_mq_hw_ctx),
> - GFP_KERNEL, node);
> - if (!hctxs[i])
> + hctx = kzalloc_node(sizeof(*hctx), GFP_KERNEL, node);
> + if (!hctx)
> break;
>
> - if (blk_mq_init_hctx(q, set, hctxs[i], i, node)) {
> - kfree(hctxs[i]);
> - hctxs[i] = NULL;
> + if (blk_mq_init_hctx(q, set, hctx, i, node)) {
> + kfree(hctx);
> break;
> }
> - blk_mq_hctx_kobj_init(hctxs[i]);
> +
> + blk_mq_hctx_kobj_init(hctx);
> + hctxs[i] = hctx;
> }
> for (j = i; j < q->nr_hw_queues; j++) {
> - struct blk_mq_hw_ctx *hctx = hctxs[j];
> + hctx = hctxs[i];
Didn't you mean hctx[j]?
Powered by blists - more mailing lists