On 2021/7/28 0:41, Daeho Jeong wrote:
How about adding this?

skip_fill:

         if (map.m_pblk == COMPRESS_ADDR) {

                 if (start_blk & (cluster_size - 1)) {

                         ret = -EFSCORRUPTED;

                         goto out;

                 }

                 compr_cluster = true;

                 count_in_cluster = 1;

         } else if (compr_appended) {

It seems we can add a separate patch to cover all cases that cluster metadata is
going to be accessed rather than just fixing fiemap() case here?

Signed-off-by: Daeho Jeong <[email protected]>
Tested-by: Eric Biggers <[email protected]>

Anyway, this patch looks good to me.

Reviewed-by: Chao Yu <[email protected]>

Thanks,


_______________________________________________
Linux-f2fs-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to