Re: [PATCH v7 0/2] btrfs: Introduce new rescue= mount options

[Anand Jain <anand.jain@oracle.com>]

On 4/6/20 3:18 pm, Qu Wenruo wrote:
> There are quite a lot btrfs extent tree corruption report in the mail
> list.

> Since btrfs will do mount time block group item search, one corrupted
> leaf containing block group item will prevent the whole fs to be
> mounted.
> 

  Can you add btrfs_info/warn() at those places to indicate
  -o rescue=skip.. might help to mount in RO.

> This patchset will try to address the problem by introducing a new mount
> option, "rescue=skipbg", as a last-resort rescue.

  Do you think there might be another check to skip during mount at some
  point. So if why not add a generic -o rescue=skipchecks? Of course
  dmesg -k must show what has been skipped.

> This "rescue=skipbg" has some advantage compared to user space tool
> like "btrfs-restore":
> - Unified recovery tool

  Yes.

Thanks, Anand