On Thu, Mar 24, 2011 at 2:48 AM, Yongqiang Yang <xiaoqiangnk@xxxxxxxxx> wrote: > Orginal computing ignores indirects themselves in ordered mode. On the worst case, data,ind,dind,tind all allocated from different bg and different gdb credits should be 2*4+4+2 for journaled and 2*4+3+2 for ordered (with 4K block: bpp = 1, indirects = 3) Acked-by: Amir Goldstein <amir73il@xxxxxxxxxxxx> > > Signed-off-by:Yongqiang Yang <xiaoqiangnk@xxxxxxxxx> > --- > fs/ext3/inode.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/fs/ext3/inode.c b/fs/ext3/inode.c > index ae94f6d..7f5db46 100644 > --- a/fs/ext3/inode.c > +++ b/fs/ext3/inode.c > @@ -3294,7 +3294,7 @@ static int ext3_writepage_trans_blocks(struct inode *inode) > if (ext3_should_journal_data(inode)) > ret = 3 * (bpp + indirects) + 2; > else > - ret = 2 * (bpp + indirects) + 2; > + ret = 2 * (bpp + indirects) + indirects + 2; > > #ifdef CONFIG_QUOTA > /* We know that structure was already allocated during dquot_initialize so > -- > 1.7.4 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-ext4" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html