From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from userp2120.oracle.com ([156.151.31.85]:39686 "EHLO userp2120.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751546AbeFFQDQ (ORCPT ); Wed, 6 Jun 2018 12:03:16 -0400 Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 11.3 \(3445.6.18\)) Subject: Re: [PATCH v1 2/2] NFSv4.0: Remove transport protocol name from non-UCS client ID From: Chuck Lever In-Reply-To: <6b892b9ad11d33f59de22427092dc41673313fa2.camel@hammerspace.com> Date: Wed, 6 Jun 2018 12:03:09 -0400 Cc: Linux NFS Mailing List Message-Id: <8F56FFE7-F5DE-4F22-BA0C-A6C7188435B1@oracle.com> References: <20180604144154.11877.19298.stgit@manet.1015granger.net> <20180604145334.11877.12862.stgit@manet.1015granger.net> <73D2F6EC-8CE6-4290-9713-6D5B8B0DC2E3@oracle.com> <6b892b9ad11d33f59de22427092dc41673313fa2.camel@hammerspace.com> To: Trond Myklebust Sender: linux-nfs-owner@vger.kernel.org List-ID: > On Jun 6, 2018, at 11:55 AM, Trond Myklebust = wrote: >=20 > On Wed, 2018-06-06 at 11:48 -0400, Chuck Lever wrote: >>> On Jun 6, 2018, at 11:44 AM, Trond Myklebust >> om> wrote: >>>=20 >>> On Mon, 2018-06-04 at 10:53 -0400, Chuck Lever wrote: >>>> Commit 69dd716c5ffd ("NFSv4: Add socket proto argument to >>>> setclientid") (2007) added the transport protocol name to the >>>> client >>>> ID string, but the patch description doesn't explain why this was >>>> necessary. >>>>=20 >>>> At that time, the only transport protocol name that would have >>>> been >>>> used is "tcp" (for both IPv4 and IPv6), resulting in no >>>> additional >>>> distinctiveness of the client ID string. >>>>=20 >>>> Since there is one client instance, the server should recognize >>>> it's >>>> state whether the client is connecting via TCP or RDMA. Same >>>> client, >>>> same lease. >>>=20 >>> The reason why this is the case now is because the trunking code >>> overrides the guardrails in nfs_get_client(). The latter does match >>> on >>> the protocol. >>=20 >> OK. Would that also true in the UCS case? >>=20 >=20 > Yes, I believe that code is independent of the migration flag. But the UCS does not include the transport type. Maybe something is broken here (that something could be my understanding). Seems to me that could impact transparent state migration between a server that is mounted with RDMA, and one that has only TCP. I'm willing to drop 2/2. -- Chuck Lever