Previously SSA is updated if it is not included in current segment info. But, defrag.f2fs doesn't handle current segment info during the process, and instead lastly update the whole current segment info at a time. So, we need to update summary entries all the time. Otherwise, we can lose the SSA entry.
Signed-off-by: Jaegeuk Kim <[email protected]> --- fsck/mount.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/fsck/mount.c b/fsck/mount.c index 79611e5..d2f1432 100644 --- a/fsck/mount.c +++ b/fsck/mount.c @@ -761,13 +761,16 @@ void update_sum_entry(struct f2fs_sb_info *sbi, block_t blk_addr, sum_blk->footer.entry_type = IS_NODESEG(se->type) ? SUM_TYPE_NODE : SUM_TYPE_DATA; - if (type == SEG_TYPE_NODE || type == SEG_TYPE_DATA || - type == SEG_TYPE_MAX) { + /* write SSA all the time */ + if (type < SEG_TYPE_MAX) { u64 ssa_blk = GET_SUM_BLKADDR(sbi, segno); ret = dev_write_block(sum_blk, ssa_blk); ASSERT(ret >= 0); - free(sum_blk); } + + if (type == SEG_TYPE_NODE || type == SEG_TYPE_DATA || + type == SEG_TYPE_MAX) + free(sum_blk); } static void restore_curseg_summaries(struct f2fs_sb_info *sbi) -- 2.5.4 (Apple Git-61) ------------------------------------------------------------------------------ _______________________________________________ Linux-f2fs-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
