2012-04-13 01:53:27

by Akira Fujita

[permalink] [raw]
Subject: [PATCH] ext4: remove needs_recovery in ext4_mb_init()

needs_recovery in ext4_mb_init() is not used,
remove it.

Signed-off-by: Akira Fujita <[email protected]>
---
fs/ext4/ext4.h | 2 +-
fs/ext4/mballoc.c | 2 +-
fs/ext4/super.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h
index ab2594a..00afad8 100644
--- a/fs/ext4/ext4.h
+++ b/fs/ext4/ext4.h
@@ -1867,7 +1867,7 @@ extern void ext4_end_bitmap_read(struct buffer_head *bh, int uptodate);
/* mballoc.c */
extern long ext4_mb_stats;
extern long ext4_mb_max_to_scan;
-extern int ext4_mb_init(struct super_block *, int);
+extern int ext4_mb_init(struct super_block *);
extern int ext4_mb_release(struct super_block *);
extern ext4_fsblk_t ext4_mb_new_blocks(handle_t *,
struct ext4_allocation_request *, int *);
diff --git a/fs/ext4/mballoc.c b/fs/ext4/mballoc.c
index 99ab428..90792dc 100644
--- a/fs/ext4/mballoc.c
+++ b/fs/ext4/mballoc.c
@@ -2375,7 +2375,7 @@ static int ext4_groupinfo_create_slab(size_t size)
return 0;
}

-int ext4_mb_init(struct super_block *sb, int needs_recovery)
+int ext4_mb_init(struct super_block *sb)
{
struct ext4_sb_info *sbi = EXT4_SB(sb);
unsigned i, j;
diff --git a/fs/ext4/super.c b/fs/ext4/super.c
index ceebaf8..0c6ded5 100644
--- a/fs/ext4/super.c
+++ b/fs/ext4/super.c
@@ -3643,7 +3643,7 @@ no_journal:
}

ext4_ext_init(sb);
- err = ext4_mb_init(sb, needs_recovery);
+ err = ext4_mb_init(sb);
if (err) {
ext4_msg(sb, KERN_ERR, "failed to initialize mballoc (%d)",
err);


2012-05-28 17:59:21

by Theodore Ts'o

[permalink] [raw]
Subject: Re: [PATCH] ext4: remove needs_recovery in ext4_mb_init()

On Fri, Apr 13, 2012 at 10:52:44AM +0900, Akira Fujita wrote:
> needs_recovery in ext4_mb_init() is not used,
> remove it.
>
> Signed-off-by: Akira Fujita <[email protected]>

Applied, thanks.

- Ted