Commit 0158f354 authored by Jim Harris's avatar Jim Harris
Browse files

nvme: only update cntlid after adminq connect 



For ioq, target is supposed to respond with the same
cntlid as specified in the connect capsule, but the
Linux kernel target doesn't actually do that.  It really
only needs to be set on the adminq connect, so just
do that.

Signed-off-by: default avatarJim Harris <james.r.harris@intel.com>
Change-Id: I4eec605f856bdad2c8614d505241566a9fe292ab

Reviewed-on: https://review.gerrithub.io/372345


Reviewed-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
parent 905f1930

lib/nvme/nvme_rdma.c

+4 −2
Original line number Diff line number Diff line
@@ -621,8 +621,10 @@ nvme_rdma_qpair_fabric_connect(struct nvme_rdma_qpair *rqpair) 
		return -1;

	}



	if (nvme_qpair_is_admin_queue(&rqpair->qpair)) {

		rsp = (struct spdk_nvmf_fabric_connect_rsp *)&status.cpl;

		rctrlr->cntlid = rsp->status_code_specific.success.cntlid;

	}

ret:

	spdk_dma_free(nvmf_data);

	return rc;