Re: [PATCH v2 0/5] New rescue mount options

[David Sterba <dsterba@suse.cz>]

On Mon, Sep 28, 2020 at 02:34:52PM -0400, Josef Bacik wrote:
> And finally we have rescue=all, which simply enables ignoredatacsums,
> ignorebadroots, and notreelogreplay.  We need an easy catch-all option for
> distros to fallback on to get users the highest probability of being able to
> recover their data, so we will use rescue=all to turn on all the fanciest rescue
> options that we have, and then use the discrete options for more fine grained
> recovery.  Thanks,
> 
> Josef
> 
> Josef Bacik (5):
>   btrfs: unify the ro checking for mount options
>   btrfs: push the NODATASUM check into btrfs_lookup_bio_sums
>   btrfs: introduce rescue=ignorebadroots
>   btrfs: introduce rescue=ignoredatacsums
>   btrfs: introduce rescue=all

About the naming, the option names are quite clear but long to type. We
could have shorter aliases, eg. 'ibadroots', where 'i' is short for
'ignore'.

For completeness, we may also want to the negated options or add an
operator that will drop the bit, like

	mount -o rescue=all:^ignoredatacsums
	mount -o rescue=all,rescue=^ignoredatacsums