Comments/Clarifications on sas1r03 SSP transport layer

[Ameen Rahman]

* From the T10 Reflector (t10 at t10.org), posted by:
* Ameen Rahman <ameen at cup.hp.com>
*
I have the following comments/clarifications on 
SAS-1.1 Revision 3, Dated 24 Jan 2004

1) COMMAND frame - handling og link layer errors (Sec 9.2.4.2, Page 282)
  
   The last sentence in this page is,
   "An SSP initiator port should re-transmit each COMMAND frame that 
   does not receive an ACK ateast one time".
  
   I think an SSP initiator port should retransmit only those COMMANDs
   which were NOT delivered to the target port. 

2) SSP frame format ( Sec 9.2.1, Page 270 )
  
   This section doesn't explain the RETRY DATA FRAMES and CHANGING DATA 
   pointer fields of SSP frame header. Probably some text needs to be 
   added in this section to explain these fields.

   Explanation for these fields can be found in later sections though.

3) SSP frame format ( Sec 9.2.1, Page 270 )
 
   Explanation for RETRANSMIT bit from the standard:

   "The RETRANSMIT bit is set to one for TASK frames, RESPONSE frames,
and 
   XFER_RDY frames under the conditions defined in FIXFIX and shall be
set 
   to zero for all other frame types. This bit indicates the frame is a 
   retransmission after the SSP target port failed in its previous
attempt 
   to transmit the frame"

   a) In the above "FIXFIX" needs to be sustituted with relevant section 
      numbers.

   b) RETRANSMIT bit is also a indication of initiator port failed in
its 
      previous attempt to transmit a frame ( TASK frame )

4) TASK frame - handling of link layer errors  ( Sec 9.2.4.3, Page 283 )
  
   If a NAK is recieved for the TASK frame, what should be the action ?
   Retry the TASK frame with RETRANSMIT bit set to 0 ??

5) XFER_RDY frame with transport layer re-tries ( Sec 9.2.4.4.3 , Page
283 )
  
   When the target recieves NAK for a XFER_RDY, probably it should 
   not set RETRANSMIT bit to 1 in the retransmitted XFER_RDY.

Regards
-Ameen
*
* For T10 Reflector information, send a message with
* 'info t10' (no quotes) in the message body to majordomo at t10.org