From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [PATCH] fs: ext4: Add metadata checksums support
Date: Wed, 7 Apr 2021 10:20:16 -0400 [thread overview]
Message-ID: <20210407142016.GA20614@bill-the-cat> (raw)
In-Reply-To: <20210212165747.207979-1-megahallon@gmail.com>
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>
This fails broadly with an error like:
aarch64: + ls1046ardb_qspi
+(ls1046ardb_qspi) In file included from ../include/linux/bitops.h:22,
+(ls1046ardb_qspi) from include/asm/arch/config.h:14,
+(ls1046ardb_qspi) from ../include/configs/ls1046a_common.h:32,
+(ls1046ardb_qspi) from ../include/configs/ls1046ardb.h:10,
+(ls1046ardb_qspi) from include/config.h:4,
+(ls1046ardb_qspi) from ../include/common.h:16,
+(ls1046ardb_qspi) from ../fs/ext4/ext4_common.c:21:
+(ls1046ardb_qspi) ../fs/ext4/ext4_common.c: In function 'ext4fs_read_inode':
+(ls1046ardb_qspi) ../include/linux/kernel.h:184:17: error: comparison of distinct pointer types lacks a cast [-Werror]
+(ls1046ardb_qspi) 184 | (void) (&_min1 == &_min2); \
+(ls1046ardb_qspi) | ^~
+(ls1046ardb_qspi) ../fs/ext4/ext4_common.c:1655:9: note: in expansion of macro 'min'
+(ls1046ardb_qspi) 1655 | size = min(sizeof(struct ext2_inode), fs->inodesz);
+(ls1046ardb_qspi) | ^~~
+(ls1046ardb_qspi) cc1: all warnings being treated as errors
+(ls1046ardb_qspi) make[3]: *** [fs/ext4/ext4_common.o] Error 1
+(ls1046ardb_qspi) make[2]: *** [fs/ext4] Error 2
+(ls1046ardb_qspi) make[1]: *** [fs] Error 2
+(ls1046ardb_qspi) make: *** [sub-make] Error 2
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210407/31a03cb6/attachment.sig>
prev parent reply other threads:[~2021-04-07 14:20 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
2021-02-22 16:54 ` Fredrik Hallenberg
2021-02-22 17:40 ` Tom Rini
2021-04-07 14:20 ` Tom Rini [this message]
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=20210407142016.GA20614@bill-the-cat \
--to=trini@konsulko.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox