Skip to content

Commit

Permalink
sky2: jumbo frame regression fix
Browse files Browse the repository at this point in the history
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 <[email protected]>
Signed-off-by: Jeff Garzik <[email protected]>
  • Loading branch information
Stephen Hemminger authored and Jeff Garzik committed Oct 3, 2007
1 parent 5c55c43 commit 529d303
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions drivers/net/sky2.c
Original file line number Diff line number Diff line change
Expand Up @@ -2163,9 +2163,6 @@ static struct sk_buff *sky2_receive(struct net_device *dev,
sky2->rx_next = (sky2->rx_next + 1) % sky2->rx_pending;
prefetch(sky2->rx_ring + sky2->rx_next);

if (length < ETH_ZLEN || length > sky2->rx_data_size)
goto len_error;

/* This chip has hardware problems that generates bogus status.
* So do only marginal checking and expect higher level protocols
* to handle crap frames.
Expand Down

0 comments on commit 529d303

Please sign in to comment.