diff options
author | Namjae Jeon <namjae.jeon@samsung.com> | 2013-01-30 14:47:16 +0100 |
---|---|---|
committer | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2013-02-11 23:15:01 +0100 |
commit | a2b52a598a4df55782462acba3bdced1a4610a60 (patch) | |
tree | 0cbc977f6959856f24ee6eae52ba528342be84b2 /fs/f2fs | |
parent | f2fs: avoid redundant call to has_not_enough_free_secs in f2fs_gc (diff) | |
download | linux-a2b52a598a4df55782462acba3bdced1a4610a60.tar.xz linux-a2b52a598a4df55782462acba3bdced1a4610a60.zip |
f2fs: reorganize code for ra_node_page
We can remove unneeded label unlock_out, avoid unnecessary jump
and reorganize the returning conditions in this function.
Signed-off-by: Namjae Jeon <namjae.jeon@samsung.com>
Signed-off-by: Amit Sahrawat <a.sahrawat@samsung.com>
Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
Diffstat (limited to 'fs/f2fs')
-rw-r--r-- | fs/f2fs/node.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/fs/f2fs/node.c b/fs/f2fs/node.c index 9bda63c9c166..f71dfbbcb2b0 100644 --- a/fs/f2fs/node.c +++ b/fs/f2fs/node.c @@ -874,15 +874,11 @@ void ra_node_page(struct f2fs_sb_info *sbi, nid_t nid) return; if (read_node_page(apage, READA)) - goto unlock_out; + unlock_page(apage); - page_cache_release(apage); - return; - -unlock_out: - unlock_page(apage); release_out: page_cache_release(apage); + return; } struct page *get_node_page(struct f2fs_sb_info *sbi, pgoff_t nid) |