Commit 5a564c0d 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>
Change-Id: Ic2ddadae371887c1377c6b52d97df1601fb66eda
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/23218


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Community-CI: Mellanox Build Bot
parent fe0d81ba

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;

		}

	}