Commit 005428e1 authored by Konrad Sztyber's avatar Konrad Sztyber Committed by Jim Harris
Browse files

accel: don't clear step_cb_fn in submit_dif functions 



Commit 47590bc4 ("accel: Use one completion callback") put task's
cb_fn and step_cb_fn into a union, as they're never used together.  This
required removing setting all step_cb_fn = NULL in all accel_submit
functions.  This patch does the same for the submit_dif functions, thus
making them usable.

Signed-off-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Change-Id: I30351004727e67b6c0480fa8b341db2dccaadfa6
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21496


Reviewed-by: default avatarJim Harris <jim.harris@samsung.com>
Reviewed-by: default avatarShuhei Matsumoto <smatsumoto@nvidia.com>
Community-CI: Mellanox Build Bot
Reviewed-by: default avatarKamil Godzwon <kamilx.godzwon@intel.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent f20a4abe

lib/accel/accel.c

+0 −3
Original line number Diff line number Diff line
@@ -858,7 +858,6 @@ spdk_accel_submit_dif_verify(struct spdk_io_channel *ch, 
	accel_task->op_code = SPDK_ACCEL_OPC_DIF_VERIFY;

	accel_task->src_domain = NULL;

	accel_task->dst_domain = NULL;

	accel_task->step_cb_fn = NULL;



	return accel_submit_task(accel_ch, accel_task);

}
@@ -885,7 +884,6 @@ spdk_accel_submit_dif_generate(struct spdk_io_channel *ch, 
	accel_task->op_code = SPDK_ACCEL_OPC_DIF_GENERATE;

	accel_task->src_domain = NULL;

	accel_task->dst_domain = NULL;

	accel_task->step_cb_fn = NULL;



	return accel_submit_task(accel_ch, accel_task);

}
@@ -914,7 +912,6 @@ spdk_accel_submit_dif_generate_copy(struct spdk_io_channel *ch, struct iovec *ds 
	accel_task->op_code = SPDK_ACCEL_OPC_DIF_GENERATE_COPY;

	accel_task->src_domain = NULL;

	accel_task->dst_domain = NULL;

	accel_task->step_cb_fn = NULL;



	return accel_submit_task(accel_ch, accel_task);

}