Re: [PATCH] f2fs: fix to correct freed section number during gc

2015-09-23 Thread Jaegeuk Kim
@lists.sourceforge.net; linux-kernel@vger.kernel.org > > Subject: Re: [PATCH] f2fs: fix to correct freed section number during gc > > > > Hi Chao, > > > > On Tue, Sep 22, 2015 at 09:18:18PM +0800, Chao Yu wrote: > > > We pass 'nfree' to has_not_enough_free_secs

RE: [PATCH] f2fs: fix to correct freed section number during gc

2015-09-23 Thread Chao Yu
Hi Jaegeuk, > -Original Message- > From: Jaegeuk Kim [mailto:jaeg...@kernel.org] > Sent: Wednesday, September 23, 2015 6:54 AM > To: Chao Yu > Cc: linux-f2fs-de...@lists.sourceforge.net; linux-kernel@vger.kernel.org > Subject: Re: [PATCH] f2fs: fix to correct freed sec

RE: [PATCH] f2fs: fix to correct freed section number during gc

2015-09-23 Thread Chao Yu
Hi Jaegeuk, > -Original Message- > From: Jaegeuk Kim [mailto:jaeg...@kernel.org] > Sent: Wednesday, September 23, 2015 6:54 AM > To: Chao Yu > Cc: linux-f2fs-de...@lists.sourceforge.net; linux-kernel@vger.kernel.org > Subject: Re: [PATCH] f2fs: fix to correct freed sec

Re: [PATCH] f2fs: fix to correct freed section number during gc

2015-09-23 Thread Jaegeuk Kim
@lists.sourceforge.net; linux-kernel@vger.kernel.org > > Subject: Re: [PATCH] f2fs: fix to correct freed section number during gc > > > > Hi Chao, > > > > On Tue, Sep 22, 2015 at 09:18:18PM +0800, Chao Yu wrote: > > > We pass 'nfree' to has_not_enough_free_secs

Re: [PATCH] f2fs: fix to correct freed section number during gc

2015-09-22 Thread Jaegeuk Kim
Hi Chao, On Tue, Sep 22, 2015 at 09:18:18PM +0800, Chao Yu wrote: > We pass 'nfree' to has_not_enough_free_secs to check whether there is > enough free section, but 'nfree' indicates the number of segment gced, > should alter the value to section number. Yeah, but I think we need to increase

Re: [PATCH] f2fs: fix to correct freed section number during gc

2015-09-22 Thread Jaegeuk Kim
Hi Chao, On Tue, Sep 22, 2015 at 09:18:18PM +0800, Chao Yu wrote: > We pass 'nfree' to has_not_enough_free_secs to check whether there is > enough free section, but 'nfree' indicates the number of segment gced, > should alter the value to section number. Yeah, but I think we need to increase