diff options
| author | Sage Weil <sage@newdream.net> | 2011-05-27 18:24:26 +0200 |
|---|---|---|
| committer | Sage Weil <sage@newdream.net> | 2011-06-08 06:34:11 +0200 |
| commit | 70b666c3b4cb2b96098d80e6f515e4bc6d37db5a (patch) | |
| tree | dd82036114adf9d0c28120500d58833cacfbc004 /fs/ceph/snap.c | |
| parent | ceph: fix cap flush race reentrancy (diff) | |
| download | linux-70b666c3b4cb2b96098d80e6f515e4bc6d37db5a.tar.xz linux-70b666c3b4cb2b96098d80e6f515e4bc6d37db5a.zip | |
ceph: use ihold when we already have an inode ref
We should use ihold whenever we already have a stable inode ref, even
when we aren't holding i_lock. This avoids adding new and unnecessary
locking dependencies.
Signed-off-by: Sage Weil <sage@newdream.net>
Diffstat (limited to 'fs/ceph/snap.c')
| -rw-r--r-- | fs/ceph/snap.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/ceph/snap.c b/fs/ceph/snap.c index 24067d68a554..54b14de2e729 100644 --- a/fs/ceph/snap.c +++ b/fs/ceph/snap.c @@ -722,7 +722,7 @@ static void flush_snaps(struct ceph_mds_client *mdsc) ci = list_first_entry(&mdsc->snap_flush_list, struct ceph_inode_info, i_snap_flush_item); inode = &ci->vfs_inode; - igrab(inode); + ihold(inode); spin_unlock(&mdsc->snap_flush_lock); spin_lock(&inode->i_lock); __ceph_flush_snaps(ci, &session, 0); |