From 831f309c624d77e41c83ee7c55d900e347cd44fe Mon Sep 17 00:00:00 2001 From: Theodore Ts'o Date: Sat, 29 Dec 2012 03:43:55 -0500 Subject: [PATCH] resize2fs: create optimized flex_bg block groups Now that we are reserving all of the bg-specific metadata before we try to allocate the metadata for the new block groups, we don't have to temporarily disable the flex_bg feature flag while we allocate the new metadata blocks --- this allows the newly created block groups to have a much more optimized layout, instead of fragmenting the inode table and block/inode bitmaps in sepraate block groups. Signed-off-by: "Theodore Ts'o" --- resize/resize2fs.c | 22 ---------------------- 1 file changed, 22 deletions(-) diff --git a/resize/resize2fs.c b/resize/resize2fs.c index 159846db..092cfbda 100644 --- a/resize/resize2fs.c +++ b/resize/resize2fs.c @@ -809,7 +809,6 @@ static errcode_t blocks_to_move(ext2_resize_t rfs) errcode_t retval; ext2_filsys fs, old_fs; ext2fs_block_bitmap meta_bmap; - __u32 save_incompat_flag; int flex_bg; fs = rfs->new_fs; @@ -975,29 +974,8 @@ static errcode_t blocks_to_move(ext2_resize_t rfs) ext2fs_block_bitmap_loc(fs, i)) continue; - /* - * XXX We have a problem with FLEX_BG and off-line - * resizing where we are growing the size of the - * filesystem. ext2fs_allocate_group_table() will try - * to reserve the inode table in the desired flex_bg - * location. However, passing rfs->reserve_blocks - * doesn't work since it only has reserved the blocks - * that will be used in the new block group -- and - * with flex_bg, we can and will allocate the tables - * outside of the block group. And we can't pass in - * the fs->block_map because it doesn't handle - * overlapping inode table movements right. So for - * now, we temporarily disable flex_bg to force - * ext2fs_allocate_group_tables() to allocate the bg - * metadata in side the block group, and the restore - * it afterwards. Ugly, until we can fix this up - * right later. - */ - save_incompat_flag = fs->super->s_feature_incompat; - fs->super->s_feature_incompat &= ~EXT4_FEATURE_INCOMPAT_FLEX_BG; retval = ext2fs_allocate_group_table(fs, i, rfs->reserve_blocks); - fs->super->s_feature_incompat = save_incompat_flag; if (retval) goto errout; -- 2.11.0