diff options
author | J. Bruce Fields <bfields@citi.umich.edu> | 2009-02-21 18:39:54 -0500 |
---|---|---|
committer | J. Bruce Fields <bfields@citi.umich.edu> | 2009-03-18 17:38:38 -0400 |
commit | d7fdcfe0aaaf6dffca6fa857bab374182fe7ca8b (patch) | |
tree | f53688b2496a98da148297972b4e27318810ec89 | |
parent | 18f82731b7784ba81ee9b1ed6a8179b577fa898b (diff) |
nfsd4: put_nfs4_client does not require state lock
Since free_client() is guaranteed to only be called once, and to only
touch the client structure itself (not any common data structures), it
has no need for the state lock.
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Cc: Alexandros Batsakis <batsakis@netapp.com>
-rw-r--r-- | fs/nfsd/nfs4callback.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/nfsd/nfs4callback.c b/fs/nfsd/nfs4callback.c index 3ddc9fb2e358..3fd7136321ca 100644 --- a/fs/nfsd/nfs4callback.c +++ b/fs/nfsd/nfs4callback.c | |||
@@ -490,8 +490,8 @@ out_put_cred: | |||
490 | * Success or failure, now we're either waiting for lease expiration | 490 | * Success or failure, now we're either waiting for lease expiration |
491 | * or deleg_return. | 491 | * or deleg_return. |
492 | */ | 492 | */ |
493 | nfs4_lock_state(); | ||
494 | put_nfs4_client(clp); | 493 | put_nfs4_client(clp); |
494 | nfs4_lock_state(); | ||
495 | nfs4_put_delegation(dp); | 495 | nfs4_put_delegation(dp); |
496 | nfs4_unlock_state(); | 496 | nfs4_unlock_state(); |
497 | return; | 497 | return; |