Thread (15 messages) flat view 15 messages, 6 authors, 2010-08-19
STALE5878d

[PATCH v2] ether1: Use net_device_stats from struct net_device

From: davem@davemloft.net (David Miller)
Date: 2010-08-18 20:10:43
Also in: kernel-janitors, netdev

From: Stephen Hemminger <redacted>
Date: Wed, 18 Aug 2010 08:33:04 -0700
On Wed, 18 Aug 2010 09:04:24 +0200
Tobias Klauser [off-list ref] wrote:
quoted
 
-	memset (&priv(dev)->stats, 0, sizeof (struct net_device_stats));
+	memset(&dev->stats, 0, sizeof(struct net_device_stats));
This is incorrect, just remove the memset.  The stats are initialized
when device is created.  The Linux device driver convention is to
keep stats when device is set down and brought back up; that is what
the majority of other drivers do.
Yep, both the ether1 and ether3 patch have this problem.  Looks
like we'll see v4 coming some :-)
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help