diff options
author | Trond Myklebust <trond.myklebust@primarydata.com> | 2014-07-30 08:27:33 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2014-08-05 10:55:18 -0400 |
commit | cb86fb1428499160eea1be7612127952eec69122 (patch) | |
tree | 17dfd150a4e97503b0eb29c4118283171b42b027 | |
parent | 3974552dcea94619b0f51c5a52f90671067cbcec (diff) |
nfsd: Remove nfs4_lock_state(): setclientid, setclientid_confirm, renew
Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
-rw-r--r-- | fs/nfsd/nfs4state.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c index e6b27ede3388..a4ae38b61933 100644 --- a/fs/nfsd/nfs4state.c +++ b/fs/nfsd/nfs4state.c | |||
@@ -2948,7 +2948,6 @@ nfsd4_setclientid(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate, | |||
2948 | if (new == NULL) | 2948 | if (new == NULL) |
2949 | return nfserr_jukebox; | 2949 | return nfserr_jukebox; |
2950 | /* Cases below refer to rfc 3530 section 14.2.33: */ | 2950 | /* Cases below refer to rfc 3530 section 14.2.33: */ |
2951 | nfs4_lock_state(); | ||
2952 | spin_lock(&nn->client_lock); | 2951 | spin_lock(&nn->client_lock); |
2953 | conf = find_confirmed_client_by_name(&clname, nn); | 2952 | conf = find_confirmed_client_by_name(&clname, nn); |
2954 | if (conf) { | 2953 | if (conf) { |
@@ -2983,7 +2982,6 @@ nfsd4_setclientid(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate, | |||
2983 | status = nfs_ok; | 2982 | status = nfs_ok; |
2984 | out: | 2983 | out: |
2985 | spin_unlock(&nn->client_lock); | 2984 | spin_unlock(&nn->client_lock); |
2986 | nfs4_unlock_state(); | ||
2987 | if (new) | 2985 | if (new) |
2988 | free_client(new); | 2986 | free_client(new); |
2989 | if (unconf) | 2987 | if (unconf) |
@@ -3006,7 +3004,6 @@ nfsd4_setclientid_confirm(struct svc_rqst *rqstp, | |||
3006 | 3004 | ||
3007 | if (STALE_CLIENTID(clid, nn)) | 3005 | if (STALE_CLIENTID(clid, nn)) |
3008 | return nfserr_stale_clientid; | 3006 | return nfserr_stale_clientid; |
3009 | nfs4_lock_state(); | ||
3010 | 3007 | ||
3011 | spin_lock(&nn->client_lock); | 3008 | spin_lock(&nn->client_lock); |
3012 | conf = find_confirmed_client(clid, false, nn); | 3009 | conf = find_confirmed_client(clid, false, nn); |
@@ -3056,7 +3053,6 @@ out: | |||
3056 | spin_unlock(&nn->client_lock); | 3053 | spin_unlock(&nn->client_lock); |
3057 | if (old) | 3054 | if (old) |
3058 | expire_client(old); | 3055 | expire_client(old); |
3059 | nfs4_unlock_state(); | ||
3060 | return status; | 3056 | return status; |
3061 | } | 3057 | } |
3062 | 3058 | ||
@@ -4113,7 +4109,6 @@ nfsd4_renew(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate, | |||
4113 | __be32 status; | 4109 | __be32 status; |
4114 | struct nfsd_net *nn = net_generic(SVC_NET(rqstp), nfsd_net_id); | 4110 | struct nfsd_net *nn = net_generic(SVC_NET(rqstp), nfsd_net_id); |
4115 | 4111 | ||
4116 | nfs4_lock_state(); | ||
4117 | dprintk("process_renew(%08x/%08x): starting\n", | 4112 | dprintk("process_renew(%08x/%08x): starting\n", |
4118 | clid->cl_boot, clid->cl_id); | 4113 | clid->cl_boot, clid->cl_id); |
4119 | status = lookup_clientid(clid, cstate, nn); | 4114 | status = lookup_clientid(clid, cstate, nn); |
@@ -4126,7 +4121,6 @@ nfsd4_renew(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate, | |||
4126 | goto out; | 4121 | goto out; |
4127 | status = nfs_ok; | 4122 | status = nfs_ok; |
4128 | out: | 4123 | out: |
4129 | nfs4_unlock_state(); | ||
4130 | return status; | 4124 | return status; |
4131 | } | 4125 | } |
4132 | 4126 | ||