diff options
author | Sam Ravnborg <sam@ravnborg.org> | 2006-03-21 01:37:04 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2006-03-21 01:37:04 -0500 |
commit | 05790c6456f144024e655710347b3df499260374 (patch) | |
tree | 389e85554c0ce7cf732998f2ac9dfeb441e85320 /net/tipc/bcast.c | |
parent | 1fc54d8f49c1270c584803437fb7c0ac543588c1 (diff) |
[TIPC]: Remove inlines from *.c
With reference to latest discussions on linux-kernel with respect to
inline here is a patch for tipc to remove all inlines as used in
the .c files. See also chapter 14 in Documentation/CodingStyle.
Before:
text data bss dec hex filename
102990 5292 1752 110034 1add2 tipc.o
Now:
text data bss dec hex filename
101190 5292 1752 108234 1a6ca tipc.o
This is a nice text size reduction which will improve icache usage.
In some cases bigger (> 4 lines) functions where declared inline
and used in many places, they are most probarly no longer inlined by gcc
resulting in the size reduction.
There are several one liners that no longer are declared inline, but gcc
should inline these just fine without the inline hint.
With this patch applied one warning is added about an unused static
function - that was hidded by utilising inline before.
The function in question were kept so this patch is solely a
inline removal patch.
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Signed-off-by: Per Liden <per.liden@ericsson.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/tipc/bcast.c')
-rw-r--r-- | net/tipc/bcast.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/net/tipc/bcast.c b/net/tipc/bcast.c index a926cec24119..64cca6479560 100644 --- a/net/tipc/bcast.c +++ b/net/tipc/bcast.c | |||
@@ -107,22 +107,22 @@ static spinlock_t bc_lock = SPIN_LOCK_UNLOCKED; | |||
107 | char tipc_bclink_name[] = "multicast-link"; | 107 | char tipc_bclink_name[] = "multicast-link"; |
108 | 108 | ||
109 | 109 | ||
110 | static inline u32 buf_seqno(struct sk_buff *buf) | 110 | static u32 buf_seqno(struct sk_buff *buf) |
111 | { | 111 | { |
112 | return msg_seqno(buf_msg(buf)); | 112 | return msg_seqno(buf_msg(buf)); |
113 | } | 113 | } |
114 | 114 | ||
115 | static inline u32 bcbuf_acks(struct sk_buff *buf) | 115 | static u32 bcbuf_acks(struct sk_buff *buf) |
116 | { | 116 | { |
117 | return (u32)(unsigned long)TIPC_SKB_CB(buf)->handle; | 117 | return (u32)(unsigned long)TIPC_SKB_CB(buf)->handle; |
118 | } | 118 | } |
119 | 119 | ||
120 | static inline void bcbuf_set_acks(struct sk_buff *buf, u32 acks) | 120 | static void bcbuf_set_acks(struct sk_buff *buf, u32 acks) |
121 | { | 121 | { |
122 | TIPC_SKB_CB(buf)->handle = (void *)(unsigned long)acks; | 122 | TIPC_SKB_CB(buf)->handle = (void *)(unsigned long)acks; |
123 | } | 123 | } |
124 | 124 | ||
125 | static inline void bcbuf_decr_acks(struct sk_buff *buf) | 125 | static void bcbuf_decr_acks(struct sk_buff *buf) |
126 | { | 126 | { |
127 | bcbuf_set_acks(buf, bcbuf_acks(buf) - 1); | 127 | bcbuf_set_acks(buf, bcbuf_acks(buf) - 1); |
128 | } | 128 | } |
@@ -134,7 +134,7 @@ static inline void bcbuf_decr_acks(struct sk_buff *buf) | |||
134 | * Called with 'node' locked, bc_lock unlocked | 134 | * Called with 'node' locked, bc_lock unlocked |
135 | */ | 135 | */ |
136 | 136 | ||
137 | static inline void bclink_set_gap(struct node *n_ptr) | 137 | static void bclink_set_gap(struct node *n_ptr) |
138 | { | 138 | { |
139 | struct sk_buff *buf = n_ptr->bclink.deferred_head; | 139 | struct sk_buff *buf = n_ptr->bclink.deferred_head; |
140 | 140 | ||
@@ -154,7 +154,7 @@ static inline void bclink_set_gap(struct node *n_ptr) | |||
154 | * distribute NACKs, but tries to use the same spacing (divide by 16). | 154 | * distribute NACKs, but tries to use the same spacing (divide by 16). |
155 | */ | 155 | */ |
156 | 156 | ||
157 | static inline int bclink_ack_allowed(u32 n) | 157 | static int bclink_ack_allowed(u32 n) |
158 | { | 158 | { |
159 | return((n % TIPC_MIN_LINK_WIN) == tipc_own_tag); | 159 | return((n % TIPC_MIN_LINK_WIN) == tipc_own_tag); |
160 | } | 160 | } |