diff options
author | Namjae Jeon <namjae.jeon@samsung.com> | 2013-02-02 14:53:15 (GMT) |
---|---|---|
committer | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2013-02-11 22:15:02 (GMT) |
commit | b1f1daf8c72d615b64163e26488d8effeed29b60 (patch) | |
tree | d623c3ed0808ea9c1f4bec9b388844aed6a30a1e | |
parent | 5ac206cf4f9aad871aa1f1dd653a3404092db56c (diff) | |
download | linux-b1f1daf8c72d615b64163e26488d8effeed29b60.tar.xz |
f2fs: optimize the return condition for has_not_enough_free_secs
Instead of evaluating the free_sections and then deciding to return
true/false from that path. We can directly use the evaluation condition
for returning proper value.
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>
-rw-r--r-- | fs/f2fs/segment.h | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/f2fs/segment.h b/fs/f2fs/segment.h index a9417b9..458bf5c 100644 --- a/fs/f2fs/segment.h +++ b/fs/f2fs/segment.h @@ -464,10 +464,8 @@ static inline bool has_not_enough_free_secs(struct f2fs_sb_info *sbi) if (sbi->por_doing) return false; - if (free_sections(sbi) <= (node_secs + 2 * dent_secs + - reserved_sections(sbi))) - return true; - return false; + return (free_sections(sbi) <= (node_secs + 2 * dent_secs + + reserved_sections(sbi))); } static inline int utilization(struct f2fs_sb_info *sbi) |