remove unused bh in calls to ext234_get_group_desc

ext[234]_get_group_desc never tests the bh argument, and only sets it if it
is passed in; it is perfectly happy with a NULL bh argument.  But, many
callers send one in and never use it.  May as well call with NULL like
other callers who don't use the bh.

Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Cc: <linux-ext4@vger.kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

2 files changed, 9 insertions, 17 deletions

diff --git a/fs/ext2/ialloc.c b/fs/ext2/ialloc.c
index 4a9746b9afb..2625a00c466 100644
--- a/fs/ext2/ialloc.c
+++ b/fs/ext2/ialloc.c
@@ -177,7 +177,6 @@ static void ext2_preread_inode(struct inode *inode)
         unsigned long block_group;
         unsigned long offset;
         unsigned long block;
-        struct buffer_head *bh;
         struct ext2_group_desc * gdp;
         struct backing_dev_info *bdi;
 
@@ -188,7 +187,7 @@ static void ext2_preread_inode(struct inode *inode)
                 return;
 
         block_group = (inode->i_ino - 1) / EXT2_INODES_PER_GROUP(inode->i_sb);
-        gdp = ext2_get_group_desc(inode->i_sb, block_group, &bh);
+        gdp = ext2_get_group_desc(inode->i_sb, block_group, NULL);
         if (gdp == NULL)
                 return;
 
@@ -217,11 +216,10 @@ static int find_group_dir(struct super_block *sb, struct inode *parent)
         int ngroups = EXT2_SB(sb)->s_groups_count;
         int avefreei = ext2_count_free_inodes(sb) / ngroups;
         struct ext2_group_desc *desc, *best_desc = NULL;
-        struct buffer_head *bh, *best_bh = NULL;
         int group, best_group = -1;
 
         for (group = 0; group < ngroups; group++) {
-                desc = ext2_get_group_desc (sb, group, &bh);
+                desc = ext2_get_group_desc (sb, group, NULL);
                 if (!desc || !desc->bg_free_inodes_count)
                         continue;
                 if (le16_to_cpu(desc->bg_free_inodes_count) < avefreei)
@@ -231,7 +229,6 @@ static int find_group_dir(struct super_block *sb, struct inode *parent)
                      le16_to_cpu(best_desc->bg_free_blocks_count))) {
                         best_group = group;
                         best_desc = desc;
-                        best_bh = bh;
                 }
         }
         if (!best_desc)
@@ -284,7 +281,6 @@ static int find_group_orlov(struct super_block *sb, struct inode *parent)
         int max_debt, max_dirs, min_blocks, min_inodes;
         int group = -1, i;
         struct ext2_group_desc *desc;
-        struct buffer_head *bh;
 
         freei = percpu_counter_read_positive(&sbi->s_freeinodes_counter);
         avefreei = freei / ngroups;
@@ -295,7 +291,6 @@ static int find_group_orlov(struct super_block *sb, struct inode *parent)
         if ((parent == sb->s_root->d_inode) ||
             (EXT2_I(parent)->i_flags & EXT2_TOPDIR_FL)) {
                 struct ext2_group_desc *best_desc = NULL;
-                struct buffer_head *best_bh = NULL;
                 int best_ndir = inodes_per_group;
                 int best_group = -1;
 
@@ -303,7 +298,7 @@ static int find_group_orlov(struct super_block *sb, struct inode *parent)
                 parent_group = (unsigned)group % ngroups;
                 for (i = 0; i < ngroups; i++) {
                         group = (parent_group + i) % ngroups;
-                        desc = ext2_get_group_desc (sb, group, &bh);
+                        desc = ext2_get_group_desc (sb, group, NULL);
                         if (!desc || !desc->bg_free_inodes_count)
                                 continue;
                         if (le16_to_cpu(desc->bg_used_dirs_count) >= best_ndir)
@@ -315,11 +310,9 @@ static int find_group_orlov(struct super_block *sb, struct inode *parent)
                         best_group = group;
                         best_ndir = le16_to_cpu(desc->bg_used_dirs_count);
                         best_desc = desc;
-                        best_bh = bh;
                 }
                 if (best_group >= 0) {
                         desc = best_desc;
-                        bh = best_bh;
                         group = best_group;
                         goto found;
                 }
@@ -345,7 +338,7 @@ static int find_group_orlov(struct super_block *sb, struct inode *parent)
 
         for (i = 0; i < ngroups; i++) {
                 group = (parent_group + i) % ngroups;
-                desc = ext2_get_group_desc (sb, group, &bh);
+                desc = ext2_get_group_desc (sb, group, NULL);
                 if (!desc || !desc->bg_free_inodes_count)
                         continue;
                 if (sbi->s_debts[group] >= max_debt)
@@ -362,7 +355,7 @@ static int find_group_orlov(struct super_block *sb, struct inode *parent)
 fallback:
         for (i = 0; i < ngroups; i++) {
                 group = (parent_group + i) % ngroups;
-                desc = ext2_get_group_desc (sb, group, &bh);
+                desc = ext2_get_group_desc (sb, group, NULL);
                 if (!desc || !desc->bg_free_inodes_count)
                         continue;
                 if (le16_to_cpu(desc->bg_free_inodes_count) >= avefreei)
@@ -389,14 +382,13 @@ static int find_group_other(struct super_block *sb, struct inode *parent)
         int parent_group = EXT2_I(parent)->i_block_group;
         int ngroups = EXT2_SB(sb)->s_groups_count;
         struct ext2_group_desc *desc;
-        struct buffer_head *bh;
         int group, i;
 
         /*
          * Try to place the inode in its parent directory
          */
         group = parent_group;
-        desc = ext2_get_group_desc (sb, group, &bh);
+        desc = ext2_get_group_desc (sb, group, NULL);
         if (desc && le16_to_cpu(desc->bg_free_inodes_count) &&
                         le16_to_cpu(desc->bg_free_blocks_count))
                 goto found;
@@ -420,7 +412,7 @@ static int find_group_other(struct super_block *sb, struct inode *parent)
                 group += i;
                 if (group >= ngroups)
                         group -= ngroups;
-                desc = ext2_get_group_desc (sb, group, &bh);
+                desc = ext2_get_group_desc (sb, group, NULL);
                 if (desc && le16_to_cpu(desc->bg_free_inodes_count) &&
                                 le16_to_cpu(desc->bg_free_blocks_count))
                         goto found;
@@ -434,7 +426,7 @@ static int find_group_other(struct super_block *sb, struct inode *parent)
         for (i = 0; i < ngroups; i++) {
                 if (++group >= ngroups)
                         group = 0;
-                desc = ext2_get_group_desc (sb, group, &bh);
+                desc = ext2_get_group_desc (sb, group, NULL);
                 if (desc && le16_to_cpu(desc->bg_free_inodes_count))
                         goto found;
         }
diff --git a/fs/ext2/inode.c b/fs/ext2/inode.c
index 1b102a1cceb..84818176fd9 100644
--- a/fs/ext2/inode.c
+++ b/fs/ext2/inode.c
@@ -1028,7 +1028,7 @@ static struct ext2_inode *ext2_get_inode(struct super_block *sb, ino_t ino,
                 goto Einval;
 
         block_group = (ino - 1) / EXT2_INODES_PER_GROUP(sb);
-        gdp = ext2_get_group_desc(sb, block_group, &bh);
+        gdp = ext2_get_group_desc(sb, block_group, NULL);
         if (!gdp)
                 goto Egdp;
         /*