BKF Comments on iSCSI version 14

[Brian Forbes <bforbes@Brocade.COM>]

I've sent a list of editorial comments directly to Julian.

Technical comments:

T1:  

Although it is alluded to in section 11.10, the document must explicitly
define the effective value of Buffer Offset for immediate data.  Possible
sections for doing so include 2.2.4 and 9.3.6.

T2:  

I agree with the recent reflector discussion that decimal is only used for
numbers, hex for numbers and binary, and base64 only for binary items.  I
believe the debate should focus on the utility of the existing iSCSI type
definitions and avoid getting derailed by anecdotal references to data types
in other RFCs.

T3:  

Section 4.3.1, page 80:  "If the reconfiguration of iSCSI portal groups is a
concern in a given environment, the iSCSI initiator MUST use this key to
ascertain that it had indeed initiated the Login Phase with the intended
target portal group."

The use of MUST in the final sentence is untestable and should probably be
lower-case "should".

T4:  

Section 11.8, page 215:  "If the TargetAddress is returned as the result of
a redirect status in a login response, the comma and portal group tag are
omitted."

For interoperability reasons, I believe "are omitted" should be "MUST be
omitted".

T5:  

Section 11.8, page 215:  "If the TargetAddress is returned within a
SendTargets response, the portal group tag is required."

For interoperability reasons, I believe "is required" should be "MUST be
included".


Brian Forbes
Technology
Brocade Communications