Re: [f2fs-dev] [PATCH] f2fs: add bio cache for IPU

2018-11-16 Thread Chao Yu
Hi Jaegeuk, On 2018/11/15 16:32, Chao Yu wrote: > Hi Jaeguek, > > On 2018/11/15 15:52, Jaegeuk Kim wrote: >> Hi Chao, >> >> I suspect this brings a system stuck by missing bio being flushed. Please >> check >> the changes. > > Oh, let me check this patch. I found one corner case below, but not

Re: [f2fs-dev] [PATCH] f2fs: add bio cache for IPU

2018-11-15 Thread Chao Yu
Hi Jaeguek, On 2018/11/15 15:52, Jaegeuk Kim wrote: > Hi Chao, > > I suspect this brings a system stuck by missing bio being flushed. Please > check > the changes. Oh, let me check this patch. Thanks, > > Thanks, > > On 10/23, Chao Yu wrote: >> SQLite in Wal mode may trigger sequential IPU

Re: [f2fs-dev] [PATCH] f2fs: add bio cache for IPU

2018-11-14 Thread Jaegeuk Kim
Hi Chao, I suspect this brings a system stuck by missing bio being flushed. Please check the changes. Thanks, On 10/23, Chao Yu wrote: > SQLite in Wal mode may trigger sequential IPU write in db-wal file, after > commit d1b3e72d5490 ("f2fs: submit bio of in-place-update pages"), we > lost the ch

[f2fs-dev] [PATCH] f2fs: add bio cache for IPU

2018-10-23 Thread Chao Yu
SQLite in Wal mode may trigger sequential IPU write in db-wal file, after commit d1b3e72d5490 ("f2fs: submit bio of in-place-update pages"), we lost the chance of merging page in inner managed bio cache, result in submitting more small-sized IO. So let's add temporary bio in writepages() to cache