diff options
| author | Chao Yu | 2016-07-03 16:05:13 +0200 |
|---|---|---|
| committer | Jaegeuk Kim | 2016-07-08 19:33:26 +0200 |
| commit | 72e1c797b57da42ce8f75c5636720f40c4f607a2 (patch) | |
| tree | a40b43480bef62e66cfc7126235fb200b419384a /fs/f2fs/gc.c | |
| parent | f2fs: fix to detect truncation prior rather than EIO during read (diff) | |
| download | kernel-qcow2-linux-72e1c797b57da42ce8f75c5636720f40c4f607a2.tar.gz kernel-qcow2-linux-72e1c797b57da42ce8f75c5636720f40c4f607a2.tar.xz kernel-qcow2-linux-72e1c797b57da42ce8f75c5636720f40c4f607a2.zip | |
f2fs: fix to redirty page if fail to gc data page
If we fail to move data page during foreground GC, we should give another
chance to writeback that page which was set dirty previously by writer.
Signed-off-by: Chao Yu <yuchao0@huawei.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'fs/f2fs/gc.c')
| -rw-r--r-- | fs/f2fs/gc.c | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c index c9602d0dc57a..c61213785914 100644 --- a/fs/f2fs/gc.c +++ b/fs/f2fs/gc.c @@ -653,12 +653,23 @@ static void move_data_page(struct inode *inode, block_t bidx, int gc_type) .page = page, .encrypted_page = NULL, }; + bool is_dirty = PageDirty(page); + int err; + +retry: set_page_dirty(page); f2fs_wait_on_page_writeback(page, DATA, true); if (clear_page_dirty_for_io(page)) inode_dec_dirty_pages(inode); + set_cold_data(page); - do_write_data_page(&fio); + + err = do_write_data_page(&fio); + if (err == -ENOMEM && is_dirty) { + congestion_wait(BLK_RW_ASYNC, HZ/50); + goto retry; + } + clear_cold_data(page); } out: |