From: Jakub Kicinski <kubak...@wp.pl>
Date: Fri, 9 Mar 2018 15:14:28 -0800

> On Fri,  9 Mar 2018 09:21:33 -0800, Anirudh Venkataramanan wrote:
>> +static const struct ice_stats ice_net_stats[] = {
>> +    ICE_NETDEV_STAT(rx_packets),
>> +    ICE_NETDEV_STAT(tx_packets),
>> +    ICE_NETDEV_STAT(rx_bytes),
>> +    ICE_NETDEV_STAT(tx_bytes),
>> +    ICE_NETDEV_STAT(rx_errors),
>> +    ICE_NETDEV_STAT(tx_errors),
>> +    ICE_NETDEV_STAT(rx_dropped),
>> +    ICE_NETDEV_STAT(tx_dropped),
>> +    ICE_NETDEV_STAT(multicast),
>> +    ICE_NETDEV_STAT(rx_length_errors),
>> +    ICE_NETDEV_STAT(rx_crc_errors),
>> +};
> 
> Please don't duplicate standard netdev stats in ethtool -S.

Indeed, unless you are providing per-queue versions of these
stats, don't do this.

Reply via email to