On Sun, 11 Mar 2012, Theodore Ts'o wrote:
> From: Lukas Czerner <lczerner@xxxxxxxxxx>
>
> Previously when running e2fsck with '-E discard' argument the end of
> the last group has not been discarded. This patch fixes it so we
> always discard the end of the last group if needed.
>
> Signed-off-by: Lukas Czerner <lczerner@xxxxxxxxxx>
> Signed-off-by: Theodore Ts'o <tytso@xxxxxxx>
Looks good. Thanks for splitting the patch, it makes sense.
-Lukas
> ---
> e2fsck/pass5.c | 9 +++++++++
> 1 files changed, 9 insertions(+), 0 deletions(-)
>
> diff --git a/e2fsck/pass5.c b/e2fsck/pass5.c
> index ca438cb..e25f080 100644
> --- a/e2fsck/pass5.c
> +++ b/e2fsck/pass5.c
> @@ -387,6 +387,15 @@ redo_counts:
> if ((blocks == fs->super->s_clusters_per_group) ||
> (EXT2FS_B2C(fs, i) ==
> EXT2FS_B2C(fs, ext2fs_blocks_count(fs->super)-1))) {
> + /*
> + * If the last block of this group is free, then we can
> + * discard it as well.
> + */
> + if (!bitmap && i >= first_free)
> + e2fsck_discard_blocks(ctx, first_free,
> + (i - first_free) + 1);
> + first_free = ext2fs_blocks_count(fs->super);
> +
> free_array[group] = group_free;
> group ++;
> blocks = 0;
>
--
--
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
