From mboxrd@z Thu Jan 1 00:00:00 1970 From: Fredrik Hallenberg Date: Mon, 22 Feb 2021 17:54:45 +0100 Subject: [PATCH] fs: ext4: Add metadata checksums support In-Reply-To: References: <20210212165747.207979-1-megahallon@gmail.com> <20210212204803.GW10169@bill-the-cat> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Anything I can do to improve the patch? On Sat, Feb 13, 2021 at 10:51 AM Fredrik Hallenberg 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 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 > > > > 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