net: Pass neighbours and dest address into NETEVENT_REDIRECT events.
authorDavid S. Miller <davem@davemloft.net>
Tue, 3 Jul 2012 08:01:51 +0000 (01:01 -0700)
committerDavid S. Miller <davem@davemloft.net>
Thu, 5 Jul 2012 09:21:55 +0000 (02:21 -0700)
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/chelsio/cxgb3/cxgb3_offload.c
include/net/netevent.h
net/ipv6/route.c

index 55cf72af69ce3311fbcdb584ce66a9d94656eb40..633c6029e53c05f61454fbe9a5bd300b435f26f2 100644 (file)
@@ -62,7 +62,8 @@ static const unsigned int MAX_ATIDS = 64 * 1024;
 static const unsigned int ATID_BASE = 0x10000;
 
 static void cxgb_neigh_update(struct neighbour *neigh);
-static void cxgb_redirect(struct dst_entry *old, struct dst_entry *new);
+static void cxgb_redirect(struct dst_entry *old, struct neighbour *old_neigh,
+                         struct dst_entry *new, struct neighbour *new_neigh);
 
 static inline int offload_activated(struct t3cdev *tdev)
 {
@@ -968,8 +969,9 @@ static int nb_callback(struct notifier_block *self, unsigned long event,
        }
        case (NETEVENT_REDIRECT):{
                struct netevent_redirect *nr = ctx;
-               cxgb_redirect(nr->old, nr->new);
-               cxgb_neigh_update(dst_get_neighbour_noref(nr->new));
+               cxgb_redirect(nr->old, nr->old_neigh,
+                             nr->new, nr->new_neigh);
+               cxgb_neigh_update(nr->new_neigh);
                break;
        }
        default:
@@ -1107,10 +1109,10 @@ static void set_l2t_ix(struct t3cdev *tdev, u32 tid, struct l2t_entry *e)
        tdev->send(tdev, skb);
 }
 
-static void cxgb_redirect(struct dst_entry *old, struct dst_entry *new)
+static void cxgb_redirect(struct dst_entry *old, struct neighbour *old_neigh,
+                         struct dst_entry *new, struct neighbour *new_neigh)
 {
        struct net_device *olddev, *newdev;
-       struct neighbour *n;
        struct tid_info *ti;
        struct t3cdev *tdev;
        u32 tid;
@@ -1118,15 +1120,8 @@ static void cxgb_redirect(struct dst_entry *old, struct dst_entry *new)
        struct l2t_entry *e;
        struct t3c_tid_entry *te;
 
-       n = dst_get_neighbour_noref(old);
-       if (!n)
-               return;
-       olddev = n->dev;
-
-       n = dst_get_neighbour_noref(new);
-       if (!n)
-               return;
-       newdev = n->dev;
+       olddev = old_neigh->dev;
+       newdev = new_neigh->dev;
 
        if (!is_offloading(olddev))
                return;
index 086f8a5b59dc3782014f40edbb10be4d25f51d0a..3ce4988c9c08a96c0837407bd7c092c043a0309b 100644 (file)
  */
 
 struct dst_entry;
+struct neighbour;
 
 struct netevent_redirect {
        struct dst_entry *old;
+       struct neighbour *old_neigh;
        struct dst_entry *new;
+       struct neighbour *new_neigh;
+       const void *daddr;
 };
 
 enum netevent_notif_type {
index 4b581c675bb2ba1f162313f0e4d717fdeff4d2b9..34b29881e22dfff27ccbb3308bd83ec0c26dc203 100644 (file)
@@ -1687,6 +1687,7 @@ void rt6_redirect(const struct in6_addr *dest, const struct in6_addr *src,
        struct rt6_info *rt, *nrt = NULL;
        struct netevent_redirect netevent;
        struct net *net = dev_net(neigh->dev);
+       struct neighbour *old_neigh;
 
        rt = ip6_route_redirect(dest, src, saddr, neigh->dev);
 
@@ -1714,7 +1715,8 @@ void rt6_redirect(const struct in6_addr *dest, const struct in6_addr *src,
        dst_confirm(&rt->dst);
 
        /* Duplicate redirect: silently ignore. */
-       if (neigh == dst_get_neighbour_noref_raw(&rt->dst))
+       old_neigh = dst_get_neighbour_noref_raw(&rt->dst);
+       if (neigh == old_neigh)
                goto out;
 
        nrt = ip6_rt_copy(rt, dest);
@@ -1732,7 +1734,10 @@ void rt6_redirect(const struct in6_addr *dest, const struct in6_addr *src,
                goto out;
 
        netevent.old = &rt->dst;
+       netevent.old_neigh = old_neigh;
        netevent.new = &nrt->dst;
+       netevent.new_neigh = neigh;
+       netevent.daddr = dest;
        call_netevent_notifiers(NETEVENT_REDIRECT, &netevent);
 
        if (rt->rt6i_flags & RTF_CACHE) {