Re: [PATCH 1/2] f2fs: fix to commit bio cache after flushing node pages

2016-09-29 Thread Jaegeuk Kim
On Thu, Sep 29, 2016 at 06:45:03PM +0800, Chao Yu wrote: > On 2016/9/29 4:19, Jaegeuk Kim wrote: > > On Tue, Sep 27, 2016 at 10:09:03AM +0800, Chao Yu wrote: > >> On 2016/9/27 9:39, Jaegeuk Kim wrote: > >>> On Tue, Sep 27, 2016 at 08:57:41AM +0800, Chao Yu wrote: > Hi Jaegeuk, > > On

[PATCH 1/2] f2fs: fix to commit bio cache after flushing node pages

2016-09-29 Thread Chao Yu
In sync_node_pages, we won't check and commit last merged pages in private bio cache of f2fs, as these pages were taged as writeback, someone who is waiting for writebacking of the page will be blocked until the cache was committed by someone else. We need to commit node type bio cache to avoid po

Re: [PATCH 1/2] f2fs: fix to commit bio cache after flushing node pages

2016-09-29 Thread Chao Yu
On 2016/9/29 4:19, Jaegeuk Kim wrote: > On Tue, Sep 27, 2016 at 10:09:03AM +0800, Chao Yu wrote: >> On 2016/9/27 9:39, Jaegeuk Kim wrote: >>> On Tue, Sep 27, 2016 at 08:57:41AM +0800, Chao Yu wrote: Hi Jaegeuk, On 2016/9/27 2:33, Jaegeuk Kim wrote: > Hi Chao, > > On Tue,

Re: [PATCH 1/2] f2fs: fix to commit bio cache after flushing node pages

2016-09-28 Thread Jaegeuk Kim
On Tue, Sep 27, 2016 at 10:09:03AM +0800, Chao Yu wrote: > On 2016/9/27 9:39, Jaegeuk Kim wrote: > > On Tue, Sep 27, 2016 at 08:57:41AM +0800, Chao Yu wrote: > >> Hi Jaegeuk, > >> > >> On 2016/9/27 2:33, Jaegeuk Kim wrote: > >>> Hi Chao, > >>> > >>> On Tue, Sep 27, 2016 at 12:09:52AM +0800, Chao Yu

Re: [PATCH 1/2] f2fs: fix to commit bio cache after flushing node pages

2016-09-26 Thread Chao Yu
On 2016/9/27 9:39, Jaegeuk Kim wrote: > On Tue, Sep 27, 2016 at 08:57:41AM +0800, Chao Yu wrote: >> Hi Jaegeuk, >> >> On 2016/9/27 2:33, Jaegeuk Kim wrote: >>> Hi Chao, >>> >>> On Tue, Sep 27, 2016 at 12:09:52AM +0800, Chao Yu wrote: From: Chao Yu In sync_node_pages, we won't check

Re: [PATCH 1/2] f2fs: fix to commit bio cache after flushing node pages

2016-09-26 Thread Jaegeuk Kim
On Tue, Sep 27, 2016 at 08:57:41AM +0800, Chao Yu wrote: > Hi Jaegeuk, > > On 2016/9/27 2:33, Jaegeuk Kim wrote: > > Hi Chao, > > > > On Tue, Sep 27, 2016 at 12:09:52AM +0800, Chao Yu wrote: > >> From: Chao Yu > >> > >> In sync_node_pages, we won't check and commit last merged pages in private >

Re: [PATCH 1/2] f2fs: fix to commit bio cache after flushing node pages

2016-09-26 Thread Chao Yu
Hi Jaegeuk, On 2016/9/27 2:33, Jaegeuk Kim wrote: > Hi Chao, > > On Tue, Sep 27, 2016 at 12:09:52AM +0800, Chao Yu wrote: >> From: Chao Yu >> >> In sync_node_pages, we won't check and commit last merged pages in private >> bio cache of f2fs, as these pages were taged as writeback, someone who is

Re: [PATCH 1/2] f2fs: fix to commit bio cache after flushing node pages

2016-09-26 Thread Jaegeuk Kim
Hi Chao, On Tue, Sep 27, 2016 at 12:09:52AM +0800, Chao Yu wrote: > From: Chao Yu > > In sync_node_pages, we won't check and commit last merged pages in private > bio cache of f2fs, as these pages were taged as writeback, someone who is > waiting for writebacking of the page will be blocked unti

[PATCH 1/2] f2fs: fix to commit bio cache after flushing node pages

2016-09-26 Thread Chao Yu
From: Chao Yu In sync_node_pages, we won't check and commit last merged pages in private bio cache of f2fs, as these pages were taged as writeback, someone who is waiting for writebacking of the page will be blocked until the cache was committed by someone else. We need to commit node type bio c