Skip to content

Commit

Permalink
ipv6: Kill ndisc_get_neigh() inline helper.
Browse files Browse the repository at this point in the history
It's only used in net/ipv6/route.c and the NULL device check is
superfluous for all of the existing call sites.

Just expand the __ndisc_lookup_errno() call at each location.

Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
David S. Miller committed Dec 3, 2011
1 parent 3830847 commit 04a6f44
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 12 deletions.
9 changes: 0 additions & 9 deletions include/net/ndisc.h
Original file line number Diff line number Diff line change
Expand Up @@ -145,13 +145,4 @@ int ndisc_ifinfo_sysctl_strategy(ctl_table *ctl,
extern void inet6_ifinfo_notify(int event,
struct inet6_dev *idev);

static inline struct neighbour * ndisc_get_neigh(struct net_device *dev, const struct in6_addr *addr)
{

if (dev)
return __neigh_lookup_errno(&nd_tbl, addr, dev);

return ERR_PTR(-ENODEV);
}

#endif
7 changes: 4 additions & 3 deletions net/ipv6/route.c
Original file line number Diff line number Diff line change
Expand Up @@ -744,7 +744,8 @@ static struct rt6_info *rt6_alloc_cow(const struct rt6_info *ort,
#endif

retry:
neigh = ndisc_get_neigh(rt->rt6i_dev, &rt->rt6i_gateway);
neigh = __neigh_lookup_errno(&nd_tbl, &rt->rt6i_gateway,
rt->rt6i_dev);
if (IS_ERR(neigh)) {
struct net *net = dev_net(rt->rt6i_dev);
int saved_rt_min_interval =
Expand Down Expand Up @@ -1085,7 +1086,7 @@ struct dst_entry *icmp6_dst_alloc(struct net_device *dev,
if (neigh)
neigh_hold(neigh);
else {
neigh = ndisc_get_neigh(dev, addr);
neigh = __neigh_lookup_errno(&nd_tbl, addr, dev);
if (IS_ERR(neigh))
neigh = NULL;
}
Expand Down Expand Up @@ -2082,7 +2083,7 @@ struct rt6_info *addrconf_dst_alloc(struct inet6_dev *idev,
rt->rt6i_flags |= RTF_ANYCAST;
else
rt->rt6i_flags |= RTF_LOCAL;
neigh = ndisc_get_neigh(rt->rt6i_dev, &rt->rt6i_gateway);
neigh = __neigh_lookup_errno(&nd_tbl, &rt->rt6i_gateway, rt->rt6i_dev);
if (IS_ERR(neigh)) {
dst_free(&rt->dst);

Expand Down

0 comments on commit 04a6f44

Please sign in to comment.