On Fri 02-03-12 13:11:56, Lukas Czerner wrote: > Currently when there is not enough free block in the block group to > discard (grp->bb_free < minlen) the 'trimmed' is bumped up anyway with > the number of discarded blocks from the previous iteration. Fix this > by bumping up 'trimmed' only if the ext4_trim_all_free() was actually > run. Looks good. You can add: Reviewed-by: Jan Kara <jack@xxxxxxx> Honza > > Signed-off-by: Lukas Czerner <lczerner@xxxxxxxxxx> > --- > v2: nothing changed > > fs/ext4/mballoc.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/fs/ext4/mballoc.c b/fs/ext4/mballoc.c > index 16a3aeb..f20688e 100644 > --- a/fs/ext4/mballoc.c > +++ b/fs/ext4/mballoc.c > @@ -5083,8 +5083,8 @@ int ext4_trim_fs(struct super_block *sb, struct fstrim_range *range) > ret = cnt; > break; > } > + trimmed += cnt; > } > - trimmed += cnt; > > /* > * For every group except the first one, we are sure > -- > 1.7.4.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 -- Jan Kara <jack@xxxxxxx> SUSE Labs, CR -- 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
