[nfsv4] AD Evaluation for draft-ietf-nfsv4-rpcrdma-bidirection-05

Spencer Dawkins at IETF <spencerdawkins.ietf@gmail.com> Tue, 10 January 2017 21:32 UTC

Return-Path: <spencerdawkins.ietf@gmail.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 D9A1A129DCD; Tue, 10 Jan 2017 13:32:52 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.999
X-Spam-Level:
X-Spam-Status: No, score=-1.999 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com
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 xYispr8a8xz0; Tue, 10 Jan 2017 13:32:51 -0800 (PST)
Received: from mail-yb0-x229.google.com (mail-yb0-x229.google.com [IPv6:2607:f8b0:4002:c09::229]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 8EC3F12956B; Tue, 10 Jan 2017 13:32:46 -0800 (PST)
Received: by mail-yb0-x229.google.com with SMTP id c125so4103513ybf.1; Tue, 10 Jan 2017 13:32:46 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:from:date:message-id:subject:to:cc; bh=DxfxQ1kzYss30SEOisdr3L0dRWwRd3J/o0MJmfS9zbA=; b=VgTbRE7BmKQlYW0xFgBP+8LUL+KnV4GYVw6iwCSMrRL5FGl2b52/JQNVhagTTAsZ5V pyzWXbcKvnGZEXDd2Em9UM70LvW0iofZncZrUg2EW+6bzx/60l63cAZXxb9EnCReM+0a 1I9mGtPiiustLWAWqqOZZKwN00Xb4R7xk/f2VhIOKZILPJaQgD5/mhTnjX4ZOL8aMuO3 EJ2vzj9hB+IW9KcMAIk2X6z9FMvGKyH+ZaDpzWFKwjhwl1V275gO2dCZDtAlah4KpR0m 0sx8R4GXnvAYMi+ePg/xb3qXBSExGLAL4G1ni0AcD5zE7XA97vuQ8xvZiNA7N3cAQ/oV 4QLQ==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:from:date:message-id:subject:to:cc; bh=DxfxQ1kzYss30SEOisdr3L0dRWwRd3J/o0MJmfS9zbA=; b=d5o51RTMCux8osV0tW7aNLV2fpDeWD+p07yWRYPkkGl5qMn80MgEQrMve8xHyTEUfe l/FOKYHrurVPetNnQk79APoq9DsKKss2ddJJ2W1vD6amiCAlSyFeqI/h9+tg6pcjUa1n qOtZ8t6/EiFzJbk5j0L4MlGk4WgJ478uPeEVrKtebCacCzW9AayRGyhN435ODBFGWLiC YBMalB3A16VwuZBeMrUQPeNup4rNwRT+tRCMsD/NbbvGWIJjebGWSRXFmf1pi2Jc6H+4 sGUXmUVOR66L+TGXu7Dr7OZlyksxU6GZmmuRY3Rtqivzj4YpK5/pUfctEeBfaSWfLcml Zlzw==
X-Gm-Message-State: AIkVDXLbCnv4hLmOJ15K+2E2v8l1dFgcAM4KbeVP9ST37mZ7O/a9/zzl0htOBFCARcSFo72FKN2peUagL7qLSA==
X-Received: by 10.37.204.193 with SMTP id l184mr1899328ybf.182.1484083965596; Tue, 10 Jan 2017 13:32:45 -0800 (PST)
MIME-Version: 1.0
Received: by 10.37.221.195 with HTTP; Tue, 10 Jan 2017 13:32:45 -0800 (PST)
From: Spencer Dawkins at IETF <spencerdawkins.ietf@gmail.com>
Date: Tue, 10 Jan 2017 15:32:45 -0600
Message-ID: <CAKKJt-fuKMwX06PerWzxBdBqQ_=eMvhQKUdSDb5xLsSX47q=yw@mail.gmail.com>
To: Spencer Shepler <sshepler@microsoft.com>
Content-Type: multipart/alternative; boundary="94eb2c083aa69647e70545c4398e"
Archived-At: <https://mailarchive.ietf.org/arch/msg/nfsv4/i5SDpoGi_gIWEHgEymlmD30Q6IY>
Cc: draft-ietf-nfsv4-rpcrdma-bidirection@ietf.org, NFSv4 <nfsv4@ietf.org>
Subject: [nfsv4] AD Evaluation for draft-ietf-nfsv4-rpcrdma-bidirection-05
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: Tue, 10 Jan 2017 21:32:53 -0000

Hi, Spencer (S),

I think this catches me up on nfsv4 AD Evaluations ...

Again, this draft was pretty clear to me, but I did have some questions
about wording, that I'd like people to look at, before requesting IETF Last
Call.

Spencer (S), could you let me know when this draft is good to go?

Thanks!

Spencer (D)

--- AD Evaluation

Could you spell out "Parallel NFS" on first use of pNFS? Or whatever pNFS
stands for :-)

Probably just for my benefit, but since you mention NAT routers in this
text,

   To facilitate operation through NAT routers, all NFSv4.1 transport
   connections are initiated by NFSv4.1 clients.  Therefore NFSv4.1
   servers send callbacks to clients in the backward direction on
   connections established by NFSv4.1 clients.

is it obvious which end has responsibility for NAT binding keep-alives? I'm
guessing that's a client responsibility, but that's just a guess.

In Sections 4.3.1 and 4.3.2, I THINK you're saying that clients and servers
both need to pre-post more receive buffers to accommodate bidirectional
operation, but that's more clear in 4.3.1,

   To receive incoming backward direction Calls, an RPC-over-RDMA client
   endpoint must pre-post enough additional receive buffers to match its
   advertised backward direction credit value.  Each outstanding forward
   direction RPC requires an additional receive buffer above this
   minimum.

than the corresponding text in 4.3.2,

   To receive incoming backward direction replies, an RPC-over-RDMA
   server endpoint must pre-post a receive buffer for each backward
   direction Call it sends.

because "additional" appears twice in the text on 4.3.1, but doesn't appear
at all in 4.3.2. If this is as symmetrical as I'm thinking it is, perhaps
it's clearer to use similar wording in both places.

In this text,

   When message direction is not fully determined by context (e.g.,
   suggested by the definition of the RPC-over-RDMA version that is in
   use) or by an accompanying RPC message payload with a call direction
   field, it is not possible for the receiver to tell with certainty
   whether the header credit value is a request or grant.  In such
   cases, the receiver MUST NOT use the header's credit value.

does RDMA work at all, if the credit value can't be used?