diff options
author | J. Bruce Fields <bfields@redhat.com> | 2011-08-30 22:15:47 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2011-09-01 11:12:47 -0400 |
commit | 7c13f344cf8bec22301c5ed7ef1d90eecb57ba43 (patch) | |
tree | 41ed822e8992c9cc37775902349779f4d5273568 /fs/nfsd/nfs4proc.c | |
parent | fff6ca9cc46857e5814cf687e5fb1b8a876766a4 (diff) |
nfsd4: drop most stateowner refcounting
Maybe we'll bring it back some day, but we don't have much real use for
it now.
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'fs/nfsd/nfs4proc.c')
-rw-r--r-- | fs/nfsd/nfs4proc.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/nfsd/nfs4proc.c b/fs/nfsd/nfs4proc.c index 50063a85f505..ce151f0ed4b9 100644 --- a/fs/nfsd/nfs4proc.c +++ b/fs/nfsd/nfs4proc.c | |||
@@ -405,10 +405,9 @@ nfsd4_open(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate, | |||
405 | */ | 405 | */ |
406 | status = nfsd4_process_open2(rqstp, &cstate->current_fh, open); | 406 | status = nfsd4_process_open2(rqstp, &cstate->current_fh, open); |
407 | out: | 407 | out: |
408 | if (open->op_stateowner) { | 408 | if (open->op_stateowner) |
409 | nfs4_get_stateowner(open->op_stateowner); | ||
410 | cstate->replay_owner = open->op_stateowner; | 409 | cstate->replay_owner = open->op_stateowner; |
411 | } else | 410 | else |
412 | nfs4_unlock_state(); | 411 | nfs4_unlock_state(); |
413 | return status; | 412 | return status; |
414 | } | 413 | } |
@@ -1228,7 +1227,6 @@ encode_op: | |||
1228 | 1227 | ||
1229 | if (cstate->replay_owner) { | 1228 | if (cstate->replay_owner) { |
1230 | nfs4_unlock_state(); | 1229 | nfs4_unlock_state(); |
1231 | nfs4_put_stateowner(cstate->replay_owner); | ||
1232 | cstate->replay_owner = NULL; | 1230 | cstate->replay_owner = NULL; |
1233 | } | 1231 | } |
1234 | /* XXX Ugh, we need to get rid of this kind of special case: */ | 1232 | /* XXX Ugh, we need to get rid of this kind of special case: */ |