summaryrefslogtreecommitdiffstats
path: root/fs/gfs2/ops_inode.c
diff options
context:
space:
mode:
authorS. Wendy Cheng <wcheng@redhat.com>2007-01-18 15:56:34 -0500
committerSteven Whitehouse <swhiteho@redhat.com>2007-02-05 13:36:15 -0500
commit5509826f1e548d14bb888c1cb6e3bbf23f855770 (patch)
tree20915fb965f5895f3a41361bdb6182ae10db242c /fs/gfs2/ops_inode.c
parente1d5b18ae92d0bbfe66dc2b4bab65006d32c5f7d (diff)
downloadop-kernel-dev-5509826f1e548d14bb888c1cb6e3bbf23f855770.zip
op-kernel-dev-5509826f1e548d14bb888c1cb6e3bbf23f855770.tar.gz
[GFS2] Fix change nlink deadlock
Bugzilla 215088 Fix deadlock in gfs2_change_nlink() while installing RHEL5 into GFS2 partition. The gfs2_rename() apparently needs block allocation for the new name (into the directory) where it requires rg locks. At the same time, while updating the nlink count for the replaced file, gfs2_change_nlink() tries to return the inode meta-data back to resource group where it needs rg locks too. Our logic doesn't allow process to acquire these locks recursively by the same process (RHEL installer) that results a BUG call. This only happens within rename code path and only if the destination file exists before the rename operation. 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, 22 insertions, 3 deletions
diff --git a/fs/gfs2/ops_inode.c b/fs/gfs2/ops_inode.c
index 636dda4..919e894 100644
--- a/fs/gfs2/ops_inode.c
+++ b/fs/gfs2/ops_inode.c
@@ -553,6 +553,7 @@ 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);
@@ -684,12 +685,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, 0);
+ RES_STATFS + RES_QUOTA + 1, 0);
if (error)
goto out_ipreserv;
} else {
error = gfs2_trans_begin(sdp, 4 * RES_DINODE +
- 5 * RES_LEAF, 0);
+ 5 * RES_LEAF + 1, 0);
if (error)
goto out_gunlock;
}
@@ -703,7 +704,25 @@ 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(nip, -1);
+ 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);
+ }
+ }
+ }
}
if (error)
goto out_end_trans;
OpenPOWER on IntegriCloud