aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/net/ethernet/marvell/sky2.h
diff options
context:
space:
mode:
authorstephen hemminger <shemminger@vyatta.com>2012-04-30 06:47:37 +0000
committerDavid S. Miller <davem@davemloft.net>2012-05-01 13:39:13 -0400
commite072b3fad5f3915102c94628b4971f52ff99dd05 (patch)
tree291a67ab2643d26bcea10919869fbe3cce05ed6f /drivers/net/ethernet/marvell/sky2.h
parentsky2: propogate rx hash when packet is copied (diff)
downloadlinux-dev-e072b3fad5f3915102c94628b4971f52ff99dd05.tar.xz
linux-dev-e072b3fad5f3915102c94628b4971f52ff99dd05.zip
sky2: fix receive length error in mixed non-VLAN/VLAN traffic
Bug: The VLAN bit of the MAC RX Status Word is unreliable in several older supported chips. Sometimes the VLAN bit is not set for valid VLAN packets and also sometimes the VLAN bit is set for non-VLAN packets that came after a VLAN packet. This results in a receive length error when VLAN hardware tagging is enabled. Fix: Variation on original fix proposed by Mirko. The VLAN information is decoded in the status loop, and can be applied to the received SKB there. This eliminates the need for the separate tag field in the interface data structure. The tag has to be copied and cleared if packet is copied. This version checked out with vlan and normal traffic. Note: vlan_tx_tag_present should be renamed vlan_tag_present, but that is outside scope of this. Reported-by: Mirko Lindner <mlindner@marvell.com> Signed-off-by: Stephen Hemminger <shemminger@vyatta.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/marvell/sky2.h')
-rw-r--r--drivers/net/ethernet/marvell/sky2.h1
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/net/ethernet/marvell/sky2.h b/drivers/net/ethernet/marvell/sky2.h
index ff6f58bf822a..3c896ce80b71 100644
--- a/drivers/net/ethernet/marvell/sky2.h
+++ b/drivers/net/ethernet/marvell/sky2.h
@@ -2241,7 +2241,6 @@ struct sky2_port {
u16 rx_pending;
u16 rx_data_size;
u16 rx_nfrags;
- u16 rx_tag;
struct {
unsigned long last;