diff options
author | Adrian Bunk <bunk@stusta.de> | 2006-12-12 11:24:39 -0500 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2007-02-05 16:58:47 -0500 |
commit | 079ca7da1e6d05c7cb82e9c4f2e1d98839332664 (patch) | |
tree | 849faa0b78df712cbfa62d2454843cb0add04c3d /drivers/net/bonding | |
parent | bd36b0ac5d06378c95b5149b6df5f413a6c985a5 (diff) |
bonding.h: "extern inline" -> "static inline"
"extern inline" generates a warning with -Wmissing-prototypes and I'm
currently working on getting the kernel cleaned up for adding this to
the CFLAGS since it will help us to avoid a nasty class of runtime
errors.
If there are places that really need a forced inline, __always_inline
would be the correct solution.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/net/bonding')
-rw-r--r-- | drivers/net/bonding/bonding.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/net/bonding/bonding.h b/drivers/net/bonding/bonding.h index 0978c9ac6d2..731cfb63b66 100644 --- a/drivers/net/bonding/bonding.h +++ b/drivers/net/bonding/bonding.h | |||
@@ -237,12 +237,13 @@ static inline struct bonding *bond_get_bond_by_slave(struct slave *slave) | |||
237 | #define BOND_ARP_VALIDATE_ALL (BOND_ARP_VALIDATE_ACTIVE | \ | 237 | #define BOND_ARP_VALIDATE_ALL (BOND_ARP_VALIDATE_ACTIVE | \ |
238 | BOND_ARP_VALIDATE_BACKUP) | 238 | BOND_ARP_VALIDATE_BACKUP) |
239 | 239 | ||
240 | extern inline int slave_do_arp_validate(struct bonding *bond, struct slave *slave) | 240 | static inline int slave_do_arp_validate(struct bonding *bond, |
241 | struct slave *slave) | ||
241 | { | 242 | { |
242 | return bond->params.arp_validate & (1 << slave->state); | 243 | return bond->params.arp_validate & (1 << slave->state); |
243 | } | 244 | } |
244 | 245 | ||
245 | extern inline unsigned long slave_last_rx(struct bonding *bond, | 246 | static inline unsigned long slave_last_rx(struct bonding *bond, |
246 | struct slave *slave) | 247 | struct slave *slave) |
247 | { | 248 | { |
248 | if (slave_do_arp_validate(bond, slave)) | 249 | if (slave_do_arp_validate(bond, slave)) |