diff options
author | Jesse Gross <jesse@nicira.com> | 2010-10-20 09:56:03 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-10-21 04:26:52 -0400 |
commit | eab6d18d20fc5b5ba04a7e7fcd6f357197870e51 (patch) | |
tree | 8a52538666747998dd16bc68a2102b05ebfe6873 /drivers/net/qlge | |
parent | b738127dfb469bb9f595cdace30e7f881e8146b2 (diff) |
vlan: Don't check for vlan group before vlan_tx_tag_present.
Many (but not all) drivers check to see whether there is a vlan
group configured before using a tag stored in the skb. There's
not much point in this check since it just throws away data that
should only be present in the expected circumstances. However,
it will soon be legal and expected to get a vlan tag when no
vlan group is configured, so remove this check from all drivers
to avoid dropping the tags.
Signed-off-by: Jesse Gross <jesse@nicira.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/qlge')
-rw-r--r-- | drivers/net/qlge/qlge_main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/qlge/qlge_main.c b/drivers/net/qlge/qlge_main.c index 4ffebe83d883..ba0053d8515e 100644 --- a/drivers/net/qlge/qlge_main.c +++ b/drivers/net/qlge/qlge_main.c | |||
@@ -2572,7 +2572,7 @@ static netdev_tx_t qlge_send(struct sk_buff *skb, struct net_device *ndev) | |||
2572 | 2572 | ||
2573 | mac_iocb_ptr->frame_len = cpu_to_le16((u16) skb->len); | 2573 | mac_iocb_ptr->frame_len = cpu_to_le16((u16) skb->len); |
2574 | 2574 | ||
2575 | if (qdev->vlgrp && vlan_tx_tag_present(skb)) { | 2575 | if (vlan_tx_tag_present(skb)) { |
2576 | netif_printk(qdev, tx_queued, KERN_DEBUG, qdev->ndev, | 2576 | netif_printk(qdev, tx_queued, KERN_DEBUG, qdev->ndev, |
2577 | "Adding a vlan tag %d.\n", vlan_tx_tag_get(skb)); | 2577 | "Adding a vlan tag %d.\n", vlan_tx_tag_get(skb)); |
2578 | mac_iocb_ptr->flags3 |= OB_MAC_IOCB_V; | 2578 | mac_iocb_ptr->flags3 |= OB_MAC_IOCB_V; |