On Fri, 2018-03-09 at 15:14 -0800, Jakub Kicinski wrote: > 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.
Jacub, Thanks for the feedback. I am not sure I understand what's being asked here. Do you mean to say that standard netdev stats should not be printed when we do ethtool -S or something else? Thanks! Ani
smime.p7s
Description: S/MIME cryptographic signature