[storm] Unassigned vs. Reserved in rdmap-ext draft

"Black, David" <david.black@emc.com> Sat, 14 September 2013 05:14 UTC

Return-Path: <david.black@emc.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 F157C11E80D1 for <storm@ietfa.amsl.com>; Fri, 13 Sep 2013 22:14:04 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.599
X-Spam-Level:
X-Spam-Status: No, score=-102.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id YSSJ7hctuGYL for <storm@ietfa.amsl.com>; Fri, 13 Sep 2013 22:14:00 -0700 (PDT)
Received: from mailuogwhop.emc.com (mailuogwhop.emc.com [168.159.213.141]) by ietfa.amsl.com (Postfix) with ESMTP id 253B421F90A7 for <storm@ietf.org>; Fri, 13 Sep 2013 22:13:59 -0700 (PDT)
Received: from maildlpprd01.lss.emc.com (maildlpprd01.lss.emc.com [10.253.24.33]) by mailuogwprd03.lss.emc.com (Sentrion-MTA-4.3.0/Sentrion-MTA-4.3.0) with ESMTP id r8E5DwmP009447 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Sat, 14 Sep 2013 01:13:58 -0400
X-DKIM: OpenDKIM Filter v2.4.3 mailuogwprd03.lss.emc.com r8E5DwmP009447
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=emc.com; s=jan2013; t=1379135638; bh=eZrXAaG93SLgHOykOuXU009UQaE=; h=From:To:Date:Subject:Message-ID:Content-Type: Content-Transfer-Encoding:MIME-Version; b=wTdpzov5IYieFkPtCxwU2k9G+Gw1cHue6qgK4YqkkC9y/CJOP0qpWGJxnE48HQgzJ BTBcJ44PYytP4boyO8e/REAahdZRHHaNvtJprceuJkD1wX3wOemrpKThJJ/pO5SCul +3Hk/rcrHzjXmJwrFB9ybuM/ewo1DArsCcUHmXjI=
X-DKIM: OpenDKIM Filter v2.4.3 mailuogwprd03.lss.emc.com r8E5DwmP009447
Received: from mailusrhubprd52.lss.emc.com (mailusrhubprd52.lss.emc.com [10.106.48.25]) by maildlpprd01.lss.emc.com (RSA Interceptor); Sat, 14 Sep 2013 01:13:50 -0400
Received: from mxhub21.corp.emc.com (mxhub21.corp.emc.com [128.222.70.133]) by mailusrhubprd52.lss.emc.com (Sentrion-MTA-4.3.0/Sentrion-MTA-4.3.0) with ESMTP id r8E5DoI4012760 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=FAIL); Sat, 14 Sep 2013 01:13:50 -0400
Received: from mx15a.corp.emc.com ([169.254.1.46]) by mxhub21.corp.emc.com ([128.222.70.133]) with mapi; Sat, 14 Sep 2013 01:13:49 -0400
From: "Black, David" <david.black@emc.com>
To: "Sharp, Robert O" <robert.o.sharp@intel.com>, Tom Talpey <ttalpey@microsoft.com>, "storm@ietf.org" <storm@ietf.org>
Date: Sat, 14 Sep 2013 01:13:49 -0400
Thread-Topic: Unassigned vs. Reserved in rdmap-ext draft
Thread-Index: Ac6w8/laSw0o3ma3RLCdlkF3uUN2yg==
Message-ID: <8D3D17ACE214DC429325B2B98F3AE712025DA1BD36@MX15A.corp.emc.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
acceptlanguage: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-Sentrion-Hostname: mailusrhubprd52.lss.emc.com
X-EMM-GWVC: 1
X-EMM-McAfeeVC: 1
X-RSA-Classifications: public
Subject: [storm] Unassigned vs. Reserved in rdmap-ext draft
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: Sat, 14 Sep 2013 05:14:05 -0000

Bob,

> > > > + Section 5.2.1 Atomic operation values
> > > >
> > > > >   ---------+-----------+----------+----------+---------+---------
> > > > >   0011b    |           |
> > > > >   to       | Reserved  |            Not Specified
> > > > >   1111b    |           |
> > > > >   ---------+-----------+-----------------------------------------
> > > >
> > > > ... and
> > > >
> > > > >   4. At the Responder, when an invalid Atomic Operation Request
> > > > >      Message is delivered to the Remote Peer's RDMAP layer, an error
> > > > >      is surfaced.
> > > >
> > > > Are the "Reserved" fields invalid?
> > > >
> > >
> > > [Authors] Yes, the reserved values are invalid.  Clarifying text has
> > > been added.
> >
> > Unbeknownst to you, "Reserved" turns out to be a reserved word for IANA
> > that has a meaning different from usage in other standards bodies (we hit
> > this in working through IANA's comments on the iscsi-sam draft).  This is
> not a
> > problem in the new IANA Considerations text, but we probably ought to also
> > expunge use of that word here.
> >
> 
> That is unfortunate.  We changed it to un-used in the rest of the doc but
> don't like that much.  Also now it is different than RFC 5040.  Are you sure
> we want to change this?

Oops, you're right - go back to Reserved in the figure and body of draft, and
just stick with Unassigned in the IANA Considerations.  Sorry for the detour ...

Thanks,
--David


> -----Original Message-----
> From: Sharp, Robert O [mailto:robert.o.sharp@intel.com]
> Sent: Wednesday, September 11, 2013 5:11 PM
> To: Black, David; Tom Talpey; storm@ietf.org
> Subject: RE: [storm] Review comments on draft-ietf-storm-rdmap-ext
> 
> Hi David,
> 
> Thanks again.  Responses below...
> 
> Thanks,
> Bob
> 
> > -----Original Message-----
> > From: Black, David [mailto:david.black@emc.com]
> > Sent: Friday, September 06, 2013 6:31 PM
> > To: Sharp, Robert O; Tom Talpey; storm@ietf.org
> > Subject: RE: [storm] Review comments on draft-ietf-storm-rdmap-ext
> >
> > One more small item:
> >
> > > > + Section 5.2.1 Atomic operation values
> > > >
> > > > >   ---------+-----------+----------+----------+---------+---------
> > > > >   0011b    |           |
> > > > >   to       | Reserved  |            Not Specified
> > > > >   1111b    |           |
> > > > >   ---------+-----------+-----------------------------------------
> > > >
> > > > ... and
> > > >
> > > > >   4. At the Responder, when an invalid Atomic Operation Request
> > > > >      Message is delivered to the Remote Peer's RDMAP layer, an error
> > > > >      is surfaced.
> > > >
> > > > Are the "Reserved" fields invalid?
> > > >
> > >
> > > [Authors] Yes, the reserved values are invalid.  Clarifying text has
> > > been added.
> >
> > Unbeknownst to you, "Reserved" turns out to be a reserved word for IANA
> > that has a meaning different from usage in other standards bodies (we hit
> > this in working through IANA's comments on the iscsi-sam draft).  This is
> not a
> > problem in the new IANA Considerations text, but we probably ought to also
> > expunge use of that word here.
> >
> 
> That is unfortunate.  We changed it to un-used in the rest of the doc but
> don't like that much.  Also now it is different than RFC 5040.  Are you sure
> we want to change this?
> 
> > I suggest deleting the above table row in Figure 5, and making the following
> > change to item 4 below the table:
> >
> > OLD
> >    4. At the Responder, when an invalid Atomic Operation Request
> >       Message with a reserved Atomic Operation Code is delivered to the
> >       Remote Peer's RDMAP layer, an error MUST be surfaced.
> > NEW
> >    4. At the Responder, an error MUST be surfaced in response to delivery
> >       to the Remote Peer's RDMAP layer of an Atomic Operation Request
> >       Message with an Atomic Operation Code that the RNIC does not support.
> >
> 
> Good suggestion.
> 
> > Thanks,
> > --David
> >
> > > -----Original Message-----
> > > From: storm-bounces@ietf.org [mailto:storm-bounces@ietf.org] On Behalf
> > > Of Sharp, Robert O
> > > Sent: Thursday, September 05, 2013 5:23 PM
> > > To: Tom Talpey; storm@ietf.org
> > > Subject: Re: [storm] Review comments on draft-ietf-storm-rdmap-ext
> > >
> > > Hi Tom,
> > >
> > > Thanks for the comments!  The author's responses are embedded below...
> > >
> > > Thanks,
> > > Bob
> > >