Move the DIAGNOSTIC check for lost iSCSI PDUs from icl_conn_close()

to icl_conn_free().  It's perfectly valid for the counter to be non-zero
in the former.

MFC after:	2 weeks
Sponsored by:	playkey.net
This commit is contained in:
Edward Tomasz Napierala 2017-12-09 15:34:40 +00:00
parent 9ae26a5cb9
commit 22d3bb2625
Notes: svn2git 2020-12-20 02:59:44 +00:00
svn path=/head/; revision=326730

View File

@ -1170,6 +1170,11 @@ void
icl_soft_conn_free(struct icl_conn *ic)
{
#ifdef DIAGNOSTIC
KASSERT(ic->ic_outstanding_pdus == 0,
("destroying session with %d outstanding PDUs",
ic->ic_outstanding_pdus));
#endif
cv_destroy(&ic->ic_send_cv);
cv_destroy(&ic->ic_receive_cv);
kobj_delete((struct kobj *)ic, M_ICL_SOFT);
@ -1410,11 +1415,6 @@ icl_soft_conn_close(struct icl_conn *ic)
KASSERT(STAILQ_EMPTY(&ic->ic_to_send),
("destroying session with non-empty send queue"));
#ifdef DIAGNOSTIC
KASSERT(ic->ic_outstanding_pdus == 0,
("destroying session with %d outstanding PDUs",
ic->ic_outstanding_pdus));
#endif
ICL_CONN_UNLOCK(ic);
}