Commit 791ee7de authored by Shuhei Matsumoto's avatar Shuhei Matsumoto Committed by Tomasz Zawadzki
Browse files

nvme_rdma: nvme_rdma_process_events() returns negated errno 



It will be convenient for the following patches to return
negated errno directly.

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


Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@mellanox.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
parent cf7f2533

lib/nvme/nvme_rdma.c

+9 −7
Original line number Diff line number Diff line
@@ -585,11 +585,10 @@ nvme_rdma_poll_events(struct nvme_rdma_ctrlr *rctrlr) 
		}

	}



	if (errno == EAGAIN || errno == EWOULDBLOCK) {

		return 0;

	} else {

		return errno;

	}

	/* rdma_get_cm_event() returns -1 on error. If an error occurs, errno

	 * will be set to indicate the failure reason. So return negated errno here.

	 */

	return -errno;

}



static int
@@ -652,11 +651,14 @@ nvme_rdma_process_event(struct nvme_rdma_qpair *rqpair, 
	rctrlr = nvme_rdma_ctrlr(rqpair->qpair.ctrlr);

	assert(rctrlr != NULL);



	while (!rqpair->evt && spdk_get_ticks() < timeout_ticks && rc == 0) {

	while (!rqpair->evt && spdk_get_ticks() < timeout_ticks) {

		rc = nvme_rdma_poll_events(rctrlr);

		if (rc == -EAGAIN || rc == -EWOULDBLOCK) {

			continue;

		}

	}



	if (rc) {

	if (rc != 0 && rc != -EAGAIN && rc != -EWOULDBLOCK) {

		goto exit;

	}