diff options
author | Namjae Jeon <namjae.jeon@samsung.com> | 2013-01-30 08:47:16 -0500 |
---|---|---|
committer | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2013-02-11 17:15:01 -0500 |
commit | a2b52a598a4df55782462acba3bdced1a4610a60 (patch) | |
tree | 0cbc977f6959856f24ee6eae52ba528342be84b2 /fs/f2fs/node.c | |
parent | 3786dfdf4f00755ce5797c0e87eac8f898acb52c (diff) |
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/node.c')
-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) | |||
874 | return; | 874 | return; |
875 | 875 | ||
876 | if (read_node_page(apage, READA)) | 876 | if (read_node_page(apage, READA)) |
877 | goto unlock_out; | 877 | unlock_page(apage); |
878 | 878 | ||
879 | page_cache_release(apage); | ||
880 | return; | ||
881 | |||
882 | unlock_out: | ||
883 | unlock_page(apage); | ||
884 | release_out: | 879 | release_out: |
885 | page_cache_release(apage); | 880 | page_cache_release(apage); |
881 | return; | ||
886 | } | 882 | } |
887 | 883 | ||
888 | struct page *get_node_page(struct f2fs_sb_info *sbi, pgoff_t nid) | 884 | struct page *get_node_page(struct f2fs_sb_info *sbi, pgoff_t nid) |