diff options
author | Benjamin Marzinski <bmarzins@redhat.com> | 2010-09-27 17:00:04 -0400 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2010-09-28 04:44:24 -0400 |
commit | bf97b6734e027cc18abad420ab88f861f65d7816 (patch) | |
tree | 74892640ed482487f3bc630ad4fd20e62dec2c54 /fs/gfs2/ops_inode.c | |
parent | d0795f912318f65b800c6b619d749c3bf7c930fb (diff) |
GFS2: reserve more blocks for transactions
Some of the functions in GFS2 were not reserving space in the transaction for
the resource group header and the resource groups bitblocks that get added
when you do allocation. GFS2 now makes sure to reserve space for the
resource group header and either all the bitblocks in the resource group, or
one for each block that it may allocate, whichever is smaller using the new
gfs2_rg_blocks() inline function.
Signed-off-by: Benjamin Marzinski <bmarzins@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.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/gfs2/ops_inode.c b/fs/gfs2/ops_inode.c index 98a94cfc2bb2..fba00171d915 100644 --- a/fs/gfs2/ops_inode.c +++ b/fs/gfs2/ops_inode.c | |||
@@ -219,7 +219,7 @@ static int gfs2_link(struct dentry *old_dentry, struct inode *dir, | |||
219 | goto out_gunlock_q; | 219 | goto out_gunlock_q; |
220 | 220 | ||
221 | error = gfs2_trans_begin(sdp, sdp->sd_max_dirres + | 221 | error = gfs2_trans_begin(sdp, sdp->sd_max_dirres + |
222 | al->al_rgd->rd_length + | 222 | gfs2_rg_blocks(al) + |
223 | 2 * RES_DINODE + RES_STATFS + | 223 | 2 * RES_DINODE + RES_STATFS + |
224 | RES_QUOTA, 0); | 224 | RES_QUOTA, 0); |
225 | if (error) | 225 | if (error) |
@@ -884,7 +884,7 @@ static int gfs2_rename(struct inode *odir, struct dentry *odentry, | |||
884 | goto out_gunlock_q; | 884 | goto out_gunlock_q; |
885 | 885 | ||
886 | error = gfs2_trans_begin(sdp, sdp->sd_max_dirres + | 886 | error = gfs2_trans_begin(sdp, sdp->sd_max_dirres + |
887 | al->al_rgd->rd_length + | 887 | gfs2_rg_blocks(al) + |
888 | 4 * RES_DINODE + 4 * RES_LEAF + | 888 | 4 * RES_DINODE + 4 * RES_LEAF + |
889 | RES_STATFS + RES_QUOTA + 4, 0); | 889 | RES_STATFS + RES_QUOTA + 4, 0); |
890 | if (error) | 890 | if (error) |
@@ -1481,7 +1481,7 @@ retry: | |||
1481 | al->al_requested = data_blocks + ind_blocks; | 1481 | al->al_requested = data_blocks + ind_blocks; |
1482 | 1482 | ||
1483 | rblocks = RES_DINODE + ind_blocks + RES_STATFS + RES_QUOTA + | 1483 | rblocks = RES_DINODE + ind_blocks + RES_STATFS + RES_QUOTA + |
1484 | RES_RG_HDR + ip->i_alloc->al_rgd->rd_length; | 1484 | RES_RG_HDR + gfs2_rg_blocks(al); |
1485 | if (gfs2_is_jdata(ip)) | 1485 | if (gfs2_is_jdata(ip)) |
1486 | rblocks += data_blocks ? data_blocks : 1; | 1486 | rblocks += data_blocks ? data_blocks : 1; |
1487 | 1487 | ||