Re: [PATCH v2] f2fs: provide f2fs_balance_fs to __write_node_page

2017-07-29 Thread Yunlong Song
Hi Chao, Nice suggestion, I have sent out a new patch to implement it. On 2017/7/29 8:19, Chao Yu wrote: Hi Yunlong, On 2017/7/27 20:13, Yunlong Song wrote: v1->v2, fix some dead lock problems under some heavy load test On 2017/7/27 20:11, Yunlong Song wrote: Let node writeback also do

Re: [PATCH v2] f2fs: provide f2fs_balance_fs to __write_node_page

2017-07-29 Thread Yunlong Song
Hi Chao, Nice suggestion, I have sent out a new patch to implement it. On 2017/7/29 8:19, Chao Yu wrote: Hi Yunlong, On 2017/7/27 20:13, Yunlong Song wrote: v1->v2, fix some dead lock problems under some heavy load test On 2017/7/27 20:11, Yunlong Song wrote: Let node writeback also do

Re: [PATCH v2] f2fs: provide f2fs_balance_fs to __write_node_page

2017-07-29 Thread Yunlong Song
Hi, Jay, Not sure yet, whether it is just the case or not, at least we strengthen f2fs_balance_fs in some corner cases. And maybe there are still some corner cases which we have not considered yet. See my new patch "provide f2fs_balance_fs to __write_data_page for dentry pages", which is

Re: [PATCH v2] f2fs: provide f2fs_balance_fs to __write_node_page

2017-07-29 Thread Yunlong Song
Hi, Jay, Not sure yet, whether it is just the case or not, at least we strengthen f2fs_balance_fs in some corner cases. And maybe there are still some corner cases which we have not considered yet. See my new patch "provide f2fs_balance_fs to __write_data_page for dentry pages", which is

Re: [PATCH v2] f2fs: provide f2fs_balance_fs to __write_node_page

2017-07-28 Thread Chao Yu
Hi Yunlong, On 2017/7/27 20:13, Yunlong Song wrote: > v1->v2, fix some dead lock problems under some heavy load test > > On 2017/7/27 20:11, Yunlong Song wrote: >> Let node writeback also do f2fs_balance_fs to ensure there are always enough >> free >> segments. Could we cover __write_data_page

Re: [PATCH v2] f2fs: provide f2fs_balance_fs to __write_node_page

2017-07-28 Thread Chao Yu
Hi Yunlong, On 2017/7/27 20:13, Yunlong Song wrote: > v1->v2, fix some dead lock problems under some heavy load test > > On 2017/7/27 20:11, Yunlong Song wrote: >> Let node writeback also do f2fs_balance_fs to ensure there are always enough >> free >> segments. Could we cover __write_data_page

Re: [PATCH v2] f2fs: provide f2fs_balance_fs to __write_node_page

2017-07-28 Thread Jaegeuk Kim
Hi Yunlong, On 07/27, Yunlong Song wrote: > v1->v2, fix some dead lock problems under some heavy load test So, does this patch resolve the previous GC problem? Thanks, > > On 2017/7/27 20:11, Yunlong Song wrote: > > Let node writeback also do f2fs_balance_fs to ensure there are always > >

Re: [PATCH v2] f2fs: provide f2fs_balance_fs to __write_node_page

2017-07-28 Thread Jaegeuk Kim
Hi Yunlong, On 07/27, Yunlong Song wrote: > v1->v2, fix some dead lock problems under some heavy load test So, does this patch resolve the previous GC problem? Thanks, > > On 2017/7/27 20:11, Yunlong Song wrote: > > Let node writeback also do f2fs_balance_fs to ensure there are always > >

Re: [PATCH v2] f2fs: provide f2fs_balance_fs to __write_node_page

2017-07-27 Thread Yunlong Song
v1->v2, fix some dead lock problems under some heavy load test On 2017/7/27 20:11, Yunlong Song wrote: Let node writeback also do f2fs_balance_fs to ensure there are always enough free segments. Signed-off-by: Yunlong Song --- fs/f2fs/checkpoint.c | 2 +-

Re: [PATCH v2] f2fs: provide f2fs_balance_fs to __write_node_page

2017-07-27 Thread Yunlong Song
v1->v2, fix some dead lock problems under some heavy load test On 2017/7/27 20:11, Yunlong Song wrote: Let node writeback also do f2fs_balance_fs to ensure there are always enough free segments. Signed-off-by: Yunlong Song --- fs/f2fs/checkpoint.c | 2 +- fs/f2fs/f2fs.h | 2 +-

[PATCH v2] f2fs: provide f2fs_balance_fs to __write_node_page

2017-07-27 Thread Yunlong Song
Let node writeback also do f2fs_balance_fs to ensure there are always enough free segments. Signed-off-by: Yunlong Song --- fs/f2fs/checkpoint.c | 2 +- fs/f2fs/f2fs.h | 2 +- fs/f2fs/node.c | 14 -- 3 files changed, 10 insertions(+), 8

[PATCH v2] f2fs: provide f2fs_balance_fs to __write_node_page

2017-07-27 Thread Yunlong Song
Let node writeback also do f2fs_balance_fs to ensure there are always enough free segments. Signed-off-by: Yunlong Song --- fs/f2fs/checkpoint.c | 2 +- fs/f2fs/f2fs.h | 2 +- fs/f2fs/node.c | 14 -- 3 files changed, 10 insertions(+), 8 deletions(-) diff --git