Re: [PATCH 5/6] tuntap: per queue 64 bit stats
From: Jason Wang <hidden>
Date: 2012-06-26 06:26:42
Also in:
lkml
On 06/26/2012 02:10 PM, Eric Dumazet wrote:
On Tue, 2012-06-26 at 14:00 +0800, Jason Wang wrote:quoted
Yes, looks like it's hard to use NETIF_F_LLTX without breaking the u64 statistics, may worth to use tx lock and alloc_netdev_mq().Yes, this probably needs percpu storage (if you really want to use include/linux/u64_stats_sync.h). But percpu storage seems a bit overkill with a raising number of cpus on typical machines. For loopback device, its fine because we only have one lo device per network namespace, and some workloads really hit hard this device. But for tuntap, I am not sure ?
The problem is that we want to collect per-queue statistics. So if we convert tuntap to use alloc_netdev_mq(), the tx statistics would be updated under tx lock which looks safe.
-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/