On Tue, Aug 17, 2010 at 06:15:33PM +0200, Tobias Klauser wrote: > struct net_device has its own struct net_device_stats member, so use > this one instead of a private copy in the ether1_priv struct. > Two were missed, one from ether3_init_for_open() memset(&priv(dev)->stats, 0, sizeof(struct net_device_stats)); and another from ether3_rx(). struct net_device_stats *stats = &priv(dev)->stats; regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html