Re: [f2fs-dev] [PATCH v5 1/2] f2fs: fix to avoid broken of dnode block list

2018-07-28 Thread Chao Yu
On 2018/7/29 10:49, Jaegeuk Kim wrote: > On 07/29, Chao Yu wrote: >> On 2018/7/29 9:33, Jaegeuk Kim wrote: >>> On 07/28, Chao Yu wrote: f2fs recovery flow is relying on dnode block link list, it means fsynced file recovery depends on previous dnode's persistence in the list, so

Re: [f2fs-dev] [PATCH v5 1/2] f2fs: fix to avoid broken of dnode block list

2018-07-28 Thread Jaegeuk Kim
On 07/29, Chao Yu wrote: > On 2018/7/29 9:33, Jaegeuk Kim wrote: > > On 07/28, Chao Yu wrote: > >> f2fs recovery flow is relying on dnode block link list, it means fsynced > >> file recovery depends on previous dnode's persistence in the list, so > >> during fsync() we should wait on all regular

Re: [f2fs-dev] [PATCH v5 1/2] f2fs: fix to avoid broken of dnode block list

2018-07-28 Thread Chao Yu
On 2018/7/29 9:33, Jaegeuk Kim wrote: > On 07/28, Chao Yu wrote: >> f2fs recovery flow is relying on dnode block link list, it means fsynced >> file recovery depends on previous dnode's persistence in the list, so >> during fsync() we should wait on all regular inode's dnode writebacked >> before

Re: [f2fs-dev] [PATCH v5 1/2] f2fs: fix to avoid broken of dnode block list

2018-07-28 Thread Jaegeuk Kim
On 07/28, Chao Yu wrote: > f2fs recovery flow is relying on dnode block link list, it means fsynced > file recovery depends on previous dnode's persistence in the list, so > during fsync() we should wait on all regular inode's dnode writebacked > before issuing flush. > > By this way, we can