summaryrefslogtreecommitdiffstats
path: root/fs/f2fs
diff options
context:
space:
mode:
authorNamjae Jeon <namjae.jeon@samsung.com>2013-01-30 22:47:16 +0900
committerJaegeuk Kim <jaegeuk.kim@samsung.com>2013-02-12 07:15:01 +0900
commita2b52a598a4df55782462acba3bdced1a4610a60 (patch)
tree0cbc977f6959856f24ee6eae52ba528342be84b2 /fs/f2fs
parent3786dfdf4f00755ce5797c0e87eac8f898acb52c (diff)
downloadop-kernel-dev-a2b52a598a4df55782462acba3bdced1a4610a60.zip
op-kernel-dev-a2b52a598a4df55782462acba3bdced1a4610a60.tar.gz
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 9bda63c..f71dfbb 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)
OpenPOWER on IntegriCloud