Commit 7161c76f authored by David S. Miller's avatar David S. Miller

atm: clip: Convert over to dst_neigh_lookup().

CLIP only support ipv4, and this is evidenced by the fact that it
is a device specific extension of arp_tbl, so this conversion is
pretty straightforward.
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 3329bdfc
...@@ -328,6 +328,8 @@ static netdev_tx_t clip_start_xmit(struct sk_buff *skb, ...@@ -328,6 +328,8 @@ static netdev_tx_t clip_start_xmit(struct sk_buff *skb,
struct atmarp_entry *entry; struct atmarp_entry *entry;
struct neighbour *n; struct neighbour *n;
struct atm_vcc *vcc; struct atm_vcc *vcc;
struct rtable *rt;
__be32 *daddr;
int old; int old;
unsigned long flags; unsigned long flags;
...@@ -338,7 +340,12 @@ static netdev_tx_t clip_start_xmit(struct sk_buff *skb, ...@@ -338,7 +340,12 @@ static netdev_tx_t clip_start_xmit(struct sk_buff *skb,
dev->stats.tx_dropped++; dev->stats.tx_dropped++;
return NETDEV_TX_OK; return NETDEV_TX_OK;
} }
n = dst_get_neighbour_noref(dst); rt = (struct rtable *) dst;
if (rt->rt_gateway)
daddr = &rt->rt_gateway;
else
daddr = &ip_hdr(skb)->daddr;
n = dst_neigh_lookup(dst, daddr);
if (!n) { if (!n) {
pr_err("NO NEIGHBOUR !\n"); pr_err("NO NEIGHBOUR !\n");
dev_kfree_skb(skb); dev_kfree_skb(skb);
...@@ -358,7 +365,7 @@ static netdev_tx_t clip_start_xmit(struct sk_buff *skb, ...@@ -358,7 +365,7 @@ static netdev_tx_t clip_start_xmit(struct sk_buff *skb,
dev_kfree_skb(skb); dev_kfree_skb(skb);
dev->stats.tx_dropped++; dev->stats.tx_dropped++;
} }
return NETDEV_TX_OK; goto out_release_neigh;
} }
pr_debug("neigh %p, vccs %p\n", entry, entry->vccs); pr_debug("neigh %p, vccs %p\n", entry, entry->vccs);
ATM_SKB(skb)->vcc = vcc = entry->vccs->vcc; ATM_SKB(skb)->vcc = vcc = entry->vccs->vcc;
...@@ -377,14 +384,14 @@ static netdev_tx_t clip_start_xmit(struct sk_buff *skb, ...@@ -377,14 +384,14 @@ static netdev_tx_t clip_start_xmit(struct sk_buff *skb,
old = xchg(&entry->vccs->xoff, 1); /* assume XOFF ... */ old = xchg(&entry->vccs->xoff, 1); /* assume XOFF ... */
if (old) { if (old) {
pr_warning("XOFF->XOFF transition\n"); pr_warning("XOFF->XOFF transition\n");
return NETDEV_TX_OK; goto out_release_neigh;
} }
dev->stats.tx_packets++; dev->stats.tx_packets++;
dev->stats.tx_bytes += skb->len; dev->stats.tx_bytes += skb->len;
vcc->send(vcc, skb); vcc->send(vcc, skb);
if (atm_may_send(vcc, 0)) { if (atm_may_send(vcc, 0)) {
entry->vccs->xoff = 0; entry->vccs->xoff = 0;
return NETDEV_TX_OK; goto out_release_neigh;
} }
spin_lock_irqsave(&clip_priv->xoff_lock, flags); spin_lock_irqsave(&clip_priv->xoff_lock, flags);
netif_stop_queue(dev); /* XOFF -> throttle immediately */ netif_stop_queue(dev); /* XOFF -> throttle immediately */
...@@ -396,6 +403,8 @@ static netdev_tx_t clip_start_xmit(struct sk_buff *skb, ...@@ -396,6 +403,8 @@ static netdev_tx_t clip_start_xmit(struct sk_buff *skb,
of the brief netif_stop_queue. If this isn't true or if it of the brief netif_stop_queue. If this isn't true or if it
changes, use netif_wake_queue instead. */ changes, use netif_wake_queue instead. */
spin_unlock_irqrestore(&clip_priv->xoff_lock, flags); spin_unlock_irqrestore(&clip_priv->xoff_lock, flags);
out_release_neigh:
neigh_release(n);
return NETDEV_TX_OK; return NETDEV_TX_OK;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment