Commit 45b7fb23 authored by Konrad Sztyber's avatar Konrad Sztyber Committed by Tomasz Zawadzki
Browse files

accel: assert that we never modify seq's ERROR state 



This should help catching bugs when a failed sequence gets cleared its
failed state.

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


Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@nvidia.com>
parent 914007c1

lib/accel/accel.c

+1 −0
Original line number Diff line number Diff line
@@ -150,6 +150,7 @@ accel_sequence_set_state(struct spdk_accel_sequence *seq, enum accel_sequence_st 
{

	SPDK_DEBUGLOG(accel, "seq=%p, setting state: %s -> %s\n", seq,

		      ACCEL_SEQUENCE_STATE_STRING(seq->state), ACCEL_SEQUENCE_STATE_STRING(state));

	assert(seq->state != ACCEL_SEQUENCE_STATE_ERROR || state == ACCEL_SEQUENCE_STATE_ERROR);

	seq->state = state;

}