[Ips] AD review of draft-ietf-ips-iser

Lars Eggert <lars.eggert@netlab.nec.de> Thu, 12 October 2006 15:30 UTC

Received: from [127.0.0.1] (helo=stiedprmman1.va.neustar.com) by megatron.ietf.org with esmtp (Exim 4.43) id 1GY2WK-0002mV-RZ; Thu, 12 Oct 2006 11:30:52 -0400
Received: from [10.91.34.44] (helo=ietf-mx.ietf.org) by megatron.ietf.org with esmtp (Exim 4.43) id 1GY2WJ-0002k4-Dc for ips@ietf.org; Thu, 12 Oct 2006 11:30:51 -0400
Received: from kyoto.netlab.nec.de ([195.37.70.21]) by ietf-mx.ietf.org with esmtp (Exim 4.43) id 1GY2WH-0004XQ-TN for ips@ietf.org; Thu, 12 Oct 2006 11:30:51 -0400
Received: from lars.local (u041251.ppp.asahi-net.or.jp [203.212.41.251]) by kyoto.netlab.nec.de (Postfix) with ESMTP id C9C4C1BAC4D; Thu, 12 Oct 2006 17:30:48 +0200 (CEST)
Received: from [127.0.0.1] (localhost [127.0.0.1]) by lars.local (Postfix) with ESMTP id C380C24BC29; Fri, 13 Oct 2006 00:29:45 +0900 (JST)
Mime-Version: 1.0 (Apple Message framework v752.2)
To: ips@ietf.org
Message-Id: <C515B950-2704-41D3-82CC-038A002D7A02@netlab.nec.de>
From: Lars Eggert <lars.eggert@netlab.nec.de>
Date: Thu, 12 Oct 2006 17:52:55 +0900
X-Mailer: Apple Mail (2.752.2)
X-Spam-Score: 0.3 (/)
X-Scan-Signature: c54bc2f42d02429833c0ca4b8725abd7
Cc: ips-ads@tools.ietf.org, ips-chairs@tools.ietf.org
Subject: [Ips] AD review of draft-ietf-ips-iser
X-BeenThere: ips@ietf.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: IP Storage <ips.ietf.org>
List-Unsubscribe: <https://www1.ietf.org/mailman/listinfo/ips>, <mailto:ips-request@ietf.org?subject=unsubscribe>
List-Post: <mailto:ips@ietf.org>
List-Help: <mailto:ips-request@ietf.org?subject=help>
List-Subscribe: <https://www1.ietf.org/mailman/listinfo/ips>, <mailto:ips-request@ietf.org?subject=subscribe>
Content-Type: multipart/mixed; boundary="===============0879880073=="
Errors-To: ips-bounces@ietf.org

Hi,

please find the AD review of draft-ietf-ips-iser-05.txt below. In a  
nutshell: No major issues but a number of comments and nits.  
Depending on the amount of changes needed, an RFC Editor Note may be  
sufficient, or a new revision may be needed.

Lars

---<snip>---

   Has idnits (unused references, etc.) and boilerplate issues (some due
   to boilerplate changes during the too-long AD review - mea culpa).


INTRODUCTION, paragraph 12:
 >    Ko et. al.                Expires April  
2006                         1

   Nit: s/et./et/ everywhere


Section 7.3.12, paragraph 3:
 >    qualified with DataDescriptorReject which defines the Rejct  
buffer.

   Nit: s/Rejct/Reject/


Section 8.1.1.3, paragraph 3:
 >    CmdSN of the PDU sent by the intiator in this category is x,  
the PDU

   Nit: s/intiator/initiator/


Section 8.1.2, paragraph 7:
 >    (Implementatin note:  SCSI task timeout and recovery can be a

   Nit: s/Implementatin/Implementation/


Section 9.1, paragraph 4:
 >         The Opcode field identifies the type of iSER Messages:

   Are there IANA actions required for these type codes? IANA section is
   empty.


Section 13.2, paragraph 2:
 >    [DA] M. Chadalapaka et al., "Datamover Architecture for iSCSI",  
IETF
 >        Internet-draft, draft-ietf-ips-da-02.txt (work in progress),


   Not normative?

 >    [RDDPSEC] J. Pinkerton et al., "DDP/RDMAP Security", IETF Internet
 >        Draft draft-ietf-rddp-security-07.txt (work in progress),  
April
 >        2005

   Not normative?

-- 
Lars Eggert                                     NEC Network Laboratories


Hi,

please find the AD review of draft-ietf-ips-iser-05.txt below. In a  
nutshell: No major issues but a number of comments and nits.  
Depending on the amount of changes needed, an RFC Editor Note may be  
sufficient, or a new revision may be needed.

Lars

---<snip>---

   Has idnits (unused references, etc.) and boilerplate issues (some due
   to boilerplate changes during the too-long AD review - mea culpa).


INTRODUCTION, paragraph 12:
 >    Ko et. al.                Expires April  
2006                         1

   Nit: s/et./et/ everywhere


Section 7.3.12, paragraph 3:
 >    qualified with DataDescriptorReject which defines the Rejct  
buffer.

   Nit: s/Rejct/Reject/


Section 8.1.1.3, paragraph 3:
 >    CmdSN of the PDU sent by the intiator in this category is x,  
the PDU

   Nit: s/intiator/initiator/


Section 8.1.2, paragraph 7:
 >    (Implementatin note:  SCSI task timeout and recovery can be a

   Nit: s/Implementatin/Implementation/


Section 9.1, paragraph 4:
 >         The Opcode field identifies the type of iSER Messages:

   Are there IANA actions required for these type codes? IANA section is
   empty.


Section 13.2, paragraph 2:
 >    [DA] M. Chadalapaka et al., "Datamover Architecture for iSCSI",  
IETF
 >        Internet-draft, draft-ietf-ips-da-02.txt (work in progress),


   Not normative?

 >    [RDDPSEC] J. Pinkerton et al., "DDP/RDMAP Security", IETF Internet
 >        Draft draft-ietf-rddp-security-07.txt (work in progress),  
April
 >        2005

   Not normative?

-- 
Lars Eggert                                     NEC Network Laboratories


_______________________________________________
Ips mailing list
Ips@ietf.org
https://www1.ietf.org/mailman/listinfo/ips