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

accel: replace runtime check with assert for crypto append 



It shall not happen by design therefore remove runtime check.

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


Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@nvidia.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent baeaf0d0

lib/accel/accel.c

+2 −8
Original line number Diff line number Diff line
@@ -1028,10 +1028,7 @@ spdk_accel_append_encrypt(struct spdk_accel_sequence **pseq, struct spdk_io_chan 
	struct spdk_accel_task *task;

	struct spdk_accel_sequence *seq = *pseq;



	if (spdk_unlikely(!dst_iovs || !dst_iovcnt || !src_iovs || !src_iovcnt || !key ||

			  !block_size)) {

		return -EINVAL;

	}

	assert(dst_iovs && dst_iovcnt && src_iovs && src_iovcnt && key && block_size);



	if (seq == NULL) {

		seq = accel_sequence_get(accel_ch);
@@ -1085,10 +1082,7 @@ spdk_accel_append_decrypt(struct spdk_accel_sequence **pseq, struct spdk_io_chan 
	struct spdk_accel_task *task;

	struct spdk_accel_sequence *seq = *pseq;



	if (spdk_unlikely(!dst_iovs || !dst_iovcnt || !src_iovs || !src_iovcnt || !key ||

			  !block_size)) {

		return -EINVAL;

	}

	assert(dst_iovs && dst_iovcnt && src_iovs && src_iovcnt && key && block_size);



	if (seq == NULL) {

		seq = accel_sequence_get(accel_ch);