Skip to content

Commit

Permalink
neigh: Kill ndisc_ops->queue_xmit
Browse files Browse the repository at this point in the history
It is always dev_queue_xmit().

Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
davem330 committed Jul 17, 2011
1 parent b23b545 commit 542d4d6
Show file tree
Hide file tree
Showing 6 changed files with 5 additions and 17 deletions.
1 change: 0 additions & 1 deletion include/net/neighbour.h
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,6 @@ struct neigh_ops {
void (*error_report)(struct neighbour *, struct sk_buff*);
int (*output)(struct sk_buff*);
int (*connected_output)(struct sk_buff*);
int (*queue_xmit)(struct sk_buff*);
};

struct pneigh_entry {
Expand Down
1 change: 0 additions & 1 deletion net/atm/clip.c
Original file line number Diff line number Diff line change
Expand Up @@ -273,7 +273,6 @@ static const struct neigh_ops clip_neigh_ops = {
.error_report = clip_neigh_error,
.output = dev_queue_xmit,
.connected_output = dev_queue_xmit,
.queue_xmit = dev_queue_xmit,
};

static int clip_constructor(struct neighbour *neigh)
Expand Down
4 changes: 2 additions & 2 deletions net/core/neighbour.c
Original file line number Diff line number Diff line change
Expand Up @@ -1257,7 +1257,7 @@ int neigh_resolve_output(struct sk_buff *skb)
} while (read_seqretry(&neigh->ha_lock, seq));

if (err >= 0)
rc = neigh->ops->queue_xmit(skb);
rc = dev_queue_xmit(skb);
else
goto out_kfree_skb;
}
Expand Down Expand Up @@ -1292,7 +1292,7 @@ int neigh_connected_output(struct sk_buff *skb)
} while (read_seqretry(&neigh->ha_lock, seq));

if (err >= 0)
err = neigh->ops->queue_xmit(skb);
err = dev_queue_xmit(skb);
else {
err = -EINVAL;
kfree_skb(skb);
Expand Down
5 changes: 1 addition & 4 deletions net/decnet/dn_neigh.c
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,6 @@ static const struct neigh_ops dn_long_ops = {
.error_report = dn_long_error_report,
.output = dn_long_output,
.connected_output = dn_long_output,
.queue_xmit = dev_queue_xmit,
};

/*
Expand All @@ -75,7 +74,6 @@ static const struct neigh_ops dn_short_ops = {
.error_report = dn_short_error_report,
.output = dn_short_output,
.connected_output = dn_short_output,
.queue_xmit = dev_queue_xmit,
};

/*
Expand All @@ -86,7 +84,6 @@ static const struct neigh_ops dn_phase3_ops = {
.error_report = dn_short_error_report, /* Can use short version here */
.output = dn_phase3_output,
.connected_output = dn_phase3_output,
.queue_xmit = dev_queue_xmit
};

static u32 dn_neigh_hash(const void *pkey,
Expand Down Expand Up @@ -212,7 +209,7 @@ static int dn_neigh_output_packet(struct sk_buff *skb)
dn_dn2eth(mac_addr, rt->rt_local_src);
if (dev_hard_header(skb, dev, ntohs(skb->protocol), neigh->ha,
mac_addr, skb->len) >= 0)
return neigh->ops->queue_xmit(skb);
return dev_queue_xmit(skb);

if (net_ratelimit())
printk(KERN_DEBUG "dn_neigh_output_packet: oops, can't send packet\n");
Expand Down
6 changes: 1 addition & 5 deletions net/ipv4/arp.c
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,6 @@ static const struct neigh_ops arp_generic_ops = {
.error_report = arp_error_report,
.output = neigh_resolve_output,
.connected_output = neigh_connected_output,
.queue_xmit = dev_queue_xmit,
};

static const struct neigh_ops arp_hh_ops = {
Expand All @@ -147,14 +146,12 @@ static const struct neigh_ops arp_hh_ops = {
.error_report = arp_error_report,
.output = neigh_resolve_output,
.connected_output = neigh_resolve_output,
.queue_xmit = dev_queue_xmit,
};

static const struct neigh_ops arp_direct_ops = {
.family = AF_INET,
.output = dev_queue_xmit,
.connected_output = dev_queue_xmit,
.queue_xmit = dev_queue_xmit,
};

static const struct neigh_ops arp_broken_ops = {
Expand All @@ -163,7 +160,6 @@ static const struct neigh_ops arp_broken_ops = {
.error_report = arp_error_report,
.output = neigh_compat_output,
.connected_output = neigh_compat_output,
.queue_xmit = dev_queue_xmit,
};

struct neigh_table arp_tbl = {
Expand Down Expand Up @@ -254,7 +250,7 @@ static int arp_constructor(struct neighbour *neigh)
if (!dev->header_ops) {
neigh->nud_state = NUD_NOARP;
neigh->ops = &arp_direct_ops;
neigh->output = neigh->ops->queue_xmit;
neigh->output = dev_queue_xmit;
} else {
/* Good devices (checked by reading texts, but only Ethernet is
tested)
Expand Down
5 changes: 1 addition & 4 deletions net/ipv6/ndisc.c
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,6 @@ static const struct neigh_ops ndisc_generic_ops = {
.error_report = ndisc_error_report,
.output = neigh_resolve_output,
.connected_output = neigh_connected_output,
.queue_xmit = dev_queue_xmit,
};

static const struct neigh_ops ndisc_hh_ops = {
Expand All @@ -116,15 +115,13 @@ static const struct neigh_ops ndisc_hh_ops = {
.error_report = ndisc_error_report,
.output = neigh_resolve_output,
.connected_output = neigh_resolve_output,
.queue_xmit = dev_queue_xmit,
};


static const struct neigh_ops ndisc_direct_ops = {
.family = AF_INET6,
.output = dev_queue_xmit,
.connected_output = dev_queue_xmit,
.queue_xmit = dev_queue_xmit,
};

struct neigh_table nd_tbl = {
Expand Down Expand Up @@ -389,7 +386,7 @@ static int ndisc_constructor(struct neighbour *neigh)
if (!dev->header_ops) {
neigh->nud_state = NUD_NOARP;
neigh->ops = &ndisc_direct_ops;
neigh->output = neigh->ops->queue_xmit;
neigh->output = dev_queue_xmit;
} else {
if (is_multicast) {
neigh->nud_state = NUD_NOARP;
Expand Down

0 comments on commit 542d4d6

Please sign in to comment.