[PATCH] fs/erofs: Remove an unnecessary assertion

Yifan Zhao zhaoyifan at sjtu.edu.cn
Wed Jul 26 06:56:04 CEST 2023


In [1] Sam points out an assertion does not hold true for 32-bit
platforms, which only impacts Large File Support (LFS) API usage
in erofs-utils according to Xiang [2]. We don't think these APIs
are used in u-boot and this restriction could be safely removed.

[1] https://lists.denx.de/pipermail/u-boot/2023-July/524679.html
[2] https://lists.denx.de/pipermail/u-boot/2023-July/524727.html

Fixes: 3a21e92fc255 ("fs/erofs: Introduce new features including ztailpacking, fragments and dedupe")
Signed-off-by: Yifan Zhao <zhaoyifan at sjtu.edu.cn>
---
 fs/erofs/internal.h | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/fs/erofs/internal.h b/fs/erofs/internal.h
index 433a3c6c1e..1875f37fcd 100644
--- a/fs/erofs/internal.h
+++ b/fs/erofs/internal.h
@@ -105,9 +105,6 @@ struct erofs_sb_info {
 	u8 xattr_prefix_count;
 };
 
-/* make sure that any user of the erofs headers has at least 64bit off_t type */
-extern int erofs_assert_largefile[sizeof(off_t) - 8];
-
 static inline erofs_off_t iloc(erofs_nid_t nid)
 {
 	return erofs_pos(sbi.meta_blkaddr) + (nid << sbi.islotbits);
-- 
2.41.0



More information about the U-Boot mailing list