diff options
| author | Bob Peterson <rpeterso@redhat.com> | 2015-12-04 12:57:00 -0600 |
|---|---|---|
| committer | Bob Peterson <rpeterso@redhat.com> | 2015-12-18 11:02:52 -0600 |
| commit | 5ea31bc0a6524b4fee8dc9ae8005d4a114a79812 (patch) | |
| tree | 27ee8727bcd8b74b71f91a1caa1b8153a8e096a2 /fs/gfs2/rgrp.c | |
| parent | 783013c0f5c7263a31703b15aeebbac279b4d4fe (diff) | |
| download | linux-5ea31bc0a6524b4fee8dc9ae8005d4a114a79812.tar.gz | |
GFS2: Always use iopen glock for gl_deletes
Before this patch, when function try_rgrp_unlink queued a glock for
delete_work to reclaim the space, it used the inode glock to do so.
That's different from the iopen callback which uses the iopen glock
for the same purpose. We should be consistent and always use the
iopen glock. This may also save us reference counting problems with
the inode glock, since clear_glock does an extra glock_put() for the
inode glock.
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Diffstat (limited to 'fs/gfs2/rgrp.c')
| -rw-r--r-- | fs/gfs2/rgrp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/gfs2/rgrp.c b/fs/gfs2/rgrp.c index b879925ce13497..07c0265aa1953a 100644 --- a/fs/gfs2/rgrp.c +++ b/fs/gfs2/rgrp.c @@ -1789,7 +1789,7 @@ static void try_rgrp_unlink(struct gfs2_rgrpd *rgd, u64 *last_unlinked, u64 skip continue; *last_unlinked = block; - error = gfs2_glock_get(sdp, block, &gfs2_inode_glops, CREATE, &gl); + error = gfs2_glock_get(sdp, block, &gfs2_iopen_glops, CREATE, &gl); if (error) continue; |