Re: [PATCH] fs/erofs: Remove an unnecessary assertion

2023-08-07 Thread Tom Rini
On Wed, Jul 26, 2023 at 12:56:04PM +0800, Yifan Zhao wrote: > 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

Re: [PATCH] fs/erofs: Remove an unnecessary assertion

2023-07-26 Thread Sam Edwards
Thanks greatly for the speedy turnaround on this patch! On 7/25/23 22:56, Yifan Zhao wrote: Fixes: 3a21e92fc255 ("fs/erofs: Introduce new features including ztailpacking, fragments and dedupe") Signed-off-by: Yifan Zhao Tested-by: Sam Edwards

[PATCH] fs/erofs: Remove an unnecessary assertion

2023-07-26 Thread Yifan Zhao
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]

Re: [PATCH] fs/erofs: Remove an unnecessary assertion

2023-07-26 Thread Gao Xiang
On 2023/7/26 12:56, Yifan Zhao wrote: 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