diff options
author | David Miller <davem@davemloft.net> | 2011-07-11 18:44:24 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-07-13 04:12:28 -0400 |
commit | 3769cffb1c48f64640ffab7ce3bffe867342c0f0 (patch) | |
tree | 8f6fcf2d822bf3a170a534e8540faa34fe9258ef /include/net | |
parent | e5b1de1f5ebe0200e988e195fefb6c7396de6e20 (diff) |
ipv4: Inline neigh binding.
Get rid of all of the useless and costly indirection
by doing the neigh hash table lookup directly inside
of the neighbour binding.
Rename from arp_bind_neighbour to rt_bind_neighbour.
Use new helpers {__,}ipv4_neigh_lookup()
In rt_bind_neighbour() get rid of useless tests which
are never true in the context this function is called,
namely dev is never NULL and the dst->neighbour is
always NULL.
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/net')
-rw-r--r-- | include/net/arp.h | 33 |
1 files changed, 32 insertions, 1 deletions
diff --git a/include/net/arp.h b/include/net/arp.h index 723bde501c64..5e669e6ffb42 100644 --- a/include/net/arp.h +++ b/include/net/arp.h | |||
@@ -15,6 +15,38 @@ static inline u32 arp_hashfn(u32 key, const struct net_device *dev, u32 hash_rnd | |||
15 | return val * hash_rnd; | 15 | return val * hash_rnd; |
16 | } | 16 | } |
17 | 17 | ||
18 | static inline struct neighbour *__ipv4_neigh_lookup(struct neigh_table *tbl, struct net_device *dev, u32 key) | ||
19 | { | ||
20 | struct neigh_hash_table *nht; | ||
21 | struct neighbour *n; | ||
22 | u32 hash_val; | ||
23 | |||
24 | rcu_read_lock_bh(); | ||
25 | nht = rcu_dereference_bh(tbl->nht); | ||
26 | hash_val = arp_hashfn(key, dev, nht->hash_rnd) >> (32 - nht->hash_shift); | ||
27 | for (n = rcu_dereference_bh(nht->hash_buckets[hash_val]); | ||
28 | n != NULL; | ||
29 | n = rcu_dereference_bh(n->next)) { | ||
30 | if (n->dev == dev && *(u32 *)n->primary_key == key) { | ||
31 | if (!atomic_inc_not_zero(&n->refcnt)) | ||
32 | n = NULL; | ||
33 | break; | ||
34 | } | ||
35 | } | ||
36 | rcu_read_unlock_bh(); | ||
37 | |||
38 | return n; | ||
39 | } | ||
40 | |||
41 | static inline struct neighbour *ipv4_neigh_lookup(struct neigh_table *tbl, struct net_device *dev, const __be32 *pkey) | ||
42 | { | ||
43 | struct neighbour *n = __ipv4_neigh_lookup(tbl, dev, | ||
44 | *(__force u32 *)pkey); | ||
45 | if (n) | ||
46 | return n; | ||
47 | return neigh_create(tbl, pkey, dev); | ||
48 | } | ||
49 | |||
18 | extern void arp_init(void); | 50 | extern void arp_init(void); |
19 | extern int arp_find(unsigned char *haddr, struct sk_buff *skb); | 51 | extern int arp_find(unsigned char *haddr, struct sk_buff *skb); |
20 | extern int arp_ioctl(struct net *net, unsigned int cmd, void __user *arg); | 52 | extern int arp_ioctl(struct net *net, unsigned int cmd, void __user *arg); |
@@ -22,7 +54,6 @@ extern void arp_send(int type, int ptype, __be32 dest_ip, | |||
22 | struct net_device *dev, __be32 src_ip, | 54 | struct net_device *dev, __be32 src_ip, |
23 | const unsigned char *dest_hw, | 55 | const unsigned char *dest_hw, |
24 | const unsigned char *src_hw, const unsigned char *th); | 56 | const unsigned char *src_hw, const unsigned char *th); |
25 | extern int arp_bind_neighbour(struct dst_entry *dst); | ||
26 | extern int arp_mc_map(__be32 addr, u8 *haddr, struct net_device *dev, int dir); | 57 | extern int arp_mc_map(__be32 addr, u8 *haddr, struct net_device *dev, int dir); |
27 | extern void arp_ifdown(struct net_device *dev); | 58 | extern void arp_ifdown(struct net_device *dev); |
28 | 59 | ||