diff options
author | Chuck Lever <chuck.lever@oracle.com> | 2012-05-21 22:44:41 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2012-05-22 16:45:43 -0400 |
commit | 591555465ec513c42416392d392fd56866cb220c (patch) | |
tree | 8b7ce3d4fe39e31e48e3e7a1a604675395607888 /fs/nfs/nfs4proc.c | |
parent | 79d4e1f0d8910f0214a57832ca6d589640d572c0 (diff) |
NFS: Use proper naming conventions for nfs_client.impl_id field
Clean up: When naming fields and data types, follow established
conventions to facilitate accurate grep/cscope searches.
Additionally, for consistency, move the impl_id field into the NFSv4-
specific part of the nfs_client, and free that memory in the logic
that shuts down NFSv4 nfs_clients.
Introduced by commit 7d2ed9ac "NFSv4: parse and display server
implementation ids," Fri Feb 17, 2012.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'fs/nfs/nfs4proc.c')
-rw-r--r-- | fs/nfs/nfs4proc.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/fs/nfs/nfs4proc.c b/fs/nfs/nfs4proc.c index 94494f24bb12..daa4e1b17313 100644 --- a/fs/nfs/nfs4proc.c +++ b/fs/nfs/nfs4proc.c | |||
@@ -5119,8 +5119,8 @@ int nfs4_proc_exchange_id(struct nfs_client *clp, struct rpc_cred *cred) | |||
5119 | 5119 | ||
5120 | if (!status) { | 5120 | if (!status) { |
5121 | /* use the most recent implementation id */ | 5121 | /* use the most recent implementation id */ |
5122 | kfree(clp->impl_id); | 5122 | kfree(clp->cl_implid); |
5123 | clp->impl_id = res.impl_id; | 5123 | clp->cl_implid = res.impl_id; |
5124 | } else | 5124 | } else |
5125 | kfree(res.impl_id); | 5125 | kfree(res.impl_id); |
5126 | 5126 | ||
@@ -5144,12 +5144,12 @@ int nfs4_proc_exchange_id(struct nfs_client *clp, struct rpc_cred *cred) | |||
5144 | out_server_scope: | 5144 | out_server_scope: |
5145 | kfree(res.server_scope); | 5145 | kfree(res.server_scope); |
5146 | out: | 5146 | out: |
5147 | if (clp->impl_id) | 5147 | if (clp->cl_implid) |
5148 | dprintk("%s: Server Implementation ID: " | 5148 | dprintk("%s: Server Implementation ID: " |
5149 | "domain: %s, name: %s, date: %llu,%u\n", | 5149 | "domain: %s, name: %s, date: %llu,%u\n", |
5150 | __func__, clp->impl_id->domain, clp->impl_id->name, | 5150 | __func__, clp->cl_implid->domain, clp->cl_implid->name, |
5151 | clp->impl_id->date.seconds, | 5151 | clp->cl_implid->date.seconds, |
5152 | clp->impl_id->date.nseconds); | 5152 | clp->cl_implid->date.nseconds); |
5153 | dprintk("<-- %s status= %d\n", __func__, status); | 5153 | dprintk("<-- %s status= %d\n", __func__, status); |
5154 | return status; | 5154 | return status; |
5155 | } | 5155 | } |