f2fs: compress: add sanity check during compressed cluster read
authorChao Yu <yuchao0@huawei.com>
Wed, 29 Jul 2020 13:21:35 +0000 (21:21 +0800)
committerJaegeuk Kim <jaegeuk@kernel.org>
Mon, 3 Aug 2020 17:32:51 +0000 (10:32 -0700)
In f2fs_read_multi_pages(), we don't have to check cluster's type
again, since overwrite or partial truncation need page lock in
cluster which has already been held by reader, so cluster's type
is stable, let's change check condition to sanity check.

Signed-off-by: Chao Yu <yuchao0@huawei.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/data.c

index bc89bc987513ec63b5f0467293f014a704aa07e7..c1b676be67b9a186de549f8d0d2f89e6ba444f4d 100644 (file)
@@ -2170,9 +2170,7 @@ int f2fs_read_multi_pages(struct compress_ctx *cc, struct bio **bio_ret,
        if (ret)
                goto out;
 
-       /* cluster was overwritten as normal cluster */
-       if (dn.data_blkaddr != COMPRESS_ADDR)
-               goto out;
+       f2fs_bug_on(sbi, dn.data_blkaddr != COMPRESS_ADDR);
 
        for (i = 1; i < cc->cluster_size; i++) {
                block_t blkaddr;