Re: [PATCHv3] net: usbnet: support 64bit stats in qmi_wwan driver

["Bjørn Mork" <bjorn-yOkvZcmFvRU@public.gmane.org>]

On March 31, 2017 3:27:59 PM CEST, Greg Ungerer <gerg@linux-m68k.org> wrote:
>On 31/03/17 18:48, Bjørn Mork wrote:
>
>>> +void usbnet_get_stats64(struct net_device *net, struct
>rtnl_link_stats64 *stats)
>>> +{
>>> +	struct usbnet *dev = netdev_priv(net);
>>> +	unsigned int start;
>>> +	int cpu;
>>> +
>>> +	netdev_stats_to_stats64(stats, &net->stats);
>>> +
>>> +	for_each_possible_cpu(cpu) {
>>> +		struct pcpu_sw_netstats *stats64;
>>> +		u64 rx_packets, rx_bytes;
>>> +		u64 tx_packets, tx_bytes;
>>> +
>>> +		stats64 = per_cpu_ptr(dev->stats64, cpu);
>>> +
>>> +		do {
>>> +			start = u64_stats_fetch_begin_irq(&stats64->syncp);
>>> +			rx_packets = stats64->rx_packets;
>>> +			rx_bytes = stats64->rx_bytes;
>>> +			tx_packets = stats64->tx_packets;
>>> +			tx_bytes = stats64->tx_bytes;
>>> +		} while (u64_stats_fetch_retry_irq(&stats64->syncp, start));
>>> +
>>> +		stats->rx_packets += rx_packets;
>>> +		stats->rx_bytes += rx_bytes;
>>> +		stats->tx_packets += tx_packets;
>>> +		stats->tx_bytes += tx_bytes;
>>> +	}
>>> +}
>>
>> So we only count packets and bytes.  No errors.  Why?
>
>All stats are counted. That call to netdev_stats_to_stats64() transfers
>all other stats struct fields (errors, etc) to the stats64 struct.
>No error counts are lost (though they are only stored as 32bits values
>on 32bit machines).


Ah, right. Thanks for explaining and sorry for being so slow. Then I have no objection to the patch as it is.


Bjørn

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html