[Gen-art] LC review: draft-ietf-ips-iser-06.txt

[David, given that all the comments are nits, and the large number of 
authors, I am not copying the authors.  If you think the comments 
useful, please relay them to the authors.  Thank you.]

I have been selected as the General Area Review Team (Gen-ART)
reviewer for this draft (for background on Gen-ART, please see
<http://www.alvestrand.no/ietf/gen/art/gen-art-FAQ.html>http://www.alvestrand.no/ietf/gen/art/gen-art-FAQ.html). 

Please resolve these comments along with any other Last Call comments
you may receive.

Document: iSCSI Extensions for RDMA Specification
Reviewer: Joel Halpern
Review Date:  January 12, 2007

Summary: This document is ready for publication as a Proposed Standard

Comments:

Nit:  The references to "remote iSER Layer" as a participant in 
communication (in the definitions section) is odd.  I've seen 
references to Layer Instances, Layer Entities, and other 
qualifiers.  But It is unusual to see Layer by itself used this 
way.  (Normally, a layer refers to the collection of communicating 
entities in that layer.  Even the definitions in this document define 
it that way.  A layer is neither local nor remote.)  I suspect that 
most of these occurrences would be better as "Remote Peer" according 
to the definitions. (or remote iSER node or more verbosely the iSER 
layer at the remote node, as per later usage.)

Nit: After the first occurrence of "RDMA-Capable Protocol" in the 
abstract, could you just put "(RCaP)"

Nit: I would include all the acronyms referenced in the definitions 
(SN, RCaP, ...) into the Acronym list.

Nit: I may have missed it, but I did not find a definitions (or 
description) of target and initiator before 2.4.  I presume that this 
is well-defined in other RDMA specs.  A paragraph here (or actual 
definitions in section 1) would be helpful.  (Yes, this can be 
understood once one gets to 2.5.  It is a matter of avoiding usage 
before description problems.) 

_______________________________________________
Gen-art mailing list
Gen-art@ietf.org
https://www1.ietf.org/mailman/listinfo/gen-art