Subject: [PATCH 5/6] sky2: receive error handling improvements

From: Stephen Hemminger <[email-masked]>
Date: Thu, 15 Feb 2007 16:40:34 -0800

Don't drop oversize frame it might be a VLAN (untagged).
Use different counter for fifo overrun vs fifo error.
Print error on fifo overrrun.

Signed-off-by: Stephen Hemminger <[email-masked]>

--- sky2-dev.orig/drivers/net/sky2.c	2007-02-15 14:33:52.000000000 -0800
+++ sky2-dev/drivers/net/sky2.c	2007-02-15 15:00:38.000000000 -0800
@@ -2056,9 +2056,6 @@
 	if (!(status & GMR_FS_RX_OK))
 		goto resubmit;
 
-	if (length > dev->mtu + ETH_HLEN)
-		goto oversize;
-
 	if (length < copybreak)
 		skb = receive_copy(sky2, re, length);
 	else
@@ -2068,14 +2065,10 @@
 
 	return skb;
 
-oversize:
-	++sky2->net_stats.rx_over_errors;
-	goto resubmit;
-
 error:
 	++sky2->net_stats.rx_errors;
 	if (status & GMR_FS_RX_FF_OV) {
-		sky2->net_stats.rx_fifo_errors++;
+		sky2->net_stats.rx_over_errors++;
 		goto resubmit;
 	}

--
Stephen Hemminger <[email-masked]>