On Jan 3, 2015, at 4:20 PM, Trond Myklebust <trond.myklebust@xxxxxxxxxxxxxxx> wrote: > On Sat, Jan 3, 2015 at 2:45 PM, Chuck Lever <chuck.lever@xxxxxxxxxx> wrote: >> Detect server trunking across transport protocols. Otherwise, an >> RDMA mount and a TCP mount of the same server will end up with >> separate nfs_clients using the same clientid4. >> >> Reported-by: Dai Ngo <dai.ngo@xxxxxxxxxx> >> Signed-off-by: Chuck Lever <chuck.lever@xxxxxxxxxx> >> --- >> Re: yesterday's discussion of "[PATCH] NFSv4.1: Fix client id >> trunking on Linux". This patch seems to address the behavior I >> mentioned. >> >> fs/nfs/nfs4client.c | 6 ------ >> 1 file changed, 6 deletions(-) >> >> diff --git a/fs/nfs/nfs4client.c b/fs/nfs/nfs4client.c >> index c0fe338..c7bc38e 100644 >> --- a/fs/nfs/nfs4client.c >> +++ b/fs/nfs/nfs4client.c >> @@ -540,9 +540,6 @@ int nfs40_walk_client_list(struct nfs_client *new, >> if (pos->rpc_ops != new->rpc_ops) >> continue; >> >> - if (pos->cl_proto != new->cl_proto) >> - continue; >> - >> if (pos->cl_minorversion != new->cl_minorversion) >> continue; >> >> @@ -675,9 +672,6 @@ int nfs41_walk_client_list(struct nfs_client *new, >> if (pos->rpc_ops != new->rpc_ops) >> continue; >> >> - if (pos->cl_proto != new->cl_proto) >> - continue; >> - >> if (pos->cl_minorversion != new->cl_minorversion) >> continue; >> >> > > Yep. Sorry, I only noticed this after I'd written my own version. I'll > replace mine with this one. Thanks, much appreciated! -- Chuck Lever chuck[dot]lever[at]oracle[dot]com -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
