Re: [f2fs-dev] [PATCH] f2fs: unify the error handling of f2fs_is_valid_blkaddr

2023-12-11 Thread Zhiguo Niu
Dear Chao On Tue, Dec 12, 2023 at 9:32 AM Chao Yu wrote: > > On 2023/12/1 13:42, Zhiguo Niu wrote: > > unify the error handling of f2fs_is_valid_blkaddr and remove some > > redundant codes in f2fs_cache_compressed_page. > > What about moving f2fs_handle_error(ERROR_INVALID_BLKADDR) into >

Re: [f2fs-dev] [PATCH] f2fs: unify the error handling of f2fs_is_valid_blkaddr

2023-12-11 Thread Chao Yu
On 2023/12/1 13:42, Zhiguo Niu wrote: unify the error handling of f2fs_is_valid_blkaddr and remove some redundant codes in f2fs_cache_compressed_page. What about moving f2fs_handle_error(ERROR_INVALID_BLKADDR) into f2fs_is_valid_blkaddr() for cleanup? Thanks, Signed-off-by: Zhiguo Niu ---

[f2fs-dev] [PATCH] f2fs: unify the error handling of f2fs_is_valid_blkaddr

2023-11-30 Thread Zhiguo Niu
unify the error handling of f2fs_is_valid_blkaddr and remove some redundant codes in f2fs_cache_compressed_page. Signed-off-by: Zhiguo Niu --- fs/f2fs/checkpoint.c | 6 +- fs/f2fs/compress.c | 8 +++- fs/f2fs/data.c | 3 ++- fs/f2fs/extent_cache.c | 7 ---