diff options
author | Stephen Hemminger <shemminger@linux-foundation.org> | 2007-10-02 21:02:09 -0400 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2007-10-03 13:39:42 -0400 |
commit | 529d303e075aa6d988f30935b8995ffb382ad38e (patch) | |
tree | 7cf9ce2b7901037cf12bea651b14135c1143656f | |
parent | 5c55c434917429f229a1bf43def97fd421f444c6 (diff) |
sky2: jumbo frame regression fix
Remove unneeded check that caused problems with jumbo frame sizes.
The check was recently added and is wrong.
When using jumbo frames the sky2 driver does fragmentation, so
rx_data_size is less than mtu.
Signed-off-by: Stephen Hemminger <shemminger@linux-foundation.org>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
-rw-r--r-- | drivers/net/sky2.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/net/sky2.c b/drivers/net/sky2.c index 162489b9f599..ea117fc3d5e3 100644 --- a/drivers/net/sky2.c +++ b/drivers/net/sky2.c | |||
@@ -2163,9 +2163,6 @@ static struct sk_buff *sky2_receive(struct net_device *dev, | |||
2163 | sky2->rx_next = (sky2->rx_next + 1) % sky2->rx_pending; | 2163 | sky2->rx_next = (sky2->rx_next + 1) % sky2->rx_pending; |
2164 | prefetch(sky2->rx_ring + sky2->rx_next); | 2164 | prefetch(sky2->rx_ring + sky2->rx_next); |
2165 | 2165 | ||
2166 | if (length < ETH_ZLEN || length > sky2->rx_data_size) | ||
2167 | goto len_error; | ||
2168 | |||
2169 | /* This chip has hardware problems that generates bogus status. | 2166 | /* This chip has hardware problems that generates bogus status. |
2170 | * So do only marginal checking and expect higher level protocols | 2167 | * So do only marginal checking and expect higher level protocols |
2171 | * to handle crap frames. | 2168 | * to handle crap frames. |