Commit 14757fe8 authored by Ben Walker's avatar Ben Walker Committed by Tomasz Zawadzki
Browse files

accel: Correctly set nbytes for copy_crc32cv tasks 



If nbytes is not set, then the desination iovec sent to the underlying
driver has a length of 0.

Change-Id: Ia55f5ece942bd70f32bfdb3bcf02134ba98fca96
Signed-off-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/12612


Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Community-CI: Mellanox Build Bot
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <smatsumoto@nvidia.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@nvidia.com>
Reviewed-by: default avatarPaul Luse <paul.e.luse@intel.com>
parent 11a5716d

lib/accel/accel_engine.c

+8 −0
Original line number Diff line number Diff line
@@ -377,6 +377,8 @@ spdk_accel_submit_copy_crc32cv(struct spdk_io_channel *ch, void *dst, 
	struct spdk_accel_task *accel_task;

	struct spdk_accel_engine *engine = g_engines_opc[ACCEL_OPC_COPY_CRC32C];

	struct spdk_io_channel *engine_ch = accel_ch->engine_ch[ACCEL_OPC_COPY_CRC32C];

	uint64_t nbytes;

	uint32_t i;



	if (src_iovs == NULL) {

		SPDK_ERRLOG("iov should not be NULL");
@@ -395,11 +397,17 @@ spdk_accel_submit_copy_crc32cv(struct spdk_io_channel *ch, void *dst, 
		return -ENOMEM;

	}



	nbytes = 0;

	for (i = 0; i < iov_cnt; i++) {

		nbytes += src_iovs[i].iov_len;

	}



	accel_task->v.iovs = src_iovs;

	accel_task->v.iovcnt = iov_cnt;

	accel_task->dst = (void *)dst;

	accel_task->crc_dst = crc_dst;

	accel_task->seed = seed;

	accel_task->nbytes = nbytes;

	accel_task->flags = flags;

	accel_task->op_code = ACCEL_OPC_COPY_CRC32C;