1 files changed, 72 insertions, 69 deletions

diff --git a/fs/ext4/ialloc.c b/fs/ext4/ialloc.c
index c61f37fd3f05..575b5215c808 100644
--- a/fs/ext4/ialloc.c
+++ b/fs/ext4/ialloc.c
@@ -64,8 +64,8 @@ void mark_bitmap_end(int start_bit, int end_bit, char *bitmap)
 }
 
 /* Initializes an uninitialized inode bitmap */
-unsigned ext4_init_inode_bitmap(struct super_block *sb,
-                                struct buffer_head *bh, int block_group,
+unsigned ext4_init_inode_bitmap(struct super_block *sb, struct buffer_head *bh,
+                                ext4_group_t block_group,
                                 struct ext4_group_desc *gdp)
 {
         struct ext4_sb_info *sbi = EXT4_SB(sb);
@@ -75,7 +75,7 @@ unsigned ext4_init_inode_bitmap(struct super_block *sb,
         /* If checksum is bad mark all blocks and inodes use to prevent
          * allocation, essentially implementing a per-group read-only flag. */
         if (!ext4_group_desc_csum_verify(sbi, block_group, gdp)) {
-                ext4_error(sb, __FUNCTION__, "Checksum bad for group %u\n",
+                ext4_error(sb, __FUNCTION__, "Checksum bad for group %lu\n",
                            block_group);
                 gdp->bg_free_blocks_count = 0;
                 gdp->bg_free_inodes_count = 0;
@@ -98,7 +98,7 @@ unsigned ext4_init_inode_bitmap(struct super_block *sb,
  * Return buffer_head of bitmap on success or NULL.
  */
 static struct buffer_head *
-read_inode_bitmap(struct super_block * sb, unsigned long block_group)
+read_inode_bitmap(struct super_block *sb, ext4_group_t block_group)
 {
         struct ext4_group_desc *desc;
         struct buffer_head *bh = NULL;
@@ -152,7 +152,7 @@ void ext4_free_inode (handle_t *handle, struct inode * inode)
         unsigned long ino;
         struct buffer_head *bitmap_bh = NULL;
         struct buffer_head *bh2;
-        unsigned long block_group;
+        ext4_group_t block_group;
         unsigned long bit;
         struct ext4_group_desc * gdp;
         struct ext4_super_block * es;
@@ -260,12 +260,14 @@ error_return:
  * For other inodes, search forward from the parent directory\'s block
  * group to find a free inode.
  */
-static int find_group_dir(struct super_block *sb, struct inode *parent)
+static int find_group_dir(struct super_block *sb, struct inode *parent,
+                                ext4_group_t *best_group)
 {
-        int ngroups = EXT4_SB(sb)->s_groups_count;
+        ext4_group_t ngroups = EXT4_SB(sb)->s_groups_count;
         unsigned int freei, avefreei;
         struct ext4_group_desc *desc, *best_desc = NULL;
-        int group, best_group = -1;
+        ext4_group_t group;
+        int ret = -1;
 
         freei = percpu_counter_read_positive(&EXT4_SB(sb)->s_freeinodes_counter);
         avefreei = freei / ngroups;
@@ -279,11 +281,12 @@ static int find_group_dir(struct super_block *sb, struct inode *parent)
                 if (!best_desc ||
                     (le16_to_cpu(desc->bg_free_blocks_count) >
                      le16_to_cpu(best_desc->bg_free_blocks_count))) {
-                        best_group = group;
+                        *best_group = group;
                         best_desc = desc;
+                        ret = 0;
                 }
         }
-        return best_group;
+        return ret;
 }
 
 /*
@@ -314,12 +317,13 @@ static int find_group_dir(struct super_block *sb, struct inode *parent)
 #define INODE_COST 64
 #define BLOCK_COST 256
 
-static int find_group_orlov(struct super_block *sb, struct inode *parent)
+static int find_group_orlov(struct super_block *sb, struct inode *parent,
+                                ext4_group_t *group)
 {
-        int parent_group = EXT4_I(parent)->i_block_group;
+        ext4_group_t parent_group = EXT4_I(parent)->i_block_group;
         struct ext4_sb_info *sbi = EXT4_SB(sb);
         struct ext4_super_block *es = sbi->s_es;
-        int ngroups = sbi->s_groups_count;
+        ext4_group_t ngroups = sbi->s_groups_count;
         int inodes_per_group = EXT4_INODES_PER_GROUP(sb);
         unsigned int freei, avefreei;
         ext4_fsblk_t freeb, avefreeb;
@@ -327,7 +331,7 @@ static int find_group_orlov(struct super_block *sb, struct inode *parent)
         unsigned int ndirs;
         int max_debt, max_dirs, min_inodes;
         ext4_grpblk_t min_blocks;
-        int group = -1, i;
+        ext4_group_t i;
         struct ext4_group_desc *desc;
 
         freei = percpu_counter_read_positive(&sbi->s_freeinodes_counter);
@@ -340,13 +344,14 @@ static int find_group_orlov(struct super_block *sb, struct inode *parent)
         if ((parent == sb->s_root->d_inode) ||
             (EXT4_I(parent)->i_flags & EXT4_TOPDIR_FL)) {
                 int best_ndir = inodes_per_group;
-                int best_group = -1;
+                ext4_group_t grp;
+                int ret = -1;
 
-                get_random_bytes(&group, sizeof(group));
-                parent_group = (unsigned)group % ngroups;
+                get_random_bytes(&grp, sizeof(grp));
+                parent_group = (unsigned)grp % ngroups;
                 for (i = 0; i < ngroups; i++) {
-                        group = (parent_group + i) % ngroups;
-                        desc = ext4_get_group_desc (sb, group, NULL);
+                        grp = (parent_group + i) % ngroups;
+                        desc = ext4_get_group_desc(sb, grp, NULL);
                         if (!desc || !desc->bg_free_inodes_count)
                                 continue;
                         if (le16_to_cpu(desc->bg_used_dirs_count) >= best_ndir)
@@ -355,11 +360,12 @@ static int find_group_orlov(struct super_block *sb, struct inode *parent)
                                 continue;
                         if (le16_to_cpu(desc->bg_free_blocks_count) < avefreeb)
                                 continue;
-                        best_group = group;
+                        *group = grp;
+                        ret = 0;
                         best_ndir = le16_to_cpu(desc->bg_used_dirs_count);
                 }
-                if (best_group >= 0)
-                        return best_group;
+                if (ret == 0)
+                        return ret;
                 goto fallback;
         }
 
@@ -380,8 +386,8 @@ static int find_group_orlov(struct super_block *sb, struct inode *parent)
                 max_debt = 1;
 
         for (i = 0; i < ngroups; i++) {
-                group = (parent_group + i) % ngroups;
-                desc = ext4_get_group_desc (sb, group, NULL);
+                *group = (parent_group + i) % ngroups;
+                desc = ext4_get_group_desc(sb, *group, NULL);
                 if (!desc || !desc->bg_free_inodes_count)
                         continue;
                 if (le16_to_cpu(desc->bg_used_dirs_count) >= max_dirs)
@@ -390,17 +396,16 @@ static int find_group_orlov(struct super_block *sb, struct inode *parent)
                         continue;
                 if (le16_to_cpu(desc->bg_free_blocks_count) < min_blocks)
                         continue;
-                return group;
+                return 0;
         }
 
 fallback:
         for (i = 0; i < ngroups; i++) {
-                group = (parent_group + i) % ngroups;
-                desc = ext4_get_group_desc (sb, group, NULL);
-                if (!desc || !desc->bg_free_inodes_count)
-                        continue;
-                if (le16_to_cpu(desc->bg_free_inodes_count) >= avefreei)
-                        return group;
+                *group = (parent_group + i) % ngroups;
+                desc = ext4_get_group_desc(sb, *group, NULL);
+                if (desc && desc->bg_free_inodes_count &&
+                        le16_to_cpu(desc->bg_free_inodes_count) >= avefreei)
+                        return 0;
         }
 
         if (avefreei) {
@@ -415,21 +420,22 @@ fallback:
         return -1;
 }
 
-static int find_group_other(struct super_block *sb, struct inode *parent)
+static int find_group_other(struct super_block *sb, struct inode *parent,
+                                ext4_group_t *group)
 {
-        int parent_group = EXT4_I(parent)->i_block_group;
-        int ngroups = EXT4_SB(sb)->s_groups_count;
+        ext4_group_t parent_group = EXT4_I(parent)->i_block_group;
+        ext4_group_t ngroups = EXT4_SB(sb)->s_groups_count;
         struct ext4_group_desc *desc;
-        int group, i;
+        ext4_group_t i;
 
         /*
          * Try to place the inode in its parent directory
          */
-        group = parent_group;
-        desc = ext4_get_group_desc (sb, group, NULL);
+        *group = parent_group;
+        desc = ext4_get_group_desc(sb, *group, NULL);
         if (desc && le16_to_cpu(desc->bg_free_inodes_count) &&
                         le16_to_cpu(desc->bg_free_blocks_count))
-                return group;
+                return 0;
 
         /*
          * We're going to place this inode in a different blockgroup from its
@@ -440,33 +446,33 @@ static int find_group_other(struct super_block *sb, struct inode *parent)
          *
          * So add our directory's i_ino into the starting point for the hash.
          */
-        group = (group + parent->i_ino) % ngroups;
+        *group = (*group + parent->i_ino) % ngroups;
 
         /*
          * Use a quadratic hash to find a group with a free inode and some free
          * blocks.
          */
         for (i = 1; i < ngroups; i <<= 1) {
-                group += i;
-                if (group >= ngroups)
-                        group -= ngroups;
-                desc = ext4_get_group_desc (sb, group, NULL);
+                *group += i;
+                if (*group >= ngroups)
+                        *group -= ngroups;
+                desc = ext4_get_group_desc(sb, *group, NULL);
                 if (desc && le16_to_cpu(desc->bg_free_inodes_count) &&
                                 le16_to_cpu(desc->bg_free_blocks_count))
-                        return group;
+                        return 0;
         }
 
         /*
          * That failed: try linear search for a free inode, even if that group
          * has no free blocks.
          */
-        group = parent_group;
+        *group = parent_group;
         for (i = 0; i < ngroups; i++) {
-                if (++group >= ngroups)
-                        group = 0;
-                desc = ext4_get_group_desc (sb, group, NULL);
+                if (++*group >= ngroups)
+                        *group = 0;
+                desc = ext4_get_group_desc(sb, *group, NULL);
                 if (desc && le16_to_cpu(desc->bg_free_inodes_count))
-                        return group;
+                        return 0;
         }
 
         return -1;
@@ -487,16 +493,17 @@ struct inode *ext4_new_inode(handle_t *handle, struct inode * dir, int mode)
         struct super_block *sb;
         struct buffer_head *bitmap_bh = NULL;
         struct buffer_head *bh2;
-        int group;
+        ext4_group_t group = 0;
         unsigned long ino = 0;
         struct inode * inode;
         struct ext4_group_desc * gdp = NULL;
         struct ext4_super_block * es;
         struct ext4_inode_info *ei;
         struct ext4_sb_info *sbi;
-        int err = 0;
+        int ret2, err = 0;
         struct inode *ret;
-        int i, free = 0;
+        ext4_group_t i;
+        int free = 0;
 
         /* Cannot create files in a deleted directory */
         if (!dir || !dir->i_nlink)
@@ -512,14 +519,14 @@ struct inode *ext4_new_inode(handle_t *handle, struct inode * dir, int mode)
         es = sbi->s_es;
         if (S_ISDIR(mode)) {
                 if (test_opt (sb, OLDALLOC))
-                        group = find_group_dir(sb, dir);
+                        ret2 = find_group_dir(sb, dir, &group);
                 else
-                        group = find_group_orlov(sb, dir);
+                        ret2 = find_group_orlov(sb, dir, &group);
         } else
-                group = find_group_other(sb, dir);
+                ret2 = find_group_other(sb, dir, &group);
 
         err = -ENOSPC;
-        if (group == -1)
+        if (ret2 == -1)
                 goto out;
 
         for (i = 0; i < sbi->s_groups_count; i++) {
@@ -583,7 +590,7 @@ got:
             ino > EXT4_INODES_PER_GROUP(sb)) {
                 ext4_error(sb, __FUNCTION__,
                            "reserved inode or inode > inodes count - "
-                           "block_group = %d, inode=%lu", group,
+                           "block_group = %lu, inode=%lu", group,
                            ino + group * EXT4_INODES_PER_GROUP(sb));
                 err = -EIO;
                 goto fail;
@@ -702,7 +709,6 @@ got:
         if (!S_ISDIR(mode))
                 ei->i_flags &= ~EXT4_DIRSYNC_FL;
         ei->i_file_acl = 0;
-        ei->i_dir_acl = 0;
         ei->i_dtime = 0;
         ei->i_block_alloc_info = NULL;
         ei->i_block_group = group;
@@ -741,13 +747,10 @@ got:
         if (test_opt(sb, EXTENTS)) {
                 EXT4_I(inode)->i_flags |= EXT4_EXTENTS_FL;
                 ext4_ext_tree_init(handle, inode);
-                if (!EXT4_HAS_INCOMPAT_FEATURE(sb, EXT4_FEATURE_INCOMPAT_EXTENTS)) {
-                        err = ext4_journal_get_write_access(handle, EXT4_SB(sb)->s_sbh);
-                        if (err) goto fail;
-                        EXT4_SET_INCOMPAT_FEATURE(sb, EXT4_FEATURE_INCOMPAT_EXTENTS);
-                        BUFFER_TRACE(EXT4_SB(sb)->s_sbh, "call ext4_journal_dirty_metadata");
-                        err = ext4_journal_dirty_metadata(handle, EXT4_SB(sb)->s_sbh);
-                }
+                err = ext4_update_incompat_feature(handle, sb,
+                                                EXT4_FEATURE_INCOMPAT_EXTENTS);
+                if (err)
+                        goto fail;
         }
 
         ext4_debug("allocating inode %lu\n", inode->i_ino);
@@ -777,7 +780,7 @@ fail_drop:
 struct inode *ext4_orphan_get(struct super_block *sb, unsigned long ino)
 {
         unsigned long max_ino = le32_to_cpu(EXT4_SB(sb)->s_es->s_inodes_count);
-        unsigned long block_group;
+        ext4_group_t block_group;
         int bit;
         struct buffer_head *bitmap_bh = NULL;
         struct inode *inode = NULL;
@@ -833,7 +836,7 @@ unsigned long ext4_count_free_inodes (struct super_block * sb)
 {
         unsigned long desc_count;
         struct ext4_group_desc *gdp;
-        int i;
+        ext4_group_t i;
 #ifdef EXT4FS_DEBUG
         struct ext4_super_block *es;
         unsigned long bitmap_count, x;
@@ -854,7 +857,7 @@ unsigned long ext4_count_free_inodes (struct super_block * sb)
                         continue;
 
                 x = ext4_count_free(bitmap_bh, EXT4_INODES_PER_GROUP(sb) / 8);
-                printk("group %d: stored = %d, counted = %lu\n",
+                printk(KERN_DEBUG "group %lu: stored = %d, counted = %lu\n",
                         i, le16_to_cpu(gdp->bg_free_inodes_count), x);
                 bitmap_count += x;
         }
@@ -879,7 +882,7 @@ unsigned long ext4_count_free_inodes (struct super_block * sb)
 unsigned long ext4_count_dirs (struct super_block * sb)
 {
         unsigned long count = 0;
-        int i;
+        ext4_group_t i;
 
         for (i = 0; i < EXT4_SB(sb)->s_groups_count; i++) {
                 struct ext4_group_desc *gdp = ext4_get_group_desc (sb, i, NULL);