Sorry, this seems wrong, please ignore this :(

On 2018/10/8 21:09, Sheng Yong wrote:
If a curseg is not used, its SIT entry may have an inconsistent type.
This will be fixed during recover.

Fixes: df8065e52928be ("fsck.f2fs: introduce fsck_chk_curseg_info")
Signed-off-by: Sheng Yong <shengyo...@huawei.com>
---
  fsck/fsck.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fsck/fsck.c b/fsck/fsck.c
index 85d9823..dd7f6ae 100644
--- a/fsck/fsck.c
+++ b/fsck/fsck.c
@@ -2485,7 +2485,7 @@ int fsck_chk_curseg_info(struct f2fs_sb_info *sbi)
                se = get_seg_entry(sbi, curseg->segno);
                sum_blk = curseg->sum_blk;
- if (se->type != i) {
+               if (se->valid_blocks && se->type != i) {
                        ASSERT_MSG("Incorrect curseg [%d]: segno [0x%x] "
                                   "type(SIT) [%d]", i, curseg->segno,
                                   se->type);




_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to