Commit 73e098f7 authored by Shuhei Matsumoto's avatar Shuhei Matsumoto Committed by Tomasz Zawadzki
Browse files

bdev/nvme: check_secondary_trid() returns -EALREADY for path duplication 



To make the error code consistent, change one return value of
bdev_nvme_check_secondary_trid() from -EEXIST to -ALREADY. Then the user
will be able to get consistent error code for path duplication errors.

Fixes issue #3383

Signed-off-by: default avatarShuhei Matsumoto <smatsumoto@nvidia.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23218

 (master)

(cherry picked from commit 5a564c0d)
Change-Id: Ic2ddadae371887c1377c6b52d97df1601fb66eda
Signed-off-by: default avatarMarek Chomnicki <marek.chomnicki@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23279


Reviewed-by: default avatarShuhei Matsumoto <smatsumoto@nvidia.com>
Reviewed-by: default avatarJim Harris <jim.harris@samsung.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent 9e3929bc

module/bdev/nvme/bdev_nvme.c

+1 −1
Original line number Diff line number Diff line
@@ -5885,7 +5885,7 @@ bdev_nvme_check_secondary_trid(struct nvme_ctrlr *nvme_ctrlr, 
		if (!spdk_nvme_transport_id_compare(&tmp_trid->trid, trid)) {

			SPDK_WARNLOG("This path (traddr: %s subnqn: %s) is already registered\n", trid->traddr,

				     trid->subnqn);

			return -EEXIST;

			return -EALREADY;

		}

	}