Re: [nfsv4] Benjamin Kaduk's No Objection on draft-ietf-nfsv4-rpcrdma-cm-pvt-data-07: (with COMMENT)

Benjamin Kaduk <kaduk@mit.edu> Wed, 26 February 2020 19:43 UTC

Return-Path: <kaduk@mit.edu>
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 8503F3A12FE; Wed, 26 Feb 2020 11:43:37 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.9
X-Spam-Level:
X-Spam-Status: No, score=-1.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, SPF_HELO_NONE=0.001, SPF_PASS=-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 5le_gbJiCYnj; Wed, 26 Feb 2020 11:43:35 -0800 (PST)
Received: from outgoing.mit.edu (outgoing-auth-1.mit.edu [18.9.28.11]) (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 8410C3A12D0; Wed, 26 Feb 2020 11:43:35 -0800 (PST)
Received: from kduck.mit.edu ([24.16.140.251]) (authenticated bits=56) (User authenticated as kaduk@ATHENA.MIT.EDU) by outgoing.mit.edu (8.14.7/8.12.4) with ESMTP id 01QJhUMZ024886 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 26 Feb 2020 14:43:32 -0500
Date: Wed, 26 Feb 2020 11:43:30 -0800
From: Benjamin Kaduk <kaduk@mit.edu>
To: Chuck Lever <chuck.lever@oracle.com>
Cc: nfsv4-chairs@ietf.org, nfsv4@ietf.org, The IESG <iesg@ietf.org>, draft-ietf-nfsv4-rpcrdma-cm-pvt-data@ietf.org
Message-ID: <20200226194330.GT56312@kduck.mit.edu>
References: <158216101788.17661.6926758160404035704.idtracker@ietfa.amsl.com> <E718C1B8-B96B-4A64-8B92-C4ABEC7B5E5F@oracle.com> <20200220195155.GJ97652@kduck.mit.edu> <0DB10D2E-CA76-49EA-8B7F-53F502114684@oracle.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <0DB10D2E-CA76-49EA-8B7F-53F502114684@oracle.com>
User-Agent: Mutt/1.12.1 (2019-06-15)
Archived-At: <https://mailarchive.ietf.org/arch/msg/nfsv4/GzxB1_9ayIcubE50TkS26p5Yo3Y>
Subject: Re: [nfsv4] Benjamin Kaduk's No Objection on draft-ietf-nfsv4-rpcrdma-cm-pvt-data-07: (with COMMENT)
X-BeenThere: nfsv4@ietf.org
X-Mailman-Version: 2.1.29
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: Wed, 26 Feb 2020 19:43:43 -0000

Hi Chuck,

On Wed, Feb 26, 2020 at 08:33:21AM -0800, Chuck Lever wrote:
> Hi Ben-
> 
> > On Feb 20, 2020, at 11:51 AM, Benjamin Kaduk <kaduk@MIT.EDU> wrote:
> > 
> > Hi Chuck,
> > 
> > On Thu, Feb 20, 2020 at 10:13:26AM -0500, Chuck Lever wrote:
> >> 
> >> 
> >>> Section 4
> >>> 
> >>>  For RPC-over-RDMA version 1, the CM Private Data field is formatted
> >>>  as described in the following subsection.  RPC clients and servers
> >>>  use the same format.  If the capacity of the Private Data field is
> >>>  too small to contain this message format, the underlying RDMA
> >>>  transport is not managed by a Connection Manager, or the underlying
> >>>  RDMA transport uses Private Data for its own purposes, the CM Private
> >>>  Data field cannot be used on behalf of RPC-over-RDMA version 1.
> >>> 
> >>> How will an implementation know if "the underlying RDMA transport uses
> >>> Private Data for its own purposes"?
> >> 
> >> That's what the Format Identifier is for. It indicates where the
> >> RPC/RDMA-specific data sits in the CM Private Data buffer. A
> >> compliant RPC/RDMA implementation does not assume it is the only
> >> user of the CM Private Data.
> > 
> > Sure ... but now we have some text saying that other uses of CM Private
> > Data can't coexist with RPC-over-RDMA v1 ("the underlying RDMA transport
> > uses Private Data for its own purposes [...] cannot be used on behalf of
> > RPC-over-RDMA version 1") that seems in conflict with the goal of using the
> > Format Identifier "magic word" to identify the RPC-over-RDMA private data
> > within a larger assembly.  So we should probably try to resolve the
> > apparent inconsistency.
> 
> I want to make sure you are happy with how I addressed this comment.
> The third paragraph of Section 4 in -08 now reads:
> 
>    For RPC-over-RDMA version 1, the CM Private Data field is formatted
>    as described in the following subsection.  RPC clients and servers
>    use the same format.  If the capacity of the Private Data field is
>    too small to contain this message format or the underlying RDMA
>    transport is not managed by a Connection Manager, the CM Private Data
>    field cannot be used on behalf of RPC-over-RDMA version 1.
> 
> I've removed the conflicting item from the last sentence.
> 
> Please confirm this is a satisfactory solution.

That should work.

Thanks!

-Ben