]> git.baikalelectronics.ru Git - kernel.git/commit
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)
commit3fef7af11e202a426efa9929c36e72f766fbfb85
tree291a67ab2643d26bcea10919869fbe3cce05ed6f
parent7af280c4808af891a905ee37cf9554f8ff4d5704
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