]> 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)
commitc27927e31fbb4683d3811622fc3098fa7cc816fb
tree291a67ab2643d26bcea10919869fbe3cce05ed6f
parent2bd87ed3e4503fe3c419cab064cb2ad5a4fddcf6
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