Re: [storm] SPC-2 reserve/release text - version 6
Julian Satran <julian@satran.net> Thu, 27 September 2012 02:25 UTC
Return-Path: <julian@satran.net>
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 E766F21F8587 for <storm@ietfa.amsl.com>; Wed, 26 Sep 2012 19:25:18 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -5.203
X-Spam-Level:
X-Spam-Status: No, score=-5.203 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, MIME_QP_LONG_LINE=1.396, RCVD_IN_DNSWL_MED=-4]
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 kXkgTgoPQikx for <storm@ietfa.amsl.com>; Wed, 26 Sep 2012 19:25:18 -0700 (PDT)
Received: from eu1sys200aog106.obsmtp.com (eu1sys200aog106.obsmtp.com [207.126.144.121]) by ietfa.amsl.com (Postfix) with SMTP id 5DFF021F861A for <storm@ietf.org>; Wed, 26 Sep 2012 19:25:17 -0700 (PDT)
Received: from mail-we0-f198.google.com ([74.125.82.198]) (using TLSv1) by eu1sys200aob106.postini.com ([207.126.147.11]) with SMTP ID DSNKUGO5CcXFgOj74R1bDhfq0CDNlRoXaSxF@postini.com; Thu, 27 Sep 2012 02:25:17 UTC
Received: by weyz53 with SMTP id z53so329384wey.1 for <storm@ietf.org>; Wed, 26 Sep 2012 19:25:13 -0700 (PDT)
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=references:in-reply-to:mime-version:content-transfer-encoding :content-type:message-id:cc:x-mailer:from:subject:date:to :x-gm-message-state; bh=r37a9JDmwokyUHYvqws2iqfV4TTf6wy1yU5MyX8AANo=; b=A12e8c7HlKv5K6qBK/zVa0Soa4YZY9y5BV4pz5Hp8e98Lrr5wR7Bl0BHepedQVLyYw tks5fODj7DAfMqjEd0/qOK/ORYaltQLi7gcSG3gqyU7dxxkymhOPxa+PU0rIXGjWLxmy MZkMFMJhVWoAHVykXetExpNR8xFOzMCjwJsSfLAKGQipZ42h65d39FcQshjLmY6otsAA Ph+VF/o/ENvUWLV1T94D/gpIrRrAuJ8OreyD11EfxsicNo4OYb9cmIqqF6nYzIoCzn9r DYkQlTdo0mcUY7J+6Lj+BZLIGT8jNzW33fb53wPziytV4G7hTEN3vqKNdSVSX524eJ8H MURw==
Received: by 10.180.82.164 with SMTP id j4mr4983142wiy.18.1348712713887; Wed, 26 Sep 2012 19:25:13 -0700 (PDT)
Received: by 10.180.82.164 with SMTP id j4mr4983127wiy.18.1348712713743; Wed, 26 Sep 2012 19:25:13 -0700 (PDT)
Received: from [192.168.0.100] (ip-54-21.sn2.eutelia.it. [83.211.54.21]) by mx.google.com with ESMTPS id hv8sm35539509wib.0.2012.09.26.19.25.10 (version=SSLv3 cipher=OTHER); Wed, 26 Sep 2012 19:25:12 -0700 (PDT)
References: <8D3D17ACE214DC429325B2B98F3AE7120DE797C4@MX15A.corp.emc.com>
In-Reply-To: <8D3D17ACE214DC429325B2B98F3AE7120DE797C4@MX15A.corp.emc.com>
Mime-Version: 1.0 (1.0)
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset="utf-8"
Message-Id: <4AE113CD-96D7-414D-8AAB-36ECF7FD89EA@satran.net>
X-Mailer: iPad Mail (10A403)
From: Julian Satran <julian@satran.net>
Date: Thu, 27 Sep 2012 04:25:22 +0200
To: "Black, David" <david.black@emc.com>
X-Gm-Message-State: ALoCoQmppvwk7tE6n+Lkq1IOy/sB/Y0Kimsjbh165Y/Ifos++LjttuQjpUeM8i+XbdCupuHhZW8+dnO80ROiCEk9cQcdbTGmFVGPtpj7j5qvnnwGaj6dYHxlsETnYKei3c7o88c2VuWWEhsIAzaIpKZ2WFooG3ik/w==
Cc: "storm@ietf.org" <storm@ietf.org>
Subject: Re: [storm] SPC-2 reserve/release text - version 6
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: Thu, 27 Sep 2012 02:25:19 -0000
David, The text in 4.4.3 states the obvious (there is no other speced method for continuation or reinstatement). Moreover if the the session is reinstated after time2wait you create a need for a UA (or at least suggest one) and that was not there. Just stating that creation/continuation of a session done according 6.x includes preservation of reserve/release state -that was already there implied by conformance to spc2 and the relation between it nexus and session. The new text you suggest to 4.4 is merely a best practices recomendations and will need additions once the new "locking" through compare-and-swap get in widespread use. If instead we refrain from stating anything specific and state that target should maintain whatever state is required by SPC while a session is continued and may discard state when a session is terminated and may indicate that it discarded state through a ua you are on safer ground and do not create new requirement. Sent from my iPad On 27 בספט 2012, at 00:25, "Black, David" <david.black@emc.com> wrote: > Here's where I think we've arrived (I've made a few minor editorial > tweaks to what's been discussed). Despite all the detailed discussion, > I think this version 6 comes close to Julian's suggestion that: > >> i would mention only that the state that must be maintained while >> the session is maintained includes the reserve/release and leave it to that. > > Although we can't actually say "must" as that would be a new requirement, > and implementations differ in what they do about reserve/release > reservation state. The current language characterizes retention of > that state as the "preferred implementation approach". > > Beyond that, I think the two additional caveats about reserve/release > reservations (possible need for a reset, potentially unexpected > interaction with persistent reservations, see below) are useful to include. > > ----------------------------------------- > > [1] For clarity, the following change should be made in > 4.4.3.1: > > OLD > That is, it should perform session recovery as > described in Chapter 6. > NEW > That is, it should reinstate the session via iSCSI session > reinstatement (Section 6.3.5) or continue via session > continuation (Section 6.3.6). > END > > [2] Add the following sentence to the end of 4.4.3.1: > > The specific Unit Attention condition that results from session > reestablishment indicates whether nexus state was preserved, > see Section 6.3.4 of [SAM4]. > > [3] NEW TEXT: > > 4.4.3.2. Reservations > > 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. > > State for persistent reservations is 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 after an I_T > Nexus failure. Nonetheless, when reserve/release reservations are > supported by an iSCSI target, the preferred implementation approach > is to preserve reserve/release reservation state for iSCSI session > reinstatement (see Section 6.3.5) or session continuation > (see Section 6.3.6). > > Two additional caveats apply to reserve/release reservations: > > - When connection failure causes the iSCSI session to fail, and > the session is not reinstated or continued, target retention > of that session's reserve/release reservation state may > require an initiator to issue a reset (e.g., LOGICAL UNIT RESET, > see section 11.5) in order to remove that reservation state. > > - 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]. > > 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] SPC-2 reserve/release text - version 6 Black, David
- Re: [storm] SPC-2 reserve/release text - version 6 Julian Satran
- Re: [storm] SPC-2 reserve/release text - version 6 Black, David
- Re: [storm] SPC-2 reserve/release text - version 6 Knight, Frederick
- Re: [storm] SPC-2 reserve/release text - version 6 Black, David
- Re: [storm] SPC-2 reserve/release text - version 6 Knight, Frederick
- Re: [storm] SPC-2 reserve/release text - version 6 Julian Satran
- Re: [storm] SPC-2 reserve/release text - version 6 Knight, Frederick
- Re: [storm] SPC-2 reserve/release text - version 6 Black, David
- Re: [storm] SPC-2 reserve/release text - version 6 Knight, Frederick
- Re: [storm] SPC-2 reserve/release text - version 6 Black, David
- Re: [storm] SPC-2 reserve/release text - version 6 Knight, Frederick
- Re: [storm] SPC-2 reserve/release text - version 6 Black, David
- Re: [storm] SPC-2 reserve/release text - version 6 Knight, Frederick