diff options
| author | Eric Dumazet <edumazet@google.com> | 2019-12-07 23:23:21 +0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-12-31 18:41:37 +0300 |
| commit | a073350c5f3b0b0b3c58e42ccd886fd6450575a9 (patch) | |
| tree | 2698103af8296dbdf75da28d82bf992d715a97ee /net | |
| parent | 37a56610e96d71d976d989ed70a5a4fc9d07f66a (diff) | |
| download | linux-a073350c5f3b0b0b3c58e42ccd886fd6450575a9.tar.xz | |
neighbour: remove neigh_cleanup() method
[ Upstream commit f394722fb0d0f701119368959d7cd0ecbc46363a ]
neigh_cleanup() has not been used for seven years, and was a wrong design.
Messing with shared pointer in bond_neigh_init() without proper
memory barriers would at least trigger syzbot complains eventually.
It is time to remove this stuff.
Fixes: b63b70d87741 ("IPoIB: Use a private hash table for path lookup in xmit path")
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'net')
| -rw-r--r-- | net/core/neighbour.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/net/core/neighbour.c b/net/core/neighbour.c index 5480edff0c86..08ebc3ac5343 100644 --- a/net/core/neighbour.c +++ b/net/core/neighbour.c @@ -98,9 +98,6 @@ static int neigh_blackhole(struct neighbour *neigh, struct sk_buff *skb) static void neigh_cleanup_and_release(struct neighbour *neigh) { - if (neigh->parms->neigh_cleanup) - neigh->parms->neigh_cleanup(neigh); - trace_neigh_cleanup_and_release(neigh, 0); __neigh_notify(neigh, RTM_DELNEIGH, 0, 0); call_netevent_notifiers(NETEVENT_NEIGH_UPDATE, neigh); |