Re: [f2fs-dev] [PATCH 2/2] f2fs: fix to avoid data update racing between GC and DIO

2016-07-06 Thread Chao Yu
On 2016/7/7 6:37, Jaegeuk Kim wrote: > On Wed, Jul 06, 2016 at 10:10:57AM +0800, Chao Yu wrote: >> On 2016/7/6 8:24, Jaegeuk Kim wrote: >>> On Fri, Jul 01, 2016 at 02:03:17PM +0800, Chao Yu wrote: Hi Jaegeuk, On 2016/7/1 8:03, Jaegeuk Kim wrote: > Hi Chao, > > On Thu, Jun

Re: [PATCH 2/2] f2fs: fix to avoid data update racing between GC and DIO

2016-07-06 Thread Jaegeuk Kim
On Wed, Jul 06, 2016 at 10:10:57AM +0800, Chao Yu wrote: > On 2016/7/6 8:24, Jaegeuk Kim wrote: > > On Fri, Jul 01, 2016 at 02:03:17PM +0800, Chao Yu wrote: > >> Hi Jaegeuk, > >> > >> On 2016/7/1 8:03, Jaegeuk Kim wrote: > >>> Hi Chao, > >>> > >>> On Thu, Jun 30, 2016 at 04:42:48PM +0800, Chao Yu w

Re: [PATCH 2/2] f2fs: fix to avoid data update racing between GC and DIO

2016-07-05 Thread Chao Yu
On 2016/7/6 8:24, Jaegeuk Kim wrote: > On Fri, Jul 01, 2016 at 02:03:17PM +0800, Chao Yu wrote: >> Hi Jaegeuk, >> >> On 2016/7/1 8:03, Jaegeuk Kim wrote: >>> Hi Chao, >>> >>> On Thu, Jun 30, 2016 at 04:42:48PM +0800, Chao Yu wrote: Datas in file can be operated by GC and DIO simultaneously, so

Re: [PATCH 2/2] f2fs: fix to avoid data update racing between GC and DIO

2016-07-05 Thread Jaegeuk Kim
On Fri, Jul 01, 2016 at 02:03:17PM +0800, Chao Yu wrote: > Hi Jaegeuk, > > On 2016/7/1 8:03, Jaegeuk Kim wrote: > > Hi Chao, > > > > On Thu, Jun 30, 2016 at 04:42:48PM +0800, Chao Yu wrote: > >> Datas in file can be operated by GC and DIO simultaneously, so we will > >> face race case as below: >

Re: [PATCH 2/2] f2fs: fix to avoid data update racing between GC and DIO

2016-06-30 Thread Chao Yu
Hi Jaegeuk, On 2016/7/1 8:03, Jaegeuk Kim wrote: > Hi Chao, > > On Thu, Jun 30, 2016 at 04:42:48PM +0800, Chao Yu wrote: >> Datas in file can be operated by GC and DIO simultaneously, so we will >> face race case as below: >> >> For write case: >> Thread A Thread B >>

Re: [PATCH 2/2] f2fs: fix to avoid data update racing between GC and DIO

2016-06-30 Thread Jaegeuk Kim
Hi Chao, On Thu, Jun 30, 2016 at 04:42:48PM +0800, Chao Yu wrote: > Datas in file can be operated by GC and DIO simultaneously, so we will > face race case as below: > > For write case: > Thread A Thread B > - generic_file_direct_write > - invalidate_inode_pages2_ran

[PATCH 2/2] f2fs: fix to avoid data update racing between GC and DIO

2016-06-30 Thread Chao Yu
Datas in file can be operated by GC and DIO simultaneously, so we will face race case as below: For write case: Thread AThread B - generic_file_direct_write - invalidate_inode_pages2_range - f2fs_direct_IO - do_blockdev_direct_IO - do_direct_IO - get_more