p71924506
/
linux_old1
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

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>
This commit is contained in:
Tiezhu Yang 2015-07-17 12:56:00 +08:00 committed by Jaegeuk Kim
parent 55f57d2c42
commit 737f18992e
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
fs/f2fs/data.c
View File

@ -1207,9 +1207,7 @@ static int f2fs_write_cache_pages(struct address_space *mapping,
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)) {