[PATCH] fs: ext4: Add metadata checksums support

[Fredrik Hallenberg]

Anything I can do to improve the patch?

On Sat, Feb 13, 2021 at 10:51 AM Fredrik Hallenberg
<megahallon at gmail.com> wrote:
>
> Thanks for trying it out. I have only used it on a custom arm-based
> board. I have done quite a lot of writing with and without metadata
> flag and have also tried to trigger errors that require journal
> recovery. It has worked well but of course it is hard to be sure that
> the code will handle every case.
>
> On Fri, Feb 12, 2021 at 9:48 PM Tom Rini <trini at konsulko.com> wrote:
> >
> > On Fri, Feb 12, 2021 at 05:57:47PM +0100, Fredrik Hallenberg wrote:
> >
> > > Support crc32c checksums in ext4 filesystems with metadata_csum flag
> > > active. This includes superblock, inodes, inode and block group tables,
> > > directory blocks and journal.
> > >
> > > Signed-off-by: Fredrik Hallenberg <megahallon at gmail.com>
> >
> > I've initially put this in a local branch and run the fs tests on
> > sandbox both with default options and forcing metadata_csum to be
> > enabled, and it worked in both cases.  Thanks for posting the patch!
> > Can you let us know on what systems you've tested this?
> >
> > --
> > Tom