diff options
| author | Alexander Aring <aahringo@redhat.com> | 2022-05-02 11:14:09 -0400 |
|---|---|---|
| committer | David Teigland <teigland@redhat.com> | 2022-05-02 11:22:56 -0500 |
| commit | 9502a7f688fe7bff297b4e1b64622e0da9b27d78 (patch) | |
| tree | 915e72a717f326c3d008cdfdf4379e2385204ec1 /fs/dlm/lock.c | |
| parent | 0ccc106052715617015277b596bd1a591bbe4416 (diff) | |
| download | linux-9502a7f688fe7bff297b4e1b64622e0da9b27d78.tar.gz | |
dlm: use kref_put_lock in put_rsb
This patch will optimize put_rsb() by using kref_put_lock(). The
function kref_put_lock() will only take the lock if the reference is
going to be zero, if not the lock will never be held.
Signed-off-by: Alexander Aring <aahringo@redhat.com>
Signed-off-by: David Teigland <teigland@redhat.com>
Diffstat (limited to 'fs/dlm/lock.c')
| -rw-r--r-- | fs/dlm/lock.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/fs/dlm/lock.c b/fs/dlm/lock.c index 97ca728dc1941..a331210434b2b 100644 --- a/fs/dlm/lock.c +++ b/fs/dlm/lock.c @@ -350,10 +350,12 @@ static void put_rsb(struct dlm_rsb *r) { struct dlm_ls *ls = r->res_ls; uint32_t bucket = r->res_bucket; + int rv; - spin_lock(&ls->ls_rsbtbl[bucket].lock); - kref_put(&r->res_ref, toss_rsb); - spin_unlock(&ls->ls_rsbtbl[bucket].lock); + rv = kref_put_lock(&r->res_ref, toss_rsb, + &ls->ls_rsbtbl[bucket].lock); + if (rv) + spin_unlock(&ls->ls_rsbtbl[bucket].lock); } void dlm_put_rsb(struct dlm_rsb *r) |