summaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorTiezhu Yang <kernelpatch@126.com>2015-07-17 12:56:00 +0800
committerJaegeuk Kim <jaegeuk@kernel.org>2015-08-05 08:08:10 -0700
commit737f18992ee81cab897336e84c5c7f4e179dfd61 (patch)
tree161ed9461b87bf7674218a6f92c37de24b87dd90 /fs
parent55f57d2c4259a9a4048cf4629a2c6ba53729188d (diff)
downloadlinux-737f18992ee81cab897336e84c5c7f4e179dfd61.tar.gz
linux-737f18992ee81cab897336e84c5c7f4e179dfd61.tar.bz2
linux-737f18992ee81cab897336e84c5c7f4e179dfd61.zip
f2fs: optimize f2fs_write_cache_pages
The if statement "goto continue_unlock" is exactly the same when each if condition is true that is depended on the value of both "step" and "is_cold_data(page)" are 0 or 1. That means when the value of "step" equals to "is_cold_data(page)", the if condition is true and the if statement "goto continue_unlock" appears only once, so it can be optimized to reduce the duplicated code. Signed-off-by: Tiezhu Yang <kernelpatch@126.com> Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'fs')
-rw-r--r--fs/f2fs/data.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
index 7f51296fbbf6..801b0b0b08f4 100644
--- a/fs/f2fs/data.c
+++ b/fs/f2fs/data.c
@@ -1207,9 +1207,7 @@ continue_unlock:
goto continue_unlock;
}
- if (step == 0 && !is_cold_data(page))
- goto continue_unlock;
- if (step == 1 && is_cold_data(page))
+ if (step == is_cold_data(page))
goto continue_unlock;
if (PageWriteback(page)) {