Re: [f2fs-dev] [PATCH 7/8] f2fs: introduce new option for controlling data flush

2015-12-16 Thread Jaegeuk Kim
On Wed, Dec 16, 2015 at 11:19:06AM +0800, Chao Yu wrote: > Hi Jaegeuk, > > > -Original Message- > > From: Jaegeuk Kim [mailto:jaeg...@kernel.org] > > Sent: Wednesday, December 16, 2015 10:49 AM > > To: Chao Yu > > Cc: linux-f2fs-devel@lists.sourceforge.net; linux-ker...@vger.kernel.org >

Re: [f2fs-dev] [PATCH 7/8] f2fs: introduce new option for controlling data flush

2015-12-15 Thread Chao Yu
Hi Jaegeuk, The 7, 8 patches could not be merged on current dev-test branch due to I generate them on incorrect base line. I will rebase and resent them tomorrow, sorry for the noisy. Thanks, On 12/15/15 1:35 PM, Chao Yu wrote: > Add a new option 'data_flush' to enable/disable data flush

Re: [f2fs-dev] [PATCH 7/8] f2fs: introduce new option for controlling data flush

2015-12-15 Thread Jaegeuk Kim
Hi, On Tue, Dec 15, 2015 at 01:35:21PM +0800, Chao Yu wrote: > Add a new option 'data_flush' to enable/disable data flush functionality > in checkpoint. > > Signed-off-by: Chao Yu > --- > Documentation/filesystems/f2fs.txt | 2 ++ > fs/f2fs/checkpoint.c | 2

Re: [f2fs-dev] [PATCH 7/8] f2fs: introduce new option for controlling data flush

2015-12-15 Thread Chao Yu
Hi Jaegeuk, > -Original Message- > From: Jaegeuk Kim [mailto:jaeg...@kernel.org] > Sent: Wednesday, December 16, 2015 6:03 AM > To: Chao Yu > Cc: linux-f2fs-devel@lists.sourceforge.net; linux-ker...@vger.kernel.org > Subject: Re: [PATCH 7/8] f2fs: introduce new option for controlling data

Re: [f2fs-dev] [PATCH 7/8] f2fs: introduce new option for controlling data flush

2015-12-15 Thread Jaegeuk Kim
Hi Chao, On Wed, Dec 16, 2015 at 10:38:27AM +0800, Chao Yu wrote: ... > > > +++ b/fs/f2fs/super.c > > > @@ -67,6 +67,7 @@ enum { > > > Opt_extent_cache, > > > Opt_noextent_cache, > > > Opt_noinline_data, > > > + Opt_data_flush, > > > > Need Opt_nodata_flush too. > > We need this unless