diff options
author | Wang Shilong <wangsl-fnst@cn.fujitsu.com> | 2013-01-12 16:28:47 -0500 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2013-01-12 16:28:47 -0500 |
commit | aebf02430d25b6bd2b8542126fdcdb90e75a24b8 (patch) | |
tree | 7f884e64b000d9bed3a59dd557b3075a2da4405d /fs/ext4/extents.c | |
parent | 860d21e2c585f7ee8a4ecc06f474fdc33c9474f4 (diff) |
ext4: use unlikely to improve the efficiency of the kernel
Because the function 'sb_getblk' seldomly fails to return NULL
value,it will be better to use 'unlikely' to optimize it.
Signed-off-by: Wang Shilong <wangsl-fnst@cn.fujitsu.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs/ext4/extents.c')
-rw-r--r-- | fs/ext4/extents.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c index d42a8c49ad69..391e53a52e58 100644 --- a/fs/ext4/extents.c +++ b/fs/ext4/extents.c | |||
@@ -956,7 +956,7 @@ static int ext4_ext_split(handle_t *handle, struct inode *inode, | |||
956 | goto cleanup; | 956 | goto cleanup; |
957 | } | 957 | } |
958 | bh = sb_getblk(inode->i_sb, newblock); | 958 | bh = sb_getblk(inode->i_sb, newblock); |
959 | if (!bh) { | 959 | if (unlikely(!bh)) { |
960 | err = -ENOMEM; | 960 | err = -ENOMEM; |
961 | goto cleanup; | 961 | goto cleanup; |
962 | } | 962 | } |
@@ -1029,7 +1029,7 @@ static int ext4_ext_split(handle_t *handle, struct inode *inode, | |||
1029 | oldblock = newblock; | 1029 | oldblock = newblock; |
1030 | newblock = ablocks[--a]; | 1030 | newblock = ablocks[--a]; |
1031 | bh = sb_getblk(inode->i_sb, newblock); | 1031 | bh = sb_getblk(inode->i_sb, newblock); |
1032 | if (!bh) { | 1032 | if (unlikely(!bh)) { |
1033 | err = -ENOMEM; | 1033 | err = -ENOMEM; |
1034 | goto cleanup; | 1034 | goto cleanup; |
1035 | } | 1035 | } |
@@ -1142,7 +1142,7 @@ static int ext4_ext_grow_indepth(handle_t *handle, struct inode *inode, | |||
1142 | return err; | 1142 | return err; |
1143 | 1143 | ||
1144 | bh = sb_getblk(inode->i_sb, newblock); | 1144 | bh = sb_getblk(inode->i_sb, newblock); |
1145 | if (!bh) | 1145 | if (unlikely(!bh)) |
1146 | return -ENOMEM; | 1146 | return -ENOMEM; |
1147 | lock_buffer(bh); | 1147 | lock_buffer(bh); |
1148 | 1148 | ||