Return-Path: <owner-ips@ece.cmu.edu>
X-Sieve: cmu-sieve 2.0
Return-Path: <owner-ips@ece.cmu.edu>
Received: (from majordom@localhost)
	by ece.cmu.edu (8.11.0/8.10.2) id h09Gt8a26845
	for ips-outgoing; Thu, 9 Jan 2003 11:55:08 -0500 (EST)
X-Authentication-Warning: ece.cmu.edu: majordom set sender to owner-ips@ece.cmu.edu using -f
Received: from d12lmsgate-2.de.ibm.com (d12lmsgate-2.de.ibm.com [194.196.100.235])
	by ece.cmu.edu (8.11.0/8.10.2) with ESMTP id h09Gt3W26835
	for <ips@ece.cmu.edu>; Thu, 9 Jan 2003 11:55:03 -0500 (EST)
Received: from d12relay02.de.ibm.com (d12relay02.de.ibm.com [9.165.215.23])
	by d12lmsgate-2.de.ibm.com (8.12.3/8.12.3) with ESMTP id h09Gstv4018900;
	Thu, 9 Jan 2003 17:54:55 +0100
Received: from d10ml001.telaviv.ibm.com (d10ml001.telaviv.ibm.com [9.148.216.55])
	by d12relay02.de.ibm.com (8.12.3/NCO/VER6.4) with ESMTP id h09GspTL250738;
	Thu, 9 Jan 2003 17:54:54 +0100
In-Reply-To: <Pine.LNX.4.50.0301091045150.29346-100000@io.iol.unh.edu>
To: "Robert D. Russell" <rdr@io.iol.unh.edu>
Cc: "" <ips@ece.cmu.edu>
Subject: Re: TargetPortalGroupTag
MIME-Version: 1.0
X-Mailer: Lotus Notes Release 6.0 September 26, 2002
From: "Julian Satran" <Julian_Satran@il.ibm.com>
Message-ID: <OFE33D90B3.A47051CA-ONC2256CA9.005B95CD-C2256CA9.005CE912@telaviv.ibm.com>
Date: Thu, 9 Jan 2003 18:54:47 +0200
X-MIMETrack: Serialize by Router on D10ML001/10/M/IBM(Release 5.0.9a |January 7, 2002) at
 09/01/2003 18:54:53,
	Serialize complete at 09/01/2003 18:54:53
Content-Type: multipart/alternative; boundary="=_alternative 005BC170C2256CA9_="
Sender: owner-ips@ece.cmu.edu
Precedence: bulk

This is a multipart message in MIME format.
--=_alternative 005BC170C2256CA9_=
Content-Type: text/plain; charset="US-ASCII"

thanks - (an unfortunate by-product of a change). Julo



"Robert D. Russell" <rdr@io.iol.unh.edu> 
09/01/03 17:47

To
Julian Satran/Haifa/IBM@IBMIL, "" <ips@ece.cmu.edu>
cc

Subject
TargetPortalGroupTag






Julian:

> I would like to drive this to a conclusion (draft) by the end of this
> week.


A minor editorial nit to consider if you are about to release
another draft:

In section 2.1 of draft 19-90, under the definition of SSID,
the last sentence says:

  "The TargetPortalGroupTag key may also be returned by
  the target as a confimation during session establishment."

I believe use of the words "may" and "session" in this sentence
is wrong and misleading, and that it should be rewritten to say:

  "The TargetPortalGroupTag key must also be returned by
  the target as a confimation during connection establishment."

The word "may" is inappropriate because the target has no choice about
returning the TargetPortalGroupTag, according to section 5.3.1 Login
Phase Start the target MUST return it:

  "During the Login Phase the iSCSI target MUST return the
  TargetPortalGroupTag key with the first Login Response PDU with
  which it is allowed to do so ..."

and again in section 12.9 TargetPortalGroupTag:

  "The iSCSI target returns this key to the
  initiator in the Login Response PDU to the first Login Request PDU
  that has the C bit set to 0."


Similarly, the word "session" is inappropriate because the
TargetPortalGroupTag is marked "IO", not "LO", in section 12.9,
and both of the statements already quoted above apply to all logins,
not just "leading only" (i.e., session establishment) logins.


Thank you for your consideration.


Bob Russell
InterOperability Lab
University of New Hampshire
rdr@iol.unh.edu
603-862-3774


--=_alternative 005BC170C2256CA9_=
Content-Type: text/html; charset="US-ASCII"


<br><font size=2 face="sans-serif">thanks - (an unfortunate by-product
of a change). Julo</font>
<br>
<br>
<br>
<table width=100%>
<tr valign=top>
<td width=40%><font size=1 face="sans-serif"><b>&quot;Robert D. Russell&quot;
&lt;rdr@io.iol.unh.edu&gt;</b> </font>
<p><font size=1 face="sans-serif">09/01/03 17:47</font>
<td width=59%>
<table width=100%>
<tr>
<td>
<div align=right><font size=1 face="sans-serif">To</font></div>
<td valign=top><font size=1 face="sans-serif">Julian Satran/Haifa/IBM@IBMIL,
&quot;&quot; &lt;ips@ece.cmu.edu&gt;</font>
<tr>
<td>
<div align=right><font size=1 face="sans-serif">cc</font></div>
<td valign=top>
<tr>
<td>
<div align=right><font size=1 face="sans-serif">Subject</font></div>
<td valign=top><font size=1 face="sans-serif">TargetPortalGroupTag</font></table>
<br>
<table>
<tr valign=top>
<td>
<td></table>
<br></table>
<br>
<br>
<br><font size=2><tt>Julian:<br>
<br>
&gt; I would like to drive this to a conclusion (draft) by the end of this<br>
&gt; week.<br>
<br>
<br>
A minor editorial nit to consider if you are about to release<br>
another draft:<br>
<br>
In section 2.1 of draft 19-90, under the definition of SSID,<br>
the last sentence says:<br>
<br>
 &nbsp;&quot;The TargetPortalGroupTag key may also be returned by<br>
 &nbsp;the target as a confimation during session establishment.&quot;<br>
<br>
I believe use of the words &quot;may&quot; and &quot;session&quot; in this
sentence<br>
is wrong and misleading, and that it should be rewritten to say:<br>
<br>
 &nbsp;&quot;The TargetPortalGroupTag key must also be returned by<br>
 &nbsp;the target as a confimation during connection establishment.&quot;<br>
<br>
The word &quot;may&quot; is inappropriate because the target has no choice
about<br>
returning the TargetPortalGroupTag, according to section 5.3.1 Login<br>
Phase Start the target MUST return it:<br>
<br>
 &nbsp;&quot;During the Login Phase the iSCSI target MUST return the<br>
 &nbsp;TargetPortalGroupTag key with the first Login Response PDU with<br>
 &nbsp;which it is allowed to do so ...&quot;<br>
<br>
and again in section 12.9 TargetPortalGroupTag:<br>
<br>
 &nbsp;&quot;The iSCSI target returns this key to the<br>
 &nbsp;initiator in the Login Response PDU to the first Login Request PDU<br>
 &nbsp;that has the C bit set to 0.&quot;<br>
<br>
<br>
Similarly, the word &quot;session&quot; is inappropriate because the<br>
TargetPortalGroupTag is marked &quot;IO&quot;, not &quot;LO&quot;, in section
12.9,<br>
and both of the statements already quoted above apply to all logins,<br>
not just &quot;leading only&quot; (i.e., session establishment) logins.<br>
<br>
<br>
Thank you for your consideration.<br>
<br>
<br>
Bob Russell<br>
InterOperability Lab<br>
University of New Hampshire<br>
rdr@iol.unh.edu<br>
603-862-3774<br>
</tt></font>
<br>
--=_alternative 005BC170C2256CA9_=--