Re: [storm] SPC-2 reserve/release: Proposed text - version 2

Mallikarjun Chadalapaka <cbm@chadalapaka.com> Tue, 25 September 2012 19:40 UTC

Return-Path: <cbm@chadalapaka.com>
X-Original-To: storm@ietfa.amsl.com
Delivered-To: storm@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id AE4421F0C86 for <storm@ietfa.amsl.com>; Tue, 25 Sep 2012 12:40:32 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.599
X-Spam-Level:
X-Spam-Status: No, score=-3.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, RCVD_IN_DNSWL_LOW=-1]
Received: from mail.ietf.org ([64.170.98.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id kpZXhnXUJiWX for <storm@ietfa.amsl.com>; Tue, 25 Sep 2012 12:40:31 -0700 (PDT)
Received: from va3outboundpool.messaging.microsoft.com (va3ehsobe002.messaging.microsoft.com [216.32.180.12]) by ietfa.amsl.com (Postfix) with ESMTP id 62EB521F8873 for <storm@ietf.org>; Tue, 25 Sep 2012 12:40:31 -0700 (PDT)
Received: from mail227-va3-R.bigfish.com (10.7.14.236) by VA3EHSOBE004.bigfish.com (10.7.40.24) with Microsoft SMTP Server id 14.1.225.23; Tue, 25 Sep 2012 19:40:30 +0000
Received: from mail227-va3 (localhost [127.0.0.1]) by mail227-va3-R.bigfish.com (Postfix) with ESMTP id DFCACA80130 for <storm@ietf.org>; Tue, 25 Sep 2012 19:40:30 +0000 (UTC)
X-Forefront-Antispam-Report: CIP:157.56.240.117; KIP:(null); UIP:(null); IPV:NLI; H:BL2PRD0610HT004.namprd06.prod.outlook.com; RD:none; EFVD:NLI
X-SpamScore: -26
X-BigFish: PS-26(zz9371I542M4015Id6f1izz1202h1d1ah1d2ahzz1033IL8275dhz2fh2a8h668h839hd25hf0ah107ah1288h12a5h12a9h12bdh137ah13b6h1155h)
Received-SPF: pass (mail227-va3: domain of chadalapaka.com designates 157.56.240.117 as permitted sender) client-ip=157.56.240.117; envelope-from=cbm@chadalapaka.com; helo=BL2PRD0610HT004.namprd06.prod.outlook.com ; .outlook.com ;
Received: from mail227-va3 (localhost.localdomain [127.0.0.1]) by mail227-va3 (MessageSwitch) id 1348602027715733_17539; Tue, 25 Sep 2012 19:40:27 +0000 (UTC)
Received: from VA3EHSMHS002.bigfish.com (unknown [10.7.14.243]) by mail227-va3.bigfish.com (Postfix) with ESMTP id A304E6C005D; Tue, 25 Sep 2012 19:40:27 +0000 (UTC)
Received: from BL2PRD0610HT004.namprd06.prod.outlook.com (157.56.240.117) by VA3EHSMHS002.bigfish.com (10.7.99.12) with Microsoft SMTP Server (TLS) id 14.1.225.23; Tue, 25 Sep 2012 19:40:25 +0000
Received: from BL2PRD0610MB361.namprd06.prod.outlook.com ([169.254.12.31]) by BL2PRD0610HT004.namprd06.prod.outlook.com ([10.255.101.39]) with mapi id 14.16.0190.008; Tue, 25 Sep 2012 19:40:25 +0000
From: Mallikarjun Chadalapaka <cbm@chadalapaka.com>
To: "Black, David" <david.black@emc.com>, "storm@ietf.org" <storm@ietf.org>
Thread-Topic: SPC-2 reserve/release: Proposed text - version 2
Thread-Index: Ac2bNKguGPJW+zTbTTC2uDGdWEV2OAAGl/vwAAE5wPA=
Date: Tue, 25 Sep 2012 19:40:23 +0000
Message-ID: <E160851FCED17643AE5F53B5D4D0783A451DBAE3@BL2PRD0610MB361.namprd06.prod.outlook.com>
References: <8D3D17ACE214DC429325B2B98F3AE7120DE79512@MX15A.corp.emc.com>
In-Reply-To: <8D3D17ACE214DC429325B2B98F3AE7120DE79512@MX15A.corp.emc.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [131.107.174.24]
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: chadalapaka.com
Subject: Re: [storm] SPC-2 reserve/release: Proposed text - version 2
X-BeenThere: storm@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Storage Maintenance WG <storm.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/storm>, <mailto:storm-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/storm>
List-Post: <mailto:storm@ietf.org>
List-Help: <mailto:storm-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/storm>, <mailto:storm-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 25 Sep 2012 19:40:32 -0000

Hi David,

Good catch on Section 4.6.3.3 - I have now fixed it.

Please see my comments and revised text that I just sent in response to your first email.

Your reference impact summary looks appropriate to me.

Thanks.

Mallikarjun


-----Original Message-----
From: storm-bounces@ietf.org [mailto:storm-bounces@ietf.org] On Behalf Of Black, David
Sent: Tuesday, September 25, 2012 12:21 PM
To: storm@ietf.org
Subject: [storm] SPC-2 reserve/release: Proposed text - version 2

I moved a paragraph break in response to Ralph's comments, and rewrote the first additional consideration to reference the connection timeout values in general.  I also found some minor problems that need correction in Section 4.6.3.3 (see end of this message).

<WG chair hat off>

Here's an initial attempt to propose text to deal with the reserve/ release topic in the consolidated iSCSI draft.  Please comment, correct, suggest edits, etc.

The new text would be a new section 4.4.3.2 that would follow 4.4.3.1:

4.4.3.1. I_T Nexus State

  Certain nexus relationships contain an explicit state (e.g.,
  initiator-specific mode pages) that may need to be preserved by
  the device server [SAM2] in a logical unit through changes or
  failures in the iSCSI layer (e.g., session failures). In order for
  that state to be restored, the iSCSI initiator should reestablish
  its session (re-login) to the same Target Portal Group using the
  previous ISID. That is, it should perform session recovery as
  described in Chapter 6. This is because the SCSI initiator port
  identifier and the SCSI target port identifier (or relative target
  port) form the datum that the SCSI logical unit device server uses
  to identify the I_T nexus.

-------------------------------

First, for clarity, the following change should be made in the above 4.4.3.1 text to better identify the recovery mechanism:

OLD
  That is, it should perform session recovery as
  described in Chapter 6.
NEW
  That is, it should recover the session via connection
  reinstatement as described in Section 6.3.4.
END

I believe that a lower-case "should" remains appropriate for this text.

-----------------------------------

NEW TEXT (second draft):

4.4.3.2. I_T Nexus State: Reservations

  The state of an I_T Nexus includes SCSI reservation state.  There
  are two reservation management methods defined in the SCSI standards,
  reserve/release reservations, based on the RESERVE and RELEASE
  commands [SPC2], and persistent reservations, based on the PERSISTENT
  RESERVE IN and PERSISTENT RESERVE OUT commands [SPC3].  Reserve/release
  reservations are obsolete [SPC3] and SHOULD NOT be used; persistent
  reservations SHOULD be used instead.

  The I_T Nexus state for persistent reservations is generally required
  to persist through changes and failures at the iSCSI layer that result
  in I_T Nexus failures, see [SPC3] for details and specific requirements.

  In contrast, [SPC2] does not specify detailed persistence
  requirements for reserve/release reservation state - nonetheless,
  when reserve/release reservations are supported by an iSCSI target,
  the preferred implementation approach is to preserve reserve/release
  reservation state along with other I_T Nexus state for iSCSI session
  recovery or continuation via connection reinstatement.

  Two additional caveats apply to reserve/release reservations:

  - Retention of reserve/release reservation state for an extended
	period of time after connection failure may require a hard reset
      (e.g., LOGICAL UNIT RESET, see section 11.5) in order to remove
      that reservation state when connection reinstatement is not
      performed.  When reserve/release reservation state is retained
      after connection failure, the need for such hard resets may be
      reduced via suitable selection of the Time2Wait and Time2Retain
      timeout values that control the duration of that retention
      (see Section 7.5).
 
  - Reserve/release reservations may not behave as expected when
      persistent reservations are also used on the same logical unit;
      see the discussion of "Exceptions to SPC-2 RESERVE and RELEASE
      behavior" in [SPC4].

-----------------------

Reference impacts: both [SPC2] and [SPC3] need to be normative references, but [SPC4] can be an informative reference.

-------------------------

I deliberately used "the preferred iSCSI implementation approach"
wording for reserve/release reservation state preservation requirements, as SPC-2 is vague on this topic and I'm rather uncomfortable with placing a requirement as strong as a "SHOULD" on an obsolete mechanism that "SHOULD NOT" be used.

---------------

New problem in Section 4.6.3.3 - in this text:

   The Logout response indicates that the connection or session
   cleanup is completed and no other responses will arrive on the
   connection (if received on the logging out connection). In
   addition, the Logout Response indicates how long the target will
   continue to hold resources for recovery (e.g., command execution
   that continues on a new connection) in the text key Time2Retain
   and how long the initiator must wait before proceeding with
   recovery in the text key Time2Wait.

Time2Wait and Time2Retain are not text keys - they're binary fields in the Logout Response PDU.  Two changes are in order:
	- "text key Time2Retain" -> "Time2Retain field"
	- "text key Time2Wait" -> "Time2Wait field"

Thanks,
--David
----------------------------------------------------
David L. Black, Distinguished Engineer
EMC Corporation, 176 South St., Hopkinton, MA  01748
+1 (508) 293-7953              FAX: +1 (508) 293-7786
david.black@emc.com        Mobile: +1 (978) 394-7754
----------------------------------------------------

_______________________________________________
storm mailing list
storm@ietf.org
https://www.ietf.org/mailman/listinfo/storm

_______________________________________________
storm mailing list
storm@ietf.org
https://www.ietf.org/mailman/listinfo/storm