|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: iSCSI: current UNH PlugfestAnshul, I don't know why the initiator should care if the sense data arrived along with the CHECK CONDITION scsi status or not. (?) Most, if not all, SCSI initiator stacks have some form of indicating sense_status from SCSI LLP (hba driver, device driver, mini-port driver, etc) to SCSI ULP (class driver, target driver, etc), which indicates what the status of the sense operation was and whether sense data is available or not. For instance, you also need to deal with a scenario where the target sends a SCSI status of CHECK CONDITION and accompanies it with the sense data in the data segment. However, the initiator encounters a data digest error on the sense data data segment and drops the sense data data segment. The initiator can always choose to complete the I/O and return the SCSI status back to its SCSI ULP, indicating that no sense data was available. I don't see why you would need the wording tightened to a MUST. Initiators must not assume that the sense data will always be available on a check condition. The sense operation may be unsuccessful and no sense data may be available. Regards, Santosh > Anshul Chadda wrote: > > 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) > > -- ################################## Santosh Rao Software Design Engineer, HP-UX iSCSI Driver Team, Hewlett Packard, Cupertino. email : santoshr@cup.hp.com Phone : 408-447-3751 ##################################
Home Last updated: Thu Nov 01 05:17:39 2001 7492 messages in chronological order |