| Message ID | 20141107215114.883.46921.stgit@birch.djwong.org |
|---|---|
| State | Accepted, archived |
| Headers | show |
On Fri, Nov 07, 2014 at 01:51:14PM -0800, Darrick J. Wong wrote: > Set BLOCK_UNINIT in any group whose blocks are all unused, so long as > it isn't the last group. This helps us speed up future e2fsck runs > and mounts because we don't need to read or checksum block bitmaps for > these groups. > > v2: Take care of this in the library, since e2fsck always updates these > summary counts. > > Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com> Applied, thanks. - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/lib/ext2fs/csum.c b/lib/ext2fs/csum.c index 12f6fe9..6dcefb9 100644 --- a/lib/ext2fs/csum.c +++ b/lib/ext2fs/csum.c @@ -854,6 +854,11 @@ errcode_t ext2fs_set_gdt_csum(ext2_filsys fs) __u32 old_unused = ext2fs_bg_itable_unused(fs, i); __u32 old_flags = ext2fs_bg_flags(fs, i); __u32 old_free_inodes_count = ext2fs_bg_free_inodes_count(fs, i); + __u32 old_free_blocks_count = ext2fs_bg_free_blocks_count(fs, i); + + if (old_free_blocks_count == sb->s_blocks_per_group && + i != fs->group_desc_count - 1) + ext2fs_bg_flags_set(fs, i, EXT2_BG_BLOCK_UNINIT); if (old_free_inodes_count == sb->s_inodes_per_group) { ext2fs_bg_flags_set(fs, i, EXT2_BG_INODE_UNINIT);
Set BLOCK_UNINIT in any group whose blocks are all unused, so long as it isn't the last group. This helps us speed up future e2fsck runs and mounts because we don't need to read or checksum block bitmaps for these groups. v2: Take care of this in the library, since e2fsck always updates these summary counts. Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com> --- lib/ext2fs/csum.c | 5 +++++ 1 file changed, 5 insertions(+) -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html