|
Hello:
As this issue has come up with setting CHECK
CONDITION in the SCSI Response. It is assumed that if CHECK CONDITION is
set in the SCSI Response PDU, then there has to be sense data accompanied with
it. So as far as I see it, it would help if the following sentence in the draft
has the MUST/must in there. In the current wording, i can think that if there is
no data segment in the SCSI Response PDU for a CHECK CONDITION, it is still
OK.
In draft 8, the sentence looks like the following:
-------------------------------------------------------
3.4.6 Data Segment - Sense and Response
Data Segment
iSCSI targets MUST support and enable autosense. If Status is
CHECK CONDITION (0x02), then the Data Segment contains sense data for the failed
command.
-------------------------------------------------------
It can be changed to the following:
-------------------------------------------------------
3.4.6 Data Segment - Sense and Response
Data Segment
iSCSI targets MUST support and enable autosense. If Status is
CHECK CONDITION (0x02), then the target MUST have sense data in
the data segment for the failed command.
-------------------------------------------------------
I don't know if there is a reason that the draft has the
wording in the current way. Apologies if this subject has already
been discussed.
Regards,
Anshul
-----------------------------------------------------------------------------------------------------------------
5. Some common error situations:
1) -
when a SCSI Response contains a CHECK CONDITION
(Status=0x02),
some targets are not including
the SenseLength as the first 2
bytes in the
data segment. Although the format of the data
segment
is clear from the diagram in section
3.4.6 on page 62 of draft 8
(page 63 of draft
8a), the last entry in the diagram for the
SCSI
Response PDU on page 58 of draft 8 (page
59 of draft 8a) is
misleading because it
mentions only the Sense Data and Response
Data
and omits the Sense Length. It would therefore be
helpful
if the last entry in the diagram on
page 58 were changed to explicitly
reference
the diagram on page 62, as
in:
Data Segment -- see
section 3.4.6 (optional)
| 