Re: [PATCH] f2fs: split bio cache

2017-05-11 Thread Chao Yu
On 2017/5/11 10:35, Jaegeuk Kim wrote: > On 05/11, Chao Yu wrote: >> On 2017/5/11 7:50, Jaegeuk Kim wrote: >>> On 05/09, Chao Yu wrote: Hi Jaegeuk, On 2017/5/9 5:23, Jaegeuk Kim wrote: > Hi Chao, > > I can't see a strong reason to split meta from data/node and rename the

Re: [PATCH] f2fs: split bio cache

2017-05-10 Thread Jaegeuk Kim
On 05/11, Chao Yu wrote: > On 2017/5/11 7:50, Jaegeuk Kim wrote: > > On 05/09, Chao Yu wrote: > >> Hi Jaegeuk, > >> > >> On 2017/5/9 5:23, Jaegeuk Kim wrote: > >>> Hi Chao, > >>> > >>> I can't see a strong reason to split meta from data/node and rename the > >>> existing > >>> function names. Inst

Re: [PATCH] f2fs: split bio cache

2017-05-10 Thread Chao Yu
On 2017/5/11 7:50, Jaegeuk Kim wrote: > On 05/09, Chao Yu wrote: >> Hi Jaegeuk, >> >> On 2017/5/9 5:23, Jaegeuk Kim wrote: >>> Hi Chao, >>> >>> I can't see a strong reason to split meta from data/node and rename the >>> existing >>> function names. Instead, how about keeping the existing one while

Re: [PATCH] f2fs: split bio cache

2017-05-10 Thread Jaegeuk Kim
On 05/09, Chao Yu wrote: > Hi Jaegeuk, > > On 2017/5/9 5:23, Jaegeuk Kim wrote: > > Hi Chao, > > > > I can't see a strong reason to split meta from data/node and rename the > > existing > > function names. Instead, how about keeping the existing one while adding > > some > > page types to deal

Re: [PATCH] f2fs: split bio cache

2017-05-09 Thread Chao Yu
On 2017/5/9 9:49, Chao Yu wrote: > Hi Jaegeuk, > > On 2017/5/9 5:23, Jaegeuk Kim wrote: >> Hi Chao, >> >> I can't see a strong reason to split meta from data/node and rename the >> existing >> function names. Instead, how about keeping the existing one while adding some >> page types to deal with

Re: [PATCH] f2fs: split bio cache

2017-05-08 Thread Chao Yu
Hi Jaegeuk, On 2017/5/9 5:23, Jaegeuk Kim wrote: > Hi Chao, > > I can't see a strong reason to split meta from data/node and rename the > existing > function names. Instead, how about keeping the existing one while adding some > page types to deal with log types? Hmm.. before write this patch,

Re: [PATCH] f2fs: split bio cache

2017-05-08 Thread Jaegeuk Kim
Hi Chao, I can't see a strong reason to split meta from data/node and rename the existing function names. Instead, how about keeping the existing one while adding some page types to deal with log types? Thanks, On 05/08, Chao Yu wrote: > Split DATA/NODE type bio cache according to different temp