Re: [Gen-art] Gen-ART Last Call review of draft-ietf-nfsv4-rpcrdma-bidirection-06

Meral Shirazipour <meral.shirazipour@ericsson.com> Tue, 07 February 2017 21:31 UTC

Return-Path: <meral.shirazipour@ericsson.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id DC5751293FB for <gen-art@ietfa.amsl.com>; Tue, 7 Feb 2017 13:31:43 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.2
X-Spam-Level:
X-Spam-Status: No, score=-4.2 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id jXawsP-JJNdh for <gen-art@ietfa.amsl.com>; Tue, 7 Feb 2017 13:31:42 -0800 (PST)
Received: from usplmg20.ericsson.net (usplmg20.ericsson.net [198.24.6.45]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id E8B621288B8 for <gen-art@ietf.org>; Tue, 7 Feb 2017 13:31:41 -0800 (PST)
X-AuditID: c618062d-d5fff700000009d8-02-589a4c6f769b
Received: from EUSAAHC007.ericsson.se (Unknown_Domain [147.117.188.93]) by (Symantec Mail Security) with SMTP id 5B.E7.02520.F6C4A985; Tue, 7 Feb 2017 23:38:43 +0100 (CET)
Received: from EUSAAMB107.ericsson.se ([147.117.188.124]) by EUSAAHC007.ericsson.se ([147.117.188.93]) with mapi id 14.03.0319.002; Tue, 7 Feb 2017 16:31:37 -0500
From: Meral Shirazipour <meral.shirazipour@ericsson.com>
To: Chuck Lever <chuck.lever@oracle.com>
Thread-Topic: Gen-ART Last Call review of draft-ietf-nfsv4-rpcrdma-bidirection-06
Thread-Index: AdKBeqZ/21QNKY8YTlG9PZCKJMeN0gAL7LSAAAg0KsA=
Date: Tue, 07 Feb 2017 21:31:36 +0000
Message-ID: <ABCAA4EF18F17B4FB619EA93DEF7939A4E883D15@eusaamb107.ericsson.se>
References: <ABCAA4EF18F17B4FB619EA93DEF7939A4E882975@eusaamb107.ericsson.se> <AB6C6DC9-7B0D-4813-9992-BA3C519DFB52@oracle.com>
In-Reply-To: <AB6C6DC9-7B0D-4813-9992-BA3C519DFB52@oracle.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [147.117.188.10]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrFLMWRmVeSWpSXmKPExsUyuXRPrG6xz6wIg2MvxCz+333OZDFl71o2 i6uvPrM4MHssWfKTyePj01ssHl8uf2YLYI7isklJzcksSy3St0vgymifsJK9YJpgxd3jXxgb GK/wdjFycEgImEg0N/h2MXJxCAmsZ5SY8XgWI4SzjFFi/78FLF2MnBxsAhYS238/ZwWxRQQ0 JHZP38EMUsQssIhRom31I3aQScICQRL/f8ZC1ARLzJ8+gQ3CtpI4+3gPM4jNIqAisfzMZrCZ vAK+EleXv2SDWNbIKHH561SwBk4BO4kpUyaCLWMUEJP4fmoNE4jNLCAucevJfDBbQkBAYsme 88wQtqjEy8f/WCFsJYlJS8+xQtTrSCzY/YkNwtaWWLbwNTPEYkGJkzOfsExgFJ2FZOwsJC2z kLTMQtKygJFlFSNHaXFBTm66kcEmRmCMHJNg093BeH+65yFGAQ5GJR7eDcdnRAixJpYVV+Ye YpTgYFYS4b2kPStCiDclsbIqtSg/vqg0J7X4EKM0B4uSOG/c6vvhQgLpiSWp2ampBalFMFkm Dk6pBsYZoas+TzUy+PP+8yr7i/+vyphlOCcuLp/+6sSTrlfrsjR/PpHs92Zn2G2p5fyhYYNw 1cknBQcFl0nXr7zwZd558XehaeXVf9TMZzVcjnGwmh8mf/KOPePDNx6Jz93LU/4syerbdPn0 MynHsofdknkcPxK7BYyF5qfdmZAcd/2pd//1Y39SA9cpsRRnJBpqMRcVJwIAgtEiiY0CAAA=
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/aAuSITOc7RbEfd0MGGQ3yEkAn_M>
Cc: "gen-art@ietf.org" <gen-art@ietf.org>, "draft-ietf-nfsv4-rpcrdma-bidirection.all@tools.ietf.org" <draft-ietf-nfsv4-rpcrdma-bidirection.all@tools.ietf.org>
Subject: Re: [Gen-art] Gen-ART Last Call review of draft-ietf-nfsv4-rpcrdma-bidirection-06
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 07 Feb 2017 21:31:44 -0000

Hi,
  Thank you.

Best,
Meral

> -----Original Message-----
> From: Chuck Lever [mailto:chuck.lever@oracle.com]
> Sent: Tuesday, February 07, 2017 12:26 PM
> To: Meral Shirazipour <meral.shirazipour@ericsson.com>
> Cc: draft-ietf-nfsv4-rpcrdma-bidirection.all@tools.ietf.org; gen-art@ietf.org
> Subject: Re: Gen-ART Last Call review of draft-ietf-nfsv4-rpcrdma-bidirection-06
> 
> 
> > On Feb 7, 2017, at 2:45 PM, Meral Shirazipour
> <meral.shirazipour@ericsson.com> wrote:
> >
> > I am the assigned Gen-ART reviewer for this draft. The General Area Review
> Team (Gen-ART) reviews all IETF documents being processed by the IESG for the
> IETF Chair.  Please treat these comments just like any other last call comments.
> > For more information, please see the FAQ at
> <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.
> >
> >
> > Document: draft-ietf-nfsv4-rpcrdma-bidirection-06
> > Reviewer: Meral Shirazipour
> > Review Date: 2017-02-07
> > IETF LC End Date:   2017-02-07
> > IESG Telechat date: -
> >
> >
> > Summary:
> > This draft is ready to be published as Standards Track RFC but I have
> comments.
> >
> > Major issues:
> > Minor issues:
> > Nits/editorial comments:
> > -General: please consider spelling out acronyms at first use and use the
> acronym consistently thereafter.
> 
> To address this comment (and Jouni's related LC comment) I've introduced the
> following forward definitions:
> 
> - RPC = Remote Procedure Call
> 
> - RDMA = Remote Direct Memory Access
> 
> - RPC-over-RDMA - defined by citation
> 
> - NFS = Network File System
> 
> - XDR - removed all usage
> 
> - XID = RPC transaction ID, and capitalized consistently
> 
> - NULL probe - replaced with "RPC NULL procedure"
> 
> - ULP - replaced with "RPC Program"
> 
> - Chunks - defined before use, and added citation
> 
> 
> Please let me know if there are other unfamiliar terms I've missed, or if there is a
> particular inconsistent usage that should be corrected.
> 
> 
> > -[Page 6], "(ie. a "---->"(i.e., a"
> 
> Fixed.
> 
> --
> Chuck Lever
> 
>