diff options
| author | Changman Lee <cm224.lee@samsung.com> | 2014-11-27 16:03:08 +0900 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk@kernel.org> | 2014-11-27 20:30:17 -0800 |
| commit | 31a3268839c1aa808a5109111ec847b95e1bb114 (patch) | |
| tree | 2d2c6f020a0c4e241d1e95c4b3d6ae5f90e66d61 /fs/f2fs | |
| parent | 95f5b0fc5e25d94ace84dbf5bb4acb8b80b4f062 (diff) | |
| download | linux-31a3268839c1aa808a5109111ec847b95e1bb114.tar.bz2 | |
f2fs: cleanup if-statement of phase in gc_data_segment
Little cleanup to distinguish each phase easily
Signed-off-by: Changman Lee <cm224.lee@samsung.com>
[Jaegeuk Kim: modify indentation for code readability]
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'fs/f2fs')
| -rw-r--r-- | fs/f2fs/gc.c | 32 |
1 files changed, 16 insertions, 16 deletions
diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c index 657683c9ee48..6acd5f240224 100644 --- a/fs/f2fs/gc.c +++ b/fs/f2fs/gc.c @@ -603,27 +603,27 @@ next_step: data_page = find_data_page(inode, start_bidx + ofs_in_node, false); - if (IS_ERR(data_page)) - goto next_iput; + if (IS_ERR(data_page)) { + iput(inode); + continue; + } f2fs_put_page(data_page, 0); add_gc_inode(inode, ilist); - } else { - inode = find_gc_inode(dni.ino, ilist); - if (inode) { - start_bidx = start_bidx_of_node(nofs, - F2FS_I(inode)); - data_page = get_lock_data_page(inode, + continue; + } + + /* phase 3 */ + inode = find_gc_inode(dni.ino, ilist); + if (inode) { + start_bidx = start_bidx_of_node(nofs, F2FS_I(inode)); + data_page = get_lock_data_page(inode, start_bidx + ofs_in_node); - if (IS_ERR(data_page)) - continue; - move_data_page(inode, data_page, gc_type); - stat_inc_data_blk_count(sbi, 1); - } + if (IS_ERR(data_page)) + continue; + move_data_page(inode, data_page, gc_type); + stat_inc_data_blk_count(sbi, 1); } - continue; -next_iput: - iput(inode); } if (++phase < 4) |