diff options
author | Hua Zhong <hzhong@gmail.com> | 2006-06-25 08:49:32 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-06-25 13:01:26 -0400 |
commit | f58a1ebb22f128fccfeafb2b18557c87741cd627 (patch) | |
tree | 02fc9331087b97f69f50ab3bc78ae1e9910aa6ec | |
parent | d84a84775bba661d5a3fd06757bbb419381937f3 (diff) |
[PATCH] remove unlikely(sb) in prune_dcache
likely profiling shows that the following is a miss.
After boot:
[+- ] Type | # True | # False | Function:Filename@Line
+unlikely | 1074| 0 prune_dcache()@:fs/dcache.c@409
After a bonnie++ run:
+unlikely | 66716| 19584 prune_dcache()@:fs/dcache.c@409
So remove it.
Signed-off-by: Hua Zhong <hzhong@gmail.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | fs/dcache.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/dcache.c b/fs/dcache.c index 313b54b2b8f2..b85fda360533 100644 --- a/fs/dcache.c +++ b/fs/dcache.c | |||
@@ -406,7 +406,7 @@ static void prune_dcache(int count, struct super_block *sb) | |||
406 | cond_resched_lock(&dcache_lock); | 406 | cond_resched_lock(&dcache_lock); |
407 | 407 | ||
408 | tmp = dentry_unused.prev; | 408 | tmp = dentry_unused.prev; |
409 | if (unlikely(sb)) { | 409 | if (sb) { |
410 | /* Try to find a dentry for this sb, but don't try | 410 | /* Try to find a dentry for this sb, but don't try |
411 | * too hard, if they aren't near the tail they will | 411 | * too hard, if they aren't near the tail they will |
412 | * be moved down again soon | 412 | * be moved down again soon |