ext4: let setup_new_group_blocks() set multiple bits at a time
Rename mb_set_bits() to ext4_set_bits() and make it a global function so that setup_new_group_blocks() can use it. Signed-off-by: Yongqiang Yang <xiaoqiangnk@gmail.com> Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
This commit is contained in:
parent
2b79b09d13
commit
c3e94d1df9
3 changed files with 17 additions and 18 deletions
|
@ -217,11 +217,6 @@ static int setup_new_group_blocks(struct super_block *sb,
|
|||
goto exit_journal;
|
||||
}
|
||||
|
||||
if (ext4_bg_has_super(sb, input->group)) {
|
||||
ext4_debug("mark backup superblock %#04llx (+0)\n", start);
|
||||
ext4_set_bit(0, bh->b_data);
|
||||
}
|
||||
|
||||
/* Copy all of the GDT blocks into the backup in this group */
|
||||
for (i = 0, bit = 1, block = start + 1;
|
||||
i < gdblocks; i++, block++, bit++) {
|
||||
|
@ -250,7 +245,6 @@ static int setup_new_group_blocks(struct super_block *sb,
|
|||
brelse(gdb);
|
||||
goto exit_bh;
|
||||
}
|
||||
ext4_set_bit(bit, bh->b_data);
|
||||
brelse(gdb);
|
||||
}
|
||||
|
||||
|
@ -261,8 +255,11 @@ static int setup_new_group_blocks(struct super_block *sb,
|
|||
GFP_NOFS);
|
||||
if (err)
|
||||
goto exit_bh;
|
||||
for (i = 0, bit = gdblocks + 1; i < reserved_gdb; i++, bit++)
|
||||
ext4_set_bit(bit, bh->b_data);
|
||||
|
||||
if (ext4_bg_has_super(sb, input->group)) {
|
||||
ext4_debug("mark backup group tables %#04llx (+0)\n", start);
|
||||
ext4_set_bits(bh->b_data, 0, gdblocks + reserved_gdb + 1);
|
||||
}
|
||||
|
||||
ext4_debug("mark block bitmap %#04llx (+%llu)\n", input->block_bitmap,
|
||||
input->block_bitmap - start);
|
||||
|
@ -278,9 +275,8 @@ static int setup_new_group_blocks(struct super_block *sb,
|
|||
err = sb_issue_zeroout(sb, block, sbi->s_itb_per_group, GFP_NOFS);
|
||||
if (err)
|
||||
goto exit_bh;
|
||||
for (i = 0, bit = input->inode_table - start;
|
||||
i < sbi->s_itb_per_group; i++, bit++)
|
||||
ext4_set_bit(bit, bh->b_data);
|
||||
ext4_set_bits(bh->b_data, input->inode_table - start,
|
||||
sbi->s_itb_per_group);
|
||||
|
||||
if ((err = extend_or_restart_transaction(handle, 2, bh)))
|
||||
goto exit_bh;
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue