Commit 1bf3c2fe authored by Jacek Kalwas's avatar Jacek Kalwas Committed by Jim Harris
Browse files

nvme/tcp: enable accel seq support unconditionally 



Sequence might be used not only for data digest offload purposes but
to handle a potential COPY operation appended as the result of
translation (incoming patch). This COPY can be either in translate
function itself or in spdk nvme tcp layer.

Change-Id: I53c1b069aa1e807f66e60bb00842d8411b4bef6a
Signed-off-by: default avatarJacek Kalwas <jacek.kalwas@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21914


Community-CI: Mellanox Build Bot
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@nvidia.com>
Reviewed-by: default avatarJim Harris <jim.harris@samsung.com>
parent b4250450

lib/nvme/nvme_tcp.c

+3 −6
Original line number Diff line number Diff line
@@ -2581,12 +2581,9 @@ nvme_tcp_ctrlr_construct(const struct spdk_nvme_transport_id *trid, 
		return NULL;

	}



	/* Only advertise support for accel sequences if data digest is enabled, otherwise it

	 * doesn't provide any benefits to finish the sequences here */

	if (opts->data_digest) {

	/* Sequence might be used not only for data digest offload purposes but

	 * to handle a potential COPY operation appended as the result of translation. */

	tctrlr->ctrlr.flags |= SPDK_NVME_CTRLR_ACCEL_SEQUENCE_SUPPORTED;

	}



	tctrlr->ctrlr.adminq = nvme_tcp_ctrlr_create_qpair(&tctrlr->ctrlr, 0,

			       tctrlr->ctrlr.opts.admin_queue_size, 0,

			       tctrlr->ctrlr.opts.admin_queue_size, true);