Re: [PATCH] fs/ext2: Ignore checksum seed incompat feature support

[Daniel Kiper]

On Fri, Jun 11, 2021 at 09:36:16PM +0200, Javier Martinez Canillas wrote:
> This incompat feature is used to denote that the filesystem stored its
> metadata checksum seed in the superblock. This is used to allow tune2fs
> to change the UUID on a mounted metadata_csum filesystem without having
> to rewrite all the disk metadata.
>
> But GRUB doesn't use the metadata checksum in anyway, so can just ignore
> this feature if is enabled. This is consistent with GRUB filesystem code
> in general which just does a best effort to access the filesystem's data.
>
> It may be removed from the ignored list in the future if supports to do
> metadata checksumming verification is added to the read-only FS driver.
>
> Suggested-by: Eric Sandeen <esandeen@redhat.com>
> Suggested-by: Lukas Czerner <lczerner@redhat.com>
> Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>

Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>

Daniel