Hi Tim,
I see the "queue_prt(mgmtq, Q_STE_ERRS,.."
call has returned back into the code AFTER the assert.
That call in now in the code TWICE, both
before the assert and after it.

See lines 512 to 519, here:
http://cr.opensolaris.org/~ts143224/iscsitgtd/usr/src/cmd/iscsi/iscsitgtd/t10_sam.c.html

Personally, I thinks it's better to log before the assert,
and the one after it is redundant.

Its good to see all those 'connection_parameters_default's.
The list of defaulted parameters has expanded significantly!
Thanks
Nigel Smith
 
 
This message posted from opensolaris.org
_______________________________________________
storage-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/storage-discuss

Reply via email to