ext4: don't call update_backups() multiple times for the same bg

[GitHub/mt8127/android_kernel_alcatel_ttab.git] / fs / ext4 / resize.c

diff --git a/fs/ext4/resize.c b/fs/ext4/resize.c
index 3c9367b9bebdadcab66b7b17eb46300a357c2949..7a75e1086961eedb674b8c2bf1bb84e657f4dfa2 100644 (file)
--- a/fs/ext4/resize.c
+++ b/fs/ext4/resize.c
@@ -1049,8 +1049,6 @@ static void update_backups(struct super_block *sb, int blk_off, char *data,
                goto exit_err;
        }
 
-       ext4_superblock_csum_set(sb, (struct ext4_super_block *)data);
-
        if (meta_bg == 0) {
                group = ext4_list_backups(sb, &three, &five, &seven);
                last = sbi->s_groups_count;
@@ -1183,17 +1181,12 @@ static struct buffer_head *ext4_get_bitmap(struct super_block *sb, __u64 block)
        struct buffer_head *bh = sb_getblk(sb, block);
        if (!bh)
                return NULL;
-
-       if (bitmap_uptodate(bh))
-               return bh;
-
-       lock_buffer(bh);
-       if (bh_submit_read(bh) < 0) {
-               unlock_buffer(bh);
-               brelse(bh);
-               return NULL;
+       if (!bh_uptodate_or_lock(bh)) {
+               if (bh_submit_read(bh) < 0) {
+                       brelse(bh);
+                       return NULL;
+               }
        }
-       unlock_buffer(bh);
 
        return bh;
 }
@@ -1467,6 +1460,7 @@ exit_journal:
                                   EXT4_DESC_PER_BLOCK(sb));
                int meta_bg = EXT4_HAS_INCOMPAT_FEATURE(sb,
                                EXT4_FEATURE_INCOMPAT_META_BG);
+               sector_t old_gdb = 0;
 
                update_backups(sb, sbi->s_sbh->b_blocknr, (char *)es,
                               sizeof(struct ext4_super_block), 0);
@@ -1474,8 +1468,11 @@ exit_journal:
                        struct buffer_head *gdb_bh;
 
                        gdb_bh = sbi->s_group_desc[gdb_num];
+                       if (old_gdb == gdb_bh->b_blocknr)
+                               continue;
                        update_backups(sb, gdb_bh->b_blocknr, gdb_bh->b_data,
                                       gdb_bh->b_size, meta_bg);
+                       old_gdb = gdb_bh->b_blocknr;
                }
        }
 exit:
@@ -1772,23 +1769,18 @@ static int ext4_convert_meta_bg(struct super_block *sb, struct inode *inode)
        handle_t *handle;
        struct ext4_sb_info *sbi = EXT4_SB(sb);
        struct ext4_super_block *es = sbi->s_es;
-       struct ext4_inode_info *ei = 0;
+       struct ext4_inode_info *ei = EXT4_I(inode);
        ext4_fsblk_t nr;
        int i, ret, err = 0;
        int credits = 1;
 
        ext4_msg(sb, KERN_INFO, "Converting file system to meta_bg");
-       if (EXT4_HAS_COMPAT_FEATURE(sb, EXT4_FEATURE_COMPAT_RESIZE_INODE)) {
+       if (inode) {
                if (es->s_reserved_gdt_blocks) {
                        ext4_error(sb, "Unexpected non-zero "
                                   "s_reserved_gdt_blocks");
                        return -EPERM;
                }
-               if (!inode) {
-                       ext4_error(sb, "Unexpected NULL resize_inode");
-                       return -EPERM;
-               }
-               ei = EXT4_I(inode);
 
                /* Do a quick sanity check of the resize inode */
                if (inode->i_blocks != 1 << (inode->i_blkbits - 9))
@@ -1873,12 +1865,19 @@ int ext4_resize_fs(struct super_block *sb, ext4_fsblk_t n_blocks_count)
        int err = 0, flexbg_size = 1 << sbi->s_log_groups_per_flex;
        int meta_bg;
 
+       /* See if the device is actually as big as what was requested */
+       bh = sb_bread(sb, n_blocks_count - 1);
+       if (!bh) {
+               ext4_warning(sb, "can't read last block, resize aborted");
+               return -ENOSPC;
+       }
+       brelse(bh);
+
 retry:
        o_blocks_count = ext4_blocks_count(es);
 
-       if (test_opt(sb, DEBUG))
-               ext4_msg(sb, KERN_DEBUG, "resizing filesystem from %llu "
-                      "to %llu blocks", o_blocks_count, n_blocks_count);
+       ext4_msg(sb, KERN_INFO, "resizing filesystem from %llu "
+                "to %llu blocks", o_blocks_count, n_blocks_count);
 
        if (n_blocks_count < o_blocks_count) {
                /* On-line shrinking not supported */
@@ -1922,7 +1921,7 @@ retry:
                }
        }
 
-       if ((!resize_inode && !meta_bg) || n_group == o_group) {
+       if ((!resize_inode && !meta_bg) || n_blocks_count == o_blocks_count) {
                err = ext4_convert_meta_bg(sb, resize_inode);
                if (err)
                        goto out;
@@ -1937,14 +1936,6 @@ retry:
                }
        }
 
-       /* See if the device is actually as big as what was requested */
-       bh = sb_bread(sb, n_blocks_count - 1);
-       if (!bh) {
-               ext4_warning(sb, "can't read last block, resize aborted");
-               return -ENOSPC;
-       }
-       brelse(bh);
-
        /* extend the last group */
        if (n_group == o_group)
                add = n_blocks_count - o_blocks_count;
@@ -2005,8 +1996,6 @@ out:
                free_flex_gd(flex_gd);
        if (resize_inode != NULL)
                iput(resize_inode);
-       if (test_opt(sb, DEBUG))
-               ext4_msg(sb, KERN_DEBUG, "resized filesystem to %llu",
-                        n_blocks_count);
+       ext4_msg(sb, KERN_INFO, "resized filesystem to %llu", n_blocks_count);
        return err;
 }