Subject: 04 draft comments

From: [email-masked] (Sandeep Joshi)
Date: Sat, 24 Feb 2001 23:45:57 -0500 (EST)

Hi Julian,

A few questions/comments..

1) The following suggestion seems to be missing from 
   the draft
      http://ips.pdl.cs.cmu.edu/mail/msg03273.html

2) There is a WN before the BHS and every AHS.
   So what are the WN bits 6-4 for a BHS ?
   Right now, there is 0=ext_cdb, 1=bidi, 2=longdata.
   All the other values seem to be reserved.

3) Section 2.17 R2T - see statement before the fig.
   "outstanding R2T MUST be fulfilled by the initiator
   in the order they were received".
   Is this _within_ a command or connection or session.
   I presume its not within a session.
   Could you please qualify and disambiguate?  

4) In Section 2.5 & Sec 2.6 for TaskMgmt commands the
   RefTaskTag, if reserved, is to be set to zero.
   In other commands (e.g. NOP-OUT), an invalid task tag is
   indicated by 0xffffffff.  How about making it uniform
   throughout as 0xff... (maybe this is an oversight)

5) statSN seems to be missing at byte 20 in R2T (Sec 2.17)
   Any particular reason?

6) In Section 2.8.1, a minor correction
   "When set to 1 it indicates that his is the last.."
								    ^^^	
   lets make that "this" for a gender-neutral protocol :-)

-Sandeep