diff options
author | Andreas Mohr <andi@rhlx01.fht-esslingen.de> | 2006-09-29 05:01:20 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-09-29 12:18:22 -0400 |
commit | fdf8cb0909b531f9ae8f9b9d7e4eb35ba3505f07 (patch) | |
tree | 0e06d558f1e349fcc26a9fb06b2c5f96776b6b86 | |
parent | 2752e401148a064a23a5a1e92d502d05017a560c (diff) |
[PATCH] lib/rwsem.c: un-inline rwsem_down_failed_common()
Un-inlining rwsem_down_failed_common() (two callsites) reduced lib/rwsem.o
on my Athlon, gcc 4.1.2 from 5935 to 5480 Bytes (455 Bytes saved).
I thus guess that reduced icache footprint (and better function caching) is
worth more than any function call overhead.
Signed-off-by: Andreas Mohr <andi@lisas.de>
Cc: David Howells <dhowells@redhat.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | lib/rwsem.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/rwsem.c b/lib/rwsem.c index b322421c2969..901d0e7da892 100644 --- a/lib/rwsem.c +++ b/lib/rwsem.c | |||
@@ -146,7 +146,7 @@ __rwsem_do_wake(struct rw_semaphore *sem, int downgrading) | |||
146 | /* | 146 | /* |
147 | * wait for a lock to be granted | 147 | * wait for a lock to be granted |
148 | */ | 148 | */ |
149 | static inline struct rw_semaphore * | 149 | static struct rw_semaphore * |
150 | rwsem_down_failed_common(struct rw_semaphore *sem, | 150 | rwsem_down_failed_common(struct rw_semaphore *sem, |
151 | struct rwsem_waiter *waiter, signed long adjustment) | 151 | struct rwsem_waiter *waiter, signed long adjustment) |
152 | { | 152 | { |