Commit 5bdad0c1 authored by paul luse's avatar paul luse Committed by Tomasz Zawadzki
Browse files

examples/accel_perf: fix small bugs with copy + crc 



Wrong opcode was used when deciding whether a dst buffer is needed
or not. Also on freeing we were freeing src instead of the vector
parts of the task.

Signed-off-by: default avatarpaul luse <paul.e.luse@intel.com>
Change-Id: I33eb9408ee20eca6c44911b4567d0367528347de
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/9189


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 avatarMonica Kenguva <monica.kenguva@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@mellanox.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarZiye Yang <ziye.yang@intel.com>
parent df42f358

examples/accel/perf/accel_perf.c

+2 −2
Original line number Diff line number Diff line
@@ -324,7 +324,7 @@ _get_task_data_bufs(struct ap_task *task) 
		}

	}



	if (g_workload_selection != ACCEL_COPY_CRC32C) {

	if (g_workload_selection != ACCEL_CRC32C) {

		task->dst = spdk_dma_zmalloc(dst_buff_len, align, NULL);

		if (task->dst == NULL) {

			fprintf(stderr, "Unable to alloc dst buffer\n");
@@ -471,7 +471,7 @@ _free_task_buffers(struct ap_task *task) 
{

	uint32_t i;



	if (g_workload_selection == ACCEL_CRC32C) {

	if (g_workload_selection == ACCEL_CRC32C || g_workload_selection == ACCEL_COPY_CRC32C) {

		if (task->iovs) {

			for (i = 0; i < task->iov_cnt; i++) {

				if (task->iovs[i].iov_base) {