Whamcloud - gitweb
tst_libext2fs: Avoid multiple definition of global variables
[tools/e2fsprogs.git] / lib / ext2fs / alloc_tables.c
index 5e6e556..971a6ce 100644 (file)
@@ -33,7 +33,7 @@
  * This routine searches for free blocks that can allocate a full
  * group of bitmaps or inode tables for a flexbg group.  Returns the
  * block number with a correct offset were the bitmaps and inode
- * tables can be allocated continously and in order.
+ * tables can be allocated continuously and in order.
  */
 static blk64_t flexbg_offset(ext2_filsys fs, dgrp_t group, blk64_t start_blk,
                             ext2fs_block_bitmap bmap, int rem_grp,
@@ -47,16 +47,17 @@ static blk64_t flexbg_offset(ext2_filsys fs, dgrp_t group, blk64_t start_blk,
        flexbg = group / flexbg_size;
        size = rem_grp * elem_size;
 
-       if (size > (int) (fs->super->s_blocks_per_group / 8))
-               size = (int) fs->super->s_blocks_per_group / 8;
+       if (size > (int) (fs->super->s_blocks_per_group / 4))
+               size = (int) fs->super->s_blocks_per_group / 4;
 
        /*
-        * Don't do a long search if the previous block
-        * search is still valid.
+        * Don't do a long search if the previous block search is still valid,
+        * but skip minor obstructions such as group descriptor backups.
         */
-       if (start_blk && ext2fs_test_block_bitmap_range2(bmap, start_blk,
-                                                        elem_size))
-               return start_blk;
+       if (start_blk && start_blk < ext2fs_blocks_count(fs->super) &&
+           ext2fs_get_free_blocks2(fs, start_blk, start_blk + size, elem_size,
+                                   bmap, &first_free) == 0)
+               return first_free;
 
        start_blk = ext2fs_group_first_block2(fs, flexbg_size * flexbg);
        last_grp = group | (flexbg_size - 1);
@@ -83,11 +84,10 @@ static blk64_t flexbg_offset(ext2_filsys fs, dgrp_t group, blk64_t start_blk,
 errcode_t ext2fs_allocate_group_table(ext2_filsys fs, dgrp_t group,
                                      ext2fs_block_bitmap bmap)
 {
-       unsigned int    j;
        errcode_t       retval;
-       blk64_t         group_blk, start_blk, last_blk, new_blk, blk;
+       blk64_t         group_blk, start_blk, last_blk, new_blk;
        dgrp_t          last_grp = 0;
-       int             rem_grps = 0, flexbg_size = 0;
+       int             rem_grps = 0, flexbg_size = 0, table_offset = 0;
 
        group_blk = ext2fs_group_first_block2(fs, group);
        last_blk = ext2fs_group_last_block2(fs, group);
@@ -95,8 +95,7 @@ errcode_t ext2fs_allocate_group_table(ext2_filsys fs, dgrp_t group,
        if (!bmap)
                bmap = fs->block_map;
 
-       if (EXT2_HAS_INCOMPAT_FEATURE(fs->super,
-                                     EXT4_FEATURE_INCOMPAT_FLEX_BG) &&
+       if (ext2fs_has_feature_flex_bg(fs->super) &&
            fs->super->s_log_groups_per_flex) {
                flexbg_size = 1 << fs->super->s_log_groups_per_flex;
                last_grp = group | (flexbg_size - 1);
@@ -108,7 +107,7 @@ errcode_t ext2fs_allocate_group_table(ext2_filsys fs, dgrp_t group,
        /*
         * Allocate the block and inode bitmaps, if necessary
         */
-       if (fs->stride) {
+       if (fs->stride && !flexbg_size) {
                retval = ext2fs_get_free_blocks2(fs, group_blk, last_blk,
                                                 1, bmap, &start_blk);
                if (retval)
@@ -124,8 +123,22 @@ errcode_t ext2fs_allocate_group_table(ext2_filsys fs, dgrp_t group,
        if (flexbg_size) {
                blk64_t prev_block = 0;
 
+               table_offset = flexbg_size;
                if (group % flexbg_size)
                        prev_block = ext2fs_block_bitmap_loc(fs, group - 1) + 1;
+               else if (last_grp == fs->group_desc_count-1) {
+                       /*
+                        * If we are allocating for the last flex_bg
+                        * keep the metadata tables contiguous
+                        */
+                       table_offset = last_grp & (flexbg_size - 1);
+                       if (table_offset == 0)
+                               table_offset = flexbg_size;
+                       else
+                               table_offset++;
+               }
+               /* FIXME: Take backup group descriptor blocks into account
+                * if the flexbg allocations will grow to overlap them... */
                start_blk = flexbg_offset(fs, group, prev_block, bmap,
                                          rem_grps, 1);
                last_blk = ext2fs_group_last_block2(fs, last_grp);
@@ -156,7 +169,9 @@ errcode_t ext2fs_allocate_group_table(ext2_filsys fs, dgrp_t group,
                        prev_block = ext2fs_inode_bitmap_loc(fs, group - 1) + 1;
                else
                        prev_block = ext2fs_block_bitmap_loc(fs, group) +
-                               flexbg_size;
+                               table_offset;
+               /* FIXME: Take backup group descriptor blocks into account
+                * if the flexbg allocations will grow to overlap them... */
                start_blk = flexbg_offset(fs, group, prev_block, bmap,
                                          rem_grps, 1);
                last_blk = ext2fs_group_last_block2(fs, last_grp);
@@ -192,8 +207,10 @@ errcode_t ext2fs_allocate_group_table(ext2_filsys fs, dgrp_t group,
                                fs->inode_blocks_per_group;
                else
                        prev_block = ext2fs_inode_bitmap_loc(fs, group) +
-                               flexbg_size;
+                               table_offset;
 
+               /* FIXME: Take backup group descriptor blocks into account
+                * if the flexbg allocations will grow to overlap them... */
                group_blk = flexbg_offset(fs, group, prev_block, bmap,
                                          rem_grps, fs->inode_blocks_per_group);
                last_blk = ext2fs_group_last_block2(fs, last_grp);
@@ -205,17 +222,30 @@ errcode_t ext2fs_allocate_group_table(ext2_filsys fs, dgrp_t group,
                                                bmap, &new_blk);
                if (retval)
                        return retval;
-               for (j=0, blk = new_blk;
-                    j < fs->inode_blocks_per_group;
-                    j++, blk++) {
-                       ext2fs_mark_block_bitmap2(bmap, blk);
-                       if (flexbg_size) {
-                               dgrp_t gr = ext2fs_group_of_blk2(fs, blk);
-                               ext2fs_bg_free_blocks_count_set(fs, gr, ext2fs_bg_free_blocks_count(fs, gr) - 1);
-                               ext2fs_free_blocks_count_add(fs->super, -1);
+
+               ext2fs_mark_block_bitmap_range2(bmap,
+                       new_blk, fs->inode_blocks_per_group);
+               if (flexbg_size) {
+                       blk64_t num, blk;
+                       num = fs->inode_blocks_per_group;
+                       blk = new_blk;
+                       while (num) {
+                               int gr = ext2fs_group_of_blk2(fs, blk);
+                               last_blk = ext2fs_group_last_block2(fs, gr);
+                               blk64_t n = num;
+
+                               if (blk + num > last_blk)
+                                       n = last_blk - blk + 1;
+
+                               ext2fs_bg_free_blocks_count_set(fs, gr,
+                                       ext2fs_bg_free_blocks_count(fs, gr) -
+                                       n/EXT2FS_CLUSTER_RATIO(fs));
                                ext2fs_bg_flags_clear(fs, gr,
-                                                    EXT2_BG_BLOCK_UNINIT);
+                                       EXT2_BG_BLOCK_UNINIT);
                                ext2fs_group_desc_csum_set(fs, gr);
+                               ext2fs_free_blocks_count_add(fs->super, -n);
+                               blk += n;
+                               num -= n;
                        }
                }
                ext2fs_inode_table_loc_set(fs, group, new_blk);