|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] RE: iSCSI: Small editorial updateOK - I'll take out "starting with 0" - Julo
The implication comes from the phrase "For input (read) or bi-directional Data-In PDUs ... (starting with 0)". Since "starting with 0" is already explained elsewhere, there should be no need to say it again here. I suggest you remove "(starting with 0)" from both 9.7.5 and 9.8.1. Here is the paragraph: For input (read) or bi-directional Data-In PDUs, the DataSN is the input PDU number (starting with 0) within the data transfer for the command identified by the Initiator Task Tag. Eddy -----Original Message----- From: Julian Satran [mailto:Julian_Satran@il.ibm.com] Sent: Monday, June 24, 2002 3:50 PM To: Eddy Quicksall Cc: Eddy Quicksall; ips@ece.cmu.edu; John Hufferd; owner-ips@ece.cmu.edu Subject: RE: iSCSI: Small editorial update Eddy, Where is the implication? I am re-reading the text and I do not see it. If it is in the order of presentation then would the reverse order imply write data before read? Julo
Julo, At least, I don't think is is a good idea to leave an implication that is not correct. I'm referring to the fact that 9.7.5 clearly implies that a bidirectional command must have data-in before data-out. Eddy -----Original Message----- From: Julian Satran [mailto:Julian_Satran@il.ibm.com] Sent: Monday, June 24, 2002 12:02 PM To: Eddy Quicksall Cc: ips@ece.cmu.edu; John Hufferd; owner-ips@ece.cmu.edu Subject: RE: iSCSI: Small editorial update Eddy - the text says already "... the input PDU number" it repeats the sharing statement - I think it has enough to get things clear. I have changed bi-directional to bidirectional. Julo
The 1st paragraph needs a little change too. It implies that a bi-directional command must have data-in before data-out. If you had data-out first and R2Ts were being used, the DataSN would start with the last R2TSN + 1. And the 1st paragraph of 9.8.1 should probably say "starting with 0 for unidirectional commands" instead of "starting with 0". Also, SAM-2 uses the term "bidirectional" not "bi-directional". Can you change that too? Eddy -----Original Message----- From: Julian Satran [mailto:Julian_Satran@il.ibm.com] Sent: Monday, June 24, 2002 1:07 AM To: John Hufferd Cc: ips@ece.cmu.edu; owner-ips@ece.cmu.edu Subject: Re: iSCSI: Small editorial update OK - Julo |---------+----------------------------> | | "John Hufferd" | | | <hufferd@us.ibm.c| | | om> | | | Sent by: | | | owner-ips@ece.cmu| | | .edu | | | | | | | | | 06/24/2002 12:34 | | | AM | | | Please respond to| | | "John Hufferd" | | | | |---------+----------------------------> >--------------------------------------------------------------------------- ---------------------------------------| | | | To: "Julian Satran" <Julian_Satran@il.ibm.com> | | cc: ips@ece.cmu.edu | | Subject: iSCSI: Small editorial update | | | | | >--------------------------------------------------------------------------- ---------------------------------------| Julian, I think we need to add a clause that denotes that a bi-directional command is being addressed in the second paragraph under 9.7.5 in your working draft 14. We should add the expression ", in the context of bidirectional commands, ". The following would be the revised paragraph. "R2T and Data-In PDUs, in the context of bi-directional commands, share the numbering sequence (see Section 2.2.2.3 Data Sequencing)." I know it should be obvious when one really thinks about it, however, it make the intent easier to understand. . . . John L. Hufferd Senior Technical Staff Member (STSM) IBM/SSG San Jose Ca Main Office (408) 256-0403, Tie: 276-0403, eFax: (408) 904-4688 Home Office (408) 997-6136, Cell: (408) 499-9702 Internet address: hufferd@us.ibm.com
Home Last updated: Wed Jun 26 09:18:57 2002 10982 messages in chronological order |