diff options
| author | Haicheng Li <haicheng.li@linux.intel.com> | 2013-06-13 10:59:29 +0200 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2013-06-14 02:04:44 +0200 |
| commit | b25958b6ecf1dce087e62b9aa27cf8f2fe9b5c86 (patch) | |
| tree | a4213d64c10bd7994e8fefce1bf1f64cd81c9288 /fs/f2fs | |
| parent | f2fs: make locate_dirty_segment() as static (diff) | |
| download | linux-b25958b6ecf1dce087e62b9aa27cf8f2fe9b5c86.tar.xz linux-b25958b6ecf1dce087e62b9aa27cf8f2fe9b5c86.zip | |
f2fs: optimize do_write_data_page()
Since "need_inplace_update() == true" is a very rare case, using unlikely()
to give compiler a chance to optimize the code.
Signed-off-by: Haicheng Li <haicheng.li@linux.intel.com>
Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
Diffstat (limited to 'fs/f2fs')
| -rw-r--r-- | fs/f2fs/data.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c index 5b145fcc2864..6d4a743caf86 100644 --- a/fs/f2fs/data.c +++ b/fs/f2fs/data.c @@ -497,8 +497,9 @@ int do_write_data_page(struct page *page) * If current allocation needs SSR, * it had better in-place writes for updated data. */ - if (old_blk_addr != NEW_ADDR && !is_cold_data(page) && - need_inplace_update(inode)) { + if (unlikely(old_blk_addr != NEW_ADDR && + !is_cold_data(page) && + need_inplace_update(inode))) { rewrite_data_page(F2FS_SB(inode->i_sb), page, old_blk_addr); } else { |