sky2: fix receive length error in mixed non-VLAN/VLAN traffic
authorstephen hemminger <shemminger@vyatta.com>
Mon, 30 Apr 2012 06:47:37 +0000 (06:47 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 1 May 2012 17:39:13 +0000 (13:39 -0400)
commite072b3fad5f3915102c94628b4971f52ff99dd05
tree291a67ab2643d26bcea10919869fbe3cce05ed6f
parent3f42941b5d1d13542b1a755a9e4f633aa72e4d3e
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>
drivers/net/ethernet/marvell/sky2.c
drivers/net/ethernet/marvell/sky2.h