>> +    { "rx_broadcast", E1000_STAT(stats.bprc) },
>> +    { "tx_broadcast", E1000_STAT(stats.bptc) },
>> +    { "rx_multicast", E1000_STAT(stats.mprc) },
>> +    { "tx_multicast", E1000_STAT(stats.mptc) },
>>      { "rx_errors", E1000_STAT(net_stats.rx_errors) },
>>      { "tx_errors", E1000_STAT(net_stats.tx_errors) },
>>      { "tx_dropped", E1000_STAT(net_stats.tx_dropped) },
>
>NAK -- you also need to remove the standard net stats, which are 
>exported elsewhere

Jeff, can you please explain the reason for this NAK a little more?
Neither Auke nor I understand why you rejected the patch.

This patch just adds the display of a few more stats in Ethtool.  It
doesn't affect any other counters, and is really just a convenience
feature.  I added this to the driver because of a customer request.

Thank you in advance for edifying us.

-Mitch

-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to