All of lore.kernel.org
 help / color / mirror / Atom feed
From: Fredrik Hallenberg <megahallon@gmail.com>
To: u-boot@lists.denx.de
Subject: [PATCH] fs: ext4: Add metadata checksums support
Date: Sat, 13 Feb 2021 10:51:55 +0100	[thread overview]
Message-ID: <CAMsZVf9y0HQy98z3NmsaDNA_ZWRszoF+u=YiUGCJo5jRVpyPfQ@mail.gmail.com> (raw)
In-Reply-To: <20210212204803.GW10169@bill-the-cat>

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@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@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

  reply	other threads:[~2021-02-13  9:51 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-12 16:57 [PATCH] fs: ext4: Add metadata checksums support Fredrik Hallenberg
2021-02-12 20:48 ` Tom Rini
2021-02-13  9:51   ` Fredrik Hallenberg [this message]
2021-02-22 16:54     ` Fredrik Hallenberg
2021-02-22 17:40       ` Tom Rini
2021-04-07 14:20 ` Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAMsZVf9y0HQy98z3NmsaDNA_ZWRszoF+u=YiUGCJo5jRVpyPfQ@mail.gmail.com' \
    --to=megahallon@gmail.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.