diff options
author | Eric Sandeen <sandeen@redhat.com> | 2009-08-17 23:51:29 -0400 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2009-08-17 23:51:29 -0400 |
commit | 0373130d5bc783751c1fbad948886916a21d4559 (patch) | |
tree | a5b4d21b84345354ff965714bd6a6c473bae4b0f /fs/ext4/mballoc.c | |
parent | bf43d84b185e2ff54598f8c58a5a8e63148b6e90 (diff) |
ext4: open-code ext4_mb_update_group_info
ext4_mb_update_group_info is only called in one place, and it's
extremely simple. There's no reason to have it in a separate function
in a separate file as far as I can tell, it just obfuscates what's
really going on.
Perhaps it was intended to keep the grp->bb_* manipulation local to
mballoc.c but we're already accessing other grp-> fields in balloc.c
directly so this seems ok.
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/ext4/mballoc.c')
-rw-r--r-- | fs/ext4/mballoc.c | 9 |
1 files changed, 0 insertions, 9 deletions
diff --git a/fs/ext4/mballoc.c b/fs/ext4/mballoc.c index faf5bd056a93..833d87236881 100644 --- a/fs/ext4/mballoc.c +++ b/fs/ext4/mballoc.c | |||
@@ -2554,15 +2554,6 @@ exit_meta_group_info: | |||
2554 | return -ENOMEM; | 2554 | return -ENOMEM; |
2555 | } /* ext4_mb_add_groupinfo */ | 2555 | } /* ext4_mb_add_groupinfo */ |
2556 | 2556 | ||
2557 | /* | ||
2558 | * Update an existing group. | ||
2559 | * This function is used for online resize | ||
2560 | */ | ||
2561 | void ext4_mb_update_group_info(struct ext4_group_info *grp, ext4_grpblk_t add) | ||
2562 | { | ||
2563 | grp->bb_free += add; | ||
2564 | } | ||
2565 | |||
2566 | static int ext4_mb_init_backend(struct super_block *sb) | 2557 | static int ext4_mb_init_backend(struct super_block *sb) |
2567 | { | 2558 | { |
2568 | ext4_group_t ngroups = ext4_get_groups_count(sb); | 2559 | ext4_group_t ngroups = ext4_get_groups_count(sb); |