netdev - [PATCH] at91_ether: late collistions are counted twice

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 8 May 2008 17:39:00 +0200
From:	Marc Pignat <marc.pignat@...s.ch>
To:	linux-arm-kernel@...ts.arm.linux.org.uk
Cc:	netdev@...r.kernel.org, linux@...im.org.za
Subject: [PATCH] at91_ether: late collistions are counted twice


According to the at91rm9200 datasheet, table 36-6, late collisions
(AT91_EMAC_LCOL) are already counted in the collision register (in scol, mcol or
ecol , depending on the type of error).

So don't count them twice in stats.collisions.

Signed-off-by: Marc Pignat <marc.pignat@...s.ch>
---

Hello!

After reviewing the "at91_ether: use netstats in net_device structure" patch, I
had a look the the datasheet and found this small error.

This is really a small error (in statistics) and I will never have collisions,
I'm connected to a switch, but I've no reason not fixing it :)

Best regards



Marc

patch against v2.6.26-rc1 + "[PATCH v2] at91_ether: use netstats in net_device structure"

--- a/drivers/net/arm/at91_ether.c
+++ b/drivers/net/arm/at91_ether.c
@@ -843,7 +843,7 @@ static int at91ether_tx(struct sk_buff *skb, struct net_device *dev)
  */
 static struct net_device_stats *at91ether_stats(struct net_device *dev)
 {
-	int ale, lenerr, seqe, lcol, ecol;
+	int ale, lenerr, seqe, ecol;
 
 	if (netif_running(dev)) {
 		dev->stats.rx_packets += at91_emac_read(AT91_EMAC_OK);		/* Good frames received */
@@ -862,12 +862,11 @@ static struct net_device_stats *at91ether_stats(struct net_device *dev)
 		dev->stats.tx_carrier_errors += at91_emac_read(AT91_EMAC_CSE);	/* Carrier Sense errors */
 		dev->stats.tx_heartbeat_errors += at91_emac_read(AT91_EMAC_SQEE);/* Heartbeat error */
 
-		lcol = at91_emac_read(AT91_EMAC_LCOL);
+		dev->stats.tx_window_errors += at91_emac_read(AT91_EMAC_LCOL);	/* Late collisions */
 		ecol = at91_emac_read(AT91_EMAC_ECOL);
-		dev->stats.tx_window_errors += lcol;			/* Late collisions */
 		dev->stats.tx_aborted_errors += ecol;			/* 16 collisions */
 
-		dev->stats.collisions += (at91_emac_read(AT91_EMAC_SCOL) + at91_emac_read(AT91_EMAC_MCOL) + lcol + ecol);
+		dev->stats.collisions += (at91_emac_read(AT91_EMAC_SCOL) + at91_emac_read(AT91_EMAC_MCOL) + ecol);
 	}
 	return &dev->stats;
 }
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html