From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jaegeuk Kim Subject: Re: [PATCH 4/7] resize.f2fs: get latest free segments Date: Thu, 3 Nov 2016 10:26:59 -0700 Message-ID: <20161103172659.GA76013@jaegeuk> References: <20161103052846.72244-1-jaegeuk@kernel.org> <20161103052846.72244-4-jaegeuk@kernel.org> <911fa82d-b2b8-5d0f-c469-09f39c9624a5@huawei.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from sog-mx-2.v43.ch3.sourceforge.com ([172.29.43.192] helo=mx.sourceforge.net) by sfs-ml-2.v29.ch3.sourceforge.com with esmtp (Exim 4.76) (envelope-from ) id 1c2M49-0000pF-3E for linux-f2fs-devel@lists.sourceforge.net; Thu, 03 Nov 2016 17:44:41 +0000 Received: from mail.kernel.org ([198.145.29.136]) by sog-mx-2.v43.ch3.sourceforge.com with esmtps (TLSv1:AES256-SHA:256) (Exim 4.76) id 1c2M48-0000bG-5v for linux-f2fs-devel@lists.sourceforge.net; Thu, 03 Nov 2016 17:44:41 +0000 Content-Disposition: inline In-Reply-To: <911fa82d-b2b8-5d0f-c469-09f39c9624a5@huawei.com> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: linux-f2fs-devel-bounces@lists.sourceforge.net To: heyunlei Cc: linux-f2fs-devel@lists.sourceforge.net 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 > > --- > > 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< In f2fs_defragment(), - flush_journal_entries() - migrate_blocks() - move_curseg_info() - zero-journal_entries() - write_curseg_info() - flush_sit_entries() - write_checkpoint() - free_segment_count = get_free_segments() - write cp pack Since I can't catch your point, could you please explain in more detail? > > 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