rx-over-length stats problem with e100

While testing my rx-all patches, I triggered a code branch that caused the rx_over_length_errors counter to increment in the e100_rx_indicate path (it can already increment there w/out my patch, it appears).

The problem is that then the stats go weird because the code in e100_update_stats adds the total rx_over_length_errors each time it's called, and does not reset the rx_over_length_errors to zero:

		ns->rx_length_errors += le32_to_cpu(s->rx_short_frame_errors) +
			nic->rx_over_length_errors;

So, should we zero out that counter after reading it (although that might race)? Any idea how this is supposed to work?

Thanks,
Ben

Ben Greear <[email protected]>
Candela Technologies Inc  http://www.candelatech.com


------------------------------------------------------------------------------
Virtualization & Cloud Management Using Capacity Planning
Cloud computing makes use of virtualization - but cloud computing 
also focuses on allowing computing to be delivered as a service.
http://www.accelacomm.com/jaw/sfnl/114/51521223/
_______________________________________________
E1000-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/e1000-devel
To learn more about Intel&#174; Ethernet, visit http://communities.intel.com/community/wired