On Thu, Nov 03, 2016 at 02:53:20PM +0800, heyunlei wrote: > > Hi, Kim > On 2016/11/3 13:28, Jaegeuk Kim wrote: > > The # of free segments can be changed by defragmentation. > > > > Signed-off-by: Jaegeuk Kim <jaeg...@kernel.org> > > --- > > fsck/resize.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/fsck/resize.c b/fsck/resize.c > > index 5dc73a8..4813de7 100644 > > --- a/fsck/resize.c > > +++ b/fsck/resize.c > > @@ -424,7 +424,7 @@ static void rebuild_checkpoint(struct f2fs_sb_info *sbi, > > set_cp(overprov_segment_count, get_cp(overprov_segment_count) + > > get_cp(rsvd_segment_count)); > > > > - free_segment_count = get_cp(free_segment_count); > > + free_segment_count = get_free_segments(sbi); > > f2fs_defragment will write a checkpoint and then update free_segment_count > by get_free_segments(sbi)o< Is it righto<
Oh, I misread your point. Yup, but, this is to handle f2fs_defragment incompletion. In that case, resize.f2fs would call migrate_main() and finally rebuild_checkpoint() with wrong free_segments. > > Thankso< > > new_segment_count = get_newsb(segment_count_main) - > > get_sb(segment_count_main); > > > > ------------------------------------------------------------------------------ Developer Access Program for Intel Xeon Phi Processors Access to Intel Xeon Phi processor-based developer platforms. With one year of Intel Parallel Studio XE. Training and support from Colfax. Order your platform today. http://sdm.link/xeonphi _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
