Commit 0ba47879 authored by Jacek Kalwas's avatar Jacek Kalwas Committed by Tomasz Zawadzki
Browse files

nvmf: allow to override sgls 



Useful for transport specific layer to inform that SGLs are not
supported or to adjust settings.

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


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Broadcom CI
Community-CI: Mellanox Build Bot
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@mellanox.com>
parent 182ca749

include/spdk/nvmf_transport.h

+1 −0
Original line number Diff line number Diff line
@@ -174,6 +174,7 @@ struct spdk_nvmf_listener { 
 */

struct spdk_nvmf_ctrlr_data {

	uint16_t kas;

	struct spdk_nvme_cdata_sgls sgls;

	struct spdk_nvme_cdata_nvmf_specific nvmf_specific;

};

lib/nvmf/ctrlr.c

+4 −3
Original line number Diff line number Diff line
@@ -1821,6 +1821,9 @@ void 
spdk_nvmf_ctrlr_data_init(struct spdk_nvmf_transport_opts *opts, struct spdk_nvmf_ctrlr_data *cdata)

{

	cdata->kas = KAS_DEFAULT_VALUE;

	cdata->sgls.supported = 1;

	cdata->sgls.keyed_sgl = 1;

	cdata->sgls.sgl_offset = 1;

	cdata->nvmf_specific.ioccsz = sizeof(struct spdk_nvme_cmd) / 16;

	cdata->nvmf_specific.ioccsz += opts->in_capsule_data_size / 16;

	cdata->nvmf_specific.iorcsz = sizeof(struct spdk_nvme_cpl) / 16;
@@ -1847,9 +1850,7 @@ spdk_nvmf_ctrlr_identify_ctrlr(struct spdk_nvmf_ctrlr *ctrlr, struct spdk_nvme_c 
	cdata->lpa.edlp = 1;

	cdata->elpe = 127;

	cdata->maxcmd = transport->opts.max_queue_depth;

	cdata->sgls.supported = 1;

	cdata->sgls.keyed_sgl = 1;

	cdata->sgls.sgl_offset = 1;

	cdata->sgls = transport->cdata.sgls;

	cdata->fuses.compare_and_write = 1;

	cdata->acwu = 1;

	spdk_strcpy_pad(cdata->subnqn, subsystem->subnqn, sizeof(cdata->subnqn), '\0');