On 2020/3/28 3:32, Jaegeuk Kim wrote: > On 03/27, Chao Yu wrote: >> Otherwise, for compressed inode, returned physical block address >> may be wrong. > > We can use bmap to check the allocated (non)compressed blocks.
Sure, I've updated it in v2. Thanks, > >> >> Signed-off-by: Chao Yu <yuch...@huawei.com> >> --- >> fs/f2fs/data.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c >> index 24643680489b..f22f3ba10a48 100644 >> --- a/fs/f2fs/data.c >> +++ b/fs/f2fs/data.c >> @@ -3591,6 +3591,8 @@ static sector_t f2fs_bmap(struct address_space >> *mapping, sector_t block) >> >> if (f2fs_has_inline_data(inode)) >> return 0; >> + if (f2fs_compressed_file(inode)) >> + return 0; >> >> /* make sure allocating whole blocks */ >> if (mapping_tagged(mapping, PAGECACHE_TAG_DIRTY)) >> -- >> 2.18.0.rc1 > . > _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
