From: hao.bigrat@gmail.com Subject: [PATCH bigalloc] modify offset to cluster in ext4_valid_block_bitmap Date: Mon, 18 Jul 2011 17:55:52 +0800 Message-ID: <1310982952-4937-1-git-send-email-hao.bigrat@gmail.com> Cc: Robin Dong , Ted Ts'o To: linux-ext4@vger.kernel.org Return-path: Received: from mail-iw0-f174.google.com ([209.85.214.174]:59493 "EHLO mail-iw0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753419Ab1GRJ4K (ORCPT ); Mon, 18 Jul 2011 05:56:10 -0400 Received: by iwn6 with SMTP id 6so2939619iwn.19 for ; Mon, 18 Jul 2011 02:56:10 -0700 (PDT) Sender: linux-ext4-owner@vger.kernel.org List-ID: From: Robin Dong When I run a test application on ext4 of bigalloc, and also a daemon which run "echo 3 > /proc/sys/vm/drop_caches" every 2 seconds, the kernel report: 15319.557145] EXT4-fs error (device sda4): ext4_valid_block_bitmap:324: comm dir_tree: Invalid block bitmap - block_group = 6, block = 3145730 The reason is ext4_valied_block_bitmap has not be modified for cluster. Signed-off-by: Robin Dong Cc: Ted Ts'o --- fs/ext4/balloc.c | 13 ++++++++----- 1 files changed, 8 insertions(+), 5 deletions(-) diff --git a/fs/ext4/balloc.c b/fs/ext4/balloc.c index 23a6375..8b982fc 100644 --- a/fs/ext4/balloc.c +++ b/fs/ext4/balloc.c @@ -282,6 +282,7 @@ static int ext4_valid_block_bitmap(struct super_block *sb, ext4_grpblk_t next_zero_bit; ext4_fsblk_t bitmap_blk; ext4_fsblk_t group_first_block; + struct ext4_sb_info *sbi = EXT4_SB(sb); if (EXT4_HAS_INCOMPAT_FEATURE(sb, EXT4_FEATURE_INCOMPAT_FLEX_BG)) { /* with FLEX_BG, the inode/block bitmaps and itable @@ -297,14 +298,14 @@ static int ext4_valid_block_bitmap(struct super_block *sb, /* check whether block bitmap block number is set */ bitmap_blk = ext4_block_bitmap(sb, desc); offset = bitmap_blk - group_first_block; - if (!ext4_test_bit(offset, bh->b_data)) + if (!ext4_test_bit(EXT4_B2C(sbi, offset), bh->b_data)) /* bad block bitmap */ goto err_out; /* check whether the inode bitmap block number is set */ bitmap_blk = ext4_inode_bitmap(sb, desc); offset = bitmap_blk - group_first_block; - if (!ext4_test_bit(offset, bh->b_data)) + if (!ext4_test_bit(EXT4_B2C(sbi, offset), bh->b_data)) /* bad block bitmap */ goto err_out; @@ -312,9 +313,11 @@ static int ext4_valid_block_bitmap(struct super_block *sb, bitmap_blk = ext4_inode_table(sb, desc); offset = bitmap_blk - group_first_block; next_zero_bit = ext4_find_next_zero_bit(bh->b_data, - offset + EXT4_SB(sb)->s_itb_per_group, - offset); - if (next_zero_bit >= offset + EXT4_SB(sb)->s_itb_per_group) + EXT4_B2C(sbi, offset + EXT4_SB(sb)->s_itb_per_group), + EXT4_B2C(sbi, offset)); + + if (next_zero_bit >= + EXT4_B2C(sbi, offset + EXT4_SB(sb)->s_itb_per_group)) /* good bitmap for inode tables */ return 1; -- 1.7.3.2