summaryrefslogtreecommitdiffstats
path: root/fs/f2fs
diff options
context:
space:
mode:
authorNamjae Jeon <namjae.jeon@samsung.com>2013-01-30 14:47:16 +0100
committerJaegeuk Kim <jaegeuk.kim@samsung.com>2013-02-11 23:15:01 +0100
commita2b52a598a4df55782462acba3bdced1a4610a60 (patch)
tree0cbc977f6959856f24ee6eae52ba528342be84b2 /fs/f2fs
parentf2fs: avoid redundant call to has_not_enough_free_secs in f2fs_gc (diff)
downloadlinux-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.c8
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)