Commit c8677839 authored Mar 30, 2022 by Ben Walker Committed by Tomasz Zawadzki Apr 07, 2022

bdev/nvme: Remove ctrlr from nvme_ctrlr_channel



This was neither set nor used.

Signed-off-by: Ben Walker <benjamin.walker@intel.com>
Change-Id: I3119135843c5fc0b8724e593db40df46e6b5bdb0
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/12097


Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Community-CI: Mellanox Build Bot
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Michael Haeuptle <michaelhaeuptle@gmail.com>
Reviewed-by: Aleksey Marchuk <alexeymar@mellanox.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>

parent 8a2fc789

module/bdev/nvme/bdev_nvme.h

+0 −1

Original line number	Diff line number	Diff line
		@@ -187,7 +187,6 @@ struct nvme_qpair {
		};

		struct nvme_ctrlr_channel {
		struct nvme_ctrlr *ctrlr;
		struct nvme_qpair *qpair;
		TAILQ_HEAD(, spdk_bdev_io) pending_resets;

test/unit/lib/bdev/nvme/bdev_nvme.c/bdev_nvme_ut.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -4001,7 +4001,7 @@ test_find_io_path(void)
		struct spdk_nvme_qpair qpair1 = {}, qpair2 = {};
		struct spdk_nvme_ctrlr ctrlr1 = {}, ctrlr2 = {};
		struct nvme_ctrlr nvme_ctrlr1 = { .ctrlr = &ctrlr1, }, nvme_ctrlr2 = { .ctrlr = &ctrlr2, };
		struct nvme_ctrlr_channel ctrlr_ch1 = { .ctrlr = &nvme_ctrlr1, }, ctrlr_ch2 = { .ctrlr = &nvme_ctrlr2, };
		struct nvme_ctrlr_channel ctrlr_ch1 = {}, ctrlr_ch2 = {};
		struct nvme_qpair nvme_qpair1 = { .ctrlr_ch = &ctrlr_ch1, .ctrlr = &nvme_ctrlr1, };
		struct nvme_qpair nvme_qpair2 = { .ctrlr_ch = &ctrlr_ch2, .ctrlr = &nvme_ctrlr2, };
		struct nvme_ns nvme_ns1 = {}, nvme_ns2 = {};