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; |
