diff options
author | David S. Miller <davem@davemloft.net> | 2008-02-17 21:39:54 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-02-17 21:39:54 -0500 |
commit | 9ff566074689e3aed1488780b97714ec43ba361d (patch) | |
tree | 10bfc8ab214696ba08e5e25018d24c3de8018801 /net/core | |
parent | 93b2d4a208eeb19bff95e98c34503ce4b03cd0f9 (diff) |
Revert "[NDISC]: Fix race in generic address resolution"
This reverts commit 69cc64d8d92bf852f933e90c888dfff083bd4fc9.
It causes recursive locking in IPV6 because unlike other
neighbour layer clients, it even needs neighbour cache
entries to send neighbour soliciation messages :-(
We'll have to find another way to fix this race.
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core')
-rw-r--r-- | net/core/neighbour.c | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/net/core/neighbour.c b/net/core/neighbour.c index 7bb6a9a1256d..a16cf1ec5e5e 100644 --- a/net/core/neighbour.c +++ b/net/core/neighbour.c | |||
@@ -834,12 +834,18 @@ static void neigh_timer_handler(unsigned long arg) | |||
834 | } | 834 | } |
835 | if (neigh->nud_state & (NUD_INCOMPLETE | NUD_PROBE)) { | 835 | if (neigh->nud_state & (NUD_INCOMPLETE | NUD_PROBE)) { |
836 | struct sk_buff *skb = skb_peek(&neigh->arp_queue); | 836 | struct sk_buff *skb = skb_peek(&neigh->arp_queue); |
837 | 837 | /* keep skb alive even if arp_queue overflows */ | |
838 | if (skb) | ||
839 | skb_get(skb); | ||
840 | write_unlock(&neigh->lock); | ||
838 | neigh->ops->solicit(neigh, skb); | 841 | neigh->ops->solicit(neigh, skb); |
839 | atomic_inc(&neigh->probes); | 842 | atomic_inc(&neigh->probes); |
840 | } | 843 | if (skb) |
844 | kfree_skb(skb); | ||
845 | } else { | ||
841 | out: | 846 | out: |
842 | write_unlock(&neigh->lock); | 847 | write_unlock(&neigh->lock); |
848 | } | ||
843 | 849 | ||
844 | if (notify) | 850 | if (notify) |
845 | neigh_update_notify(neigh); | 851 | neigh_update_notify(neigh); |