summaryrefslogtreecommitdiff
path: root/fs/gfs2/ops_inode.c
diff options
context:
space:
mode:
authorS. Wendy Cheng <wcheng@redhat.com>2007-01-19 00:07:03 +0300
committerSteven Whitehouse <swhiteho@redhat.com>2007-02-05 21:36:31 +0300
commit87d21e07f3880b8d489f0b4a639deb1362101838 (patch)
tree20b41fc6dad45363b81a9e44daf5b278f532f183 /fs/gfs2/ops_inode.c
parent6c93fd1e578669364e026a0d44c669b871e2a8c4 (diff)
downloadlinux-87d21e07f3880b8d489f0b4a639deb1362101838.tar.xz
[GFS2] Fix gfs2_rename deadlock
Second round of gfs2_rename lock re-ordering to allow Anaconda adding root partition on top of gfs2. Previous to this patch the recursive lock detector in glock.c can be triggered due to attempting to lock the rgrp twice. This fixes it by checking to see whether the rgrp is already locked. This fixes Red Hat bugzilla #221237 Signed-off-by: S. Wendy Cheng <wcheng@redhat.com> Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/ops_inode.c')
-rw-r--r--fs/gfs2/ops_inode.c25
1 files changed, 3 insertions, 22 deletions
diff --git a/fs/gfs2/ops_inode.c b/fs/gfs2/ops_inode.c
index 919e8947e710..b2a12f44f59d 100644
--- a/fs/gfs2/ops_inode.c
+++ b/fs/gfs2/ops_inode.c
@@ -553,7 +553,6 @@ static int gfs2_rename(struct inode *odir, struct dentry *odentry,
int alloc_required;
unsigned int x;
int error;
- struct gfs2_rgrpd *rgd;
if (ndentry->d_inode) {
nip = GFS2_I(ndentry->d_inode);
@@ -685,12 +684,12 @@ static int gfs2_rename(struct inode *odir, struct dentry *odentry,
error = gfs2_trans_begin(sdp, sdp->sd_max_dirres +
al->al_rgd->rd_ri.ri_length +
4 * RES_DINODE + 4 * RES_LEAF +
- RES_STATFS + RES_QUOTA + 1, 0);
+ RES_STATFS + RES_QUOTA + 4, 0);
if (error)
goto out_ipreserv;
} else {
error = gfs2_trans_begin(sdp, 4 * RES_DINODE +
- 5 * RES_LEAF + 1, 0);
+ 5 * RES_LEAF + 4, 0);
if (error)
goto out_gunlock;
}
@@ -704,25 +703,7 @@ static int gfs2_rename(struct inode *odir, struct dentry *odentry,
error = gfs2_dir_del(ndip, &ndentry->d_name);
if (error)
goto out_end_trans;
- error = gfs2_change_nlink_i(nip, -1);
- if ((!error) && (nip->i_inode.i_nlink == 0)) {
- error = -EIO;
- rgd = gfs2_blk2rgrpd(sdp, nip->i_num.no_addr);
- if (rgd) {
- struct gfs2_holder nlink_rg_gh;
- if (rgd != nip->i_alloc.al_rgd)
- error = gfs2_glock_nq_init(
- rgd->rd_gl, LM_ST_EXCLUSIVE,
- 0, &nlink_rg_gh);
- else
- error = 0;
- if (!error) {
- gfs2_unlink_di(&nip->i_inode);
- if (rgd != nip->i_alloc.al_rgd)
- gfs2_glock_dq_uninit(&nlink_rg_gh);
- }
- }
- }
+ error = gfs2_change_nlink(nip, -1);
}
if (error)
goto out_end_trans;