Commit 2dd7f2ad authored by Trond Myklebust's avatar Trond Myklebust Committed by J. Bruce Fields

nfsd: Remove nfs4_lock_state(): nfsd4_lock/locku/lockt()

Signed-off-by: default avatarTrond Myklebust <trond.myklebust@primarydata.com>
Signed-off-by: default avatarJ. Bruce Fields <bfields@redhat.com>
parent 51f5e783
...@@ -5205,8 +5205,6 @@ nfsd4_lock(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate, ...@@ -5205,8 +5205,6 @@ nfsd4_lock(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
return status; return status;
} }
nfs4_lock_state();
if (lock->lk_is_new) { if (lock->lk_is_new) {
if (nfsd4_has_session(cstate)) if (nfsd4_has_session(cstate))
/* See rfc 5661 18.10.3: given clientid is ignored: */ /* See rfc 5661 18.10.3: given clientid is ignored: */
...@@ -5349,7 +5347,6 @@ nfsd4_lock(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate, ...@@ -5349,7 +5347,6 @@ nfsd4_lock(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
if (open_stp) if (open_stp)
nfs4_put_stid(&open_stp->st_stid); nfs4_put_stid(&open_stp->st_stid);
nfsd4_bump_seqid(cstate, status); nfsd4_bump_seqid(cstate, status);
nfs4_unlock_state();
if (file_lock) if (file_lock)
locks_free_lock(file_lock); locks_free_lock(file_lock);
if (conflock) if (conflock)
...@@ -5392,8 +5389,6 @@ nfsd4_lockt(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate, ...@@ -5392,8 +5389,6 @@ nfsd4_lockt(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
if (check_lock_length(lockt->lt_offset, lockt->lt_length)) if (check_lock_length(lockt->lt_offset, lockt->lt_length))
return nfserr_inval; return nfserr_inval;
nfs4_lock_state();
if (!nfsd4_has_session(cstate)) { if (!nfsd4_has_session(cstate)) {
status = lookup_clientid(&lockt->lt_clientid, cstate, nn); status = lookup_clientid(&lockt->lt_clientid, cstate, nn);
if (status) if (status)
...@@ -5448,7 +5443,6 @@ nfsd4_lockt(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate, ...@@ -5448,7 +5443,6 @@ nfsd4_lockt(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
out: out:
if (lo) if (lo)
nfs4_put_stateowner(&lo->lo_owner); nfs4_put_stateowner(&lo->lo_owner);
nfs4_unlock_state();
if (file_lock) if (file_lock)
locks_free_lock(file_lock); locks_free_lock(file_lock);
return status; return status;
...@@ -5472,8 +5466,6 @@ nfsd4_locku(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate, ...@@ -5472,8 +5466,6 @@ nfsd4_locku(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
if (check_lock_length(locku->lu_offset, locku->lu_length)) if (check_lock_length(locku->lu_offset, locku->lu_length))
return nfserr_inval; return nfserr_inval;
nfs4_lock_state();
status = nfs4_preprocess_seqid_op(cstate, locku->lu_seqid, status = nfs4_preprocess_seqid_op(cstate, locku->lu_seqid,
&locku->lu_stateid, NFS4_LOCK_STID, &locku->lu_stateid, NFS4_LOCK_STID,
&stp, nn); &stp, nn);
...@@ -5516,7 +5508,6 @@ nfsd4_locku(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate, ...@@ -5516,7 +5508,6 @@ nfsd4_locku(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
nfs4_put_stid(&stp->st_stid); nfs4_put_stid(&stp->st_stid);
out: out:
nfsd4_bump_seqid(cstate, status); nfsd4_bump_seqid(cstate, status);
nfs4_unlock_state();
if (file_lock) if (file_lock)
locks_free_lock(file_lock); locks_free_lock(file_lock);
return status; return status;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment