Re: [PATCH] f2fs: fix out-of-free problem caused by atomic write

2017-10-27 Thread Jaegeuk Kim
On 10/26, Christoph Hellwig wrote: > On Thu, Oct 26, 2017 at 04:30:19PM +0200, Jaegeuk Kim wrote: > > On 10/26, Yunlong Song wrote: > > > f2fs_balance_fs only actives once in the commit_inmem_pages, but there > > > are more than one page to commit, so all the other pages will miss the > > > check.

Re: [PATCH] f2fs: fix out-of-free problem caused by atomic write

2017-10-26 Thread Christoph Hellwig
On Thu, Oct 26, 2017 at 04:30:19PM +0200, Jaegeuk Kim wrote: > On 10/26, Yunlong Song wrote: > > f2fs_balance_fs only actives once in the commit_inmem_pages, but there > > are more than one page to commit, so all the other pages will miss the > > check. This will lead to out-of-free problem when co

Re: [PATCH] f2fs: fix out-of-free problem caused by atomic write

2017-10-26 Thread Jaegeuk Kim
On 10/26, Yunlong Song wrote: > f2fs_balance_fs only actives once in the commit_inmem_pages, but there > are more than one page to commit, so all the other pages will miss the > check. This will lead to out-of-free problem when commit a very large > file. To fix it, we should do f2fs_balance_fs for

[PATCH] f2fs: fix out-of-free problem caused by atomic write

2017-10-26 Thread Yunlong Song
f2fs_balance_fs only actives once in the commit_inmem_pages, but there are more than one page to commit, so all the other pages will miss the check. This will lead to out-of-free problem when commit a very large file. To fix it, we should do f2fs_balance_fs for each inmem page. Signed-off-by: Yunl