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
- [nfsv4] Benjamin Kaduk's No Objection on draft-ie… Benjamin Kaduk via Datatracker
- Re: [nfsv4] Benjamin Kaduk's No Objection on draf… Chuck Lever
- Re: [nfsv4] Benjamin Kaduk's No Objection on draf… Benjamin Kaduk
- Re: [nfsv4] Benjamin Kaduk's No Objection on draf… Chuck Lever
- Re: [nfsv4] Benjamin Kaduk's No Objection on draf… Benjamin Kaduk
- Re: [nfsv4] Benjamin Kaduk's No Objection on draf… Tom Talpey
- Re: [nfsv4] Benjamin Kaduk's No Objection on draf… Chuck Lever
- Re: [nfsv4] Benjamin Kaduk's No Objection on draf… Chuck Lever
- Re: [nfsv4] Benjamin Kaduk's No Objection on draf… Tom Talpey
- Re: [nfsv4] Benjamin Kaduk's No Objection on draf… Chuck Lever