Re: [PATCH 4/4] ext4: mark block bitmap corrupted when found

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Apr 17, 2018 at 5:08 PM, Wang Shilong <wangshilong1991@xxxxxxxxx> wrote:
> From: Wang Shilong <wshilong@xxxxxxx>
>
> There are still some cases that we missed to set
> block bitmaps corrupted bit properly:
>
> 1) block bitmap number is wrong.
> 2) failed to read block bitmap due to disk errors.
> 3) double free block bitmaps..
> 4) some mismatch check with bitmaps vs buddy information.
>
> Suggested-by: Theodore Y. Ts'o <tytso@xxxxxxx>
> Signed-off-by: Liu Bo <bo.liu@xxxxxxxxxxxxxxxxx>
> Signed-off-by: Wang Shilong <wshilong@xxxxxxx>
> ---
>  fs/ext4/balloc.c  | 4 ++++
>  fs/ext4/mballoc.c | 6 ++++++
>  2 files changed, 10 insertions(+)
>
> diff --git a/fs/ext4/balloc.c b/fs/ext4/balloc.c
> index 0ac442aff7b8..8ef65251d019 100644
> --- a/fs/ext4/balloc.c
> +++ b/fs/ext4/balloc.c
> @@ -420,6 +420,8 @@ ext4_read_block_bitmap_nowait(struct super_block *sb, ext4_group_t block_group)
>             (bitmap_blk >= ext4_blocks_count(sbi->s_es))) {
>                 ext4_error(sb, "Invalid block bitmap block %llu in "
>                            "block_group %u", bitmap_blk, block_group);
> +               ext4_mark_group_bitmap_corrupted(sb, block_group,
> +                                       EXT4_GROUP_INFO_BBITMAP_CORRUPT);
>                 return ERR_PTR(-EFSCORRUPTED);
>         }
>         bh = sb_getblk(sb, bitmap_blk);
> @@ -498,6 +500,8 @@ int ext4_wait_block_bitmap(struct super_block *sb, ext4_group_t block_group,
>                 ext4_error(sb, "Cannot read block bitmap - "
>                            "block_group = %u, block_bitmap = %llu",
>                            block_group, (unsigned long long) bh->b_blocknr);
> +               ext4_mark_group_bitmap_corrupted(sb, block_group,
> +                                       EXT4_GROUP_INFO_BBITMAP_CORRUPT);

For this particular part, looks good to me.

Reviewed-by: Liu Bo <bo.liu@xxxxxxxxxxxxxxxxx>

thanks,
liubo

>                 return -EIO;
>         }
>         clear_buffer_new(bh);
> diff --git a/fs/ext4/mballoc.c b/fs/ext4/mballoc.c
> index bc2d1eb9fd5d..0d473991eebd 100644
> --- a/fs/ext4/mballoc.c
> +++ b/fs/ext4/mballoc.c
> @@ -470,6 +470,8 @@ static void mb_free_blocks_double(struct inode *inode, struct ext4_buddy *e4b,
>                                               "freeing block already freed "
>                                               "(bit %u)",
>                                               first + i);
> +                       ext4_mark_group_bitmap_corrupted(sb, e4b->bd_group,
> +                                       EXT4_GROUP_INFO_BBITMAP_CORRUPT);
>                 }
>                 mb_clear_bit(first + i, e4b->bd_info->bb_bitmap);
>         }
> @@ -1950,6 +1952,8 @@ void ext4_mb_complex_scan_group(struct ext4_allocation_context *ac,
>                                         "%d free clusters as per "
>                                         "group info. But bitmap says 0",
>                                         free);
> +                       ext4_mark_group_bitmap_corrupted(sb, e4b->bd_group,
> +                                       EXT4_GROUP_INFO_BBITMAP_CORRUPT);
>                         break;
>                 }
>
> @@ -1960,6 +1964,8 @@ void ext4_mb_complex_scan_group(struct ext4_allocation_context *ac,
>                                         "%d free clusters as per "
>                                         "group info. But got %d blocks",
>                                         free, ex.fe_len);
> +                       ext4_mark_group_bitmap_corrupted(sb, e4b->bd_group,
> +                                       EXT4_GROUP_INFO_BBITMAP_CORRUPT);
>                         /*
>                          * The number of free blocks differs. This mostly
>                          * indicate that the bitmap is corrupt. So exit
> --
> 2.14.3
>



[Index of Archives]     [Reiser Filesystem Development]     [Ceph FS]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite National Park]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Device Mapper]     [Linux Media]

  Powered by Linux