Re: [nfsv4] Ben Campbell's No Objection on draft-ietf-nfsv4-rpcrdma-bidirection-07: (with COMMENT)

Chuck Lever <chuck.lever@oracle.com> Thu, 02 March 2017 20:18 UTC

Return-Path: <chuck.lever@oracle.com>
X-Original-To: nfsv4@ietfa.amsl.com
Delivered-To: nfsv4@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 4DDB2129541; Thu, 2 Mar 2017 12:18:32 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.202
X-Spam-Level:
X-Spam-Status: No, score=-4.202 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H2=-0.001, RP_MATCHES_RCVD=-0.001, SPF_PASS=-0.001, UNPARSEABLE_RELAY=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 C0od1TZLSGUN; Thu, 2 Mar 2017 12:18:30 -0800 (PST)
Received: from aserp1040.oracle.com (aserp1040.oracle.com [141.146.126.69]) (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 CC4D2129476; Thu, 2 Mar 2017 12:18:27 -0800 (PST)
Received: from aserv0022.oracle.com (aserv0022.oracle.com [141.146.126.234]) by aserp1040.oracle.com (Sentrion-MTA-4.3.2/Sentrion-MTA-4.3.2) with ESMTP id v22KIQqT030866 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Thu, 2 Mar 2017 20:18:27 GMT
Received: from aserv0122.oracle.com (aserv0122.oracle.com [141.146.126.236]) by aserv0022.oracle.com (8.14.4/8.14.4) with ESMTP id v22KIQ65002144 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Thu, 2 Mar 2017 20:18:26 GMT
Received: from abhmp0019.oracle.com (abhmp0019.oracle.com [141.146.116.25]) by aserv0122.oracle.com (8.14.4/8.14.4) with ESMTP id v22KIQGv028072; Thu, 2 Mar 2017 20:18:26 GMT
Received: from dhcp184.cthon.org (/70.213.12.118) by default (Oracle Beehive Gateway v4.0) with ESMTP ; Thu, 02 Mar 2017 12:18:25 -0800
Content-Type: text/plain; charset="us-ascii"
Mime-Version: 1.0 (Mac OS X Mail 9.3 \(3124\))
From: Chuck Lever <chuck.lever@oracle.com>
In-Reply-To: <148841062843.7079.17570170832542180553.idtracker@ietfa.amsl.com>
Date: Thu, 02 Mar 2017 12:18:23 -0800
Content-Transfer-Encoding: 7bit
Message-Id: <96C67098-9B19-43DE-86AE-44D12EBE8B48@oracle.com>
References: <148841062843.7079.17570170832542180553.idtracker@ietfa.amsl.com>
To: Ben Campbell <ben@nostrum.com>
X-Mailer: Apple Mail (2.3124)
X-Source-IP: aserv0022.oracle.com [141.146.126.234]
Archived-At: <https://mailarchive.ietf.org/arch/msg/nfsv4/NpKDRRY6GvxNIRcM72Yy1Sadtig>
Cc: nfsv4-chairs@ietf.org, draft-ietf-nfsv4-rpcrdma-bidirection@ietf.org, The IESG <iesg@ietf.org>, nfsv4@ietf.org
Subject: Re: [nfsv4] Ben Campbell's No Objection on draft-ietf-nfsv4-rpcrdma-bidirection-07: (with COMMENT)
X-BeenThere: nfsv4@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: NFSv4 Working Group <nfsv4.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/nfsv4>, <mailto:nfsv4-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/nfsv4/>
List-Post: <mailto:nfsv4@ietf.org>
List-Help: <mailto:nfsv4-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/nfsv4>, <mailto:nfsv4-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 02 Mar 2017 20:18:32 -0000

> On Mar 1, 2017, at 3:23 PM, Ben Campbell <ben@nostrum.com> wrote:
> 
> Ben Campbell has entered the following ballot position for
> draft-ietf-nfsv4-rpcrdma-bidirection-07: No Objection
> 
> When responding, please keep the subject line intact and reply to all
> email addresses included in the To and CC lines. (Feel free to cut this
> introductory paragraph, however.)
> 
> 
> Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
> for more information about IESG DISCUSS and COMMENT positions.
> 
> 
> The document, along with other ballot positions, can be found here:
> https://datatracker.ietf.org/doc/draft-ietf-nfsv4-rpcrdma-bidirection/
> 
> 
> 
> ----------------------------------------------------------------------
> COMMENT:
> ----------------------------------------------------------------------
> 
> Some minor comments:
> 
> -4.1: This is the first mention of "credits", and there is no definition.
> I realize that the term is defined in the reference from the previous
> section. It would be helpful to mention that in the context of that
> reference.

I can add that.


> --  Are there any head-of-line-blocking issues introduced by
> bidirectional transactions? For example, can a reply get stuck behind
> requests that are blocked by flow control?

Head-of-line-blocking issues that affect one direction
should have no impact on the other direction because the
two directions have independent credit accounting
mechanisms.


> -5.4, 4th paragraph, last sentence: Can a reverse requestor reasonably
> give up or time out, rather than wait "indefinitely"?

Yes, a timeout is a reasonable response to this situation.
I can soften this sentence to include that possibility.


> -8:  This implies that reverse direction transactions do not change
> anything.If that is the case, please say so explicitly. For example, Is
> there any change to authentication for reverse calls?

Reverse calls would make use of a separate authentication
mechanism and credentials, but would otherwise operate in
the same fashion as requests in the forward direction. I
can add a sentence that states this.


> I am not an expert
> in direct memory access transport protocols; are there every situation
> where authentication depends on an initial request from the client?

The authentication mechanisms are handled by the RPC layer.
There is no authentication (or integrity or confidentiality
protection) in RPC transports, whose only responsibility is
moving bytes. I can add an introductory sentence that states
this.


--
Chuck Lever