diff options
author | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2013-12-06 01:00:58 -0500 |
---|---|---|
committer | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2013-12-22 20:18:06 -0500 |
commit | 6bacf52fb58aeb3e89d9a62970b85a5570aa8ace (patch) | |
tree | 422eb46c144b7ffbae193570a001de2eed490e09 /fs/f2fs/xattr.c | |
parent | cfb271d485d0ec31eb92b51f4fbe54bf6542e8e6 (diff) |
f2fs: add unlikely() macro for compiler more aggressively
This patch adds unlikely() macro into the most of codes.
The basic rule is to add that when:
- checking unusual errors,
- checking page mappings,
- and the other unlikely conditions.
Change log from v1:
- Don't add unlikely for the NULL test and error test: advised by Andi Kleen.
Cc: Chao Yu <chao2.yu@samsung.com>
Cc: Andi Kleen <andi@firstfloor.org>
Reviewed-by: Chao Yu <chao2.yu@samsung.com>
Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
Diffstat (limited to 'fs/f2fs/xattr.c')
-rw-r--r-- | fs/f2fs/xattr.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/f2fs/xattr.c b/fs/f2fs/xattr.c index aa7a3f139fe5..b0fb8a27f3da 100644 --- a/fs/f2fs/xattr.c +++ b/fs/f2fs/xattr.c | |||
@@ -522,7 +522,7 @@ static int __f2fs_setxattr(struct inode *inode, int name_index, | |||
522 | if (found) | 522 | if (found) |
523 | free = free + ENTRY_SIZE(here); | 523 | free = free + ENTRY_SIZE(here); |
524 | 524 | ||
525 | if (free < newsize) { | 525 | if (unlikely(free < newsize)) { |
526 | error = -ENOSPC; | 526 | error = -ENOSPC; |
527 | goto exit; | 527 | goto exit; |
528 | } | 528 | } |