Re: [V2 RFC net-next PATCH 2/2] virtio_net: export more statistics through ethtool
From: Ben Hutchings <hidden>
Date: 2012-06-07 20:58:10
Also in:
lkml, virtualization
On Thu, 2012-06-07 at 21:56 +0100, Ben Hutchings wrote:
On Thu, 2012-06-07 at 13:39 -0700, Rick Jones wrote:quoted
On 06/07/2012 01:24 PM, Ben Hutchings wrote:quoted
On Thu, 2012-06-07 at 13:05 -0700, David Miller wrote:quoted
From: Ben Hutchings<redacted> Date: Thu, 7 Jun 2012 18:15:06 +0100quoted
I would really like to see some sort of convention for presenting per-queue statistics through ethtool. At the moment we have a complete mess of different formats:Indeed. Probably ${QUEUE_TYPE}-${INDEX}-${STATISTIC} is best. With an agreed upon list of queue types such as "rx", "tx", "rxtx" etc.I think we should leave the type names open-ended, as there are other useful groupings like per-virtual-port. In that case the separator should be chosen to allow arbitrary type names without ambiguity.So you mean like something along the lines of the presence of say '.' indicating indent a level: rx_bytes: 1234 myqueue1.rx_bytes: 234 myqueue2.rx_bytes: 345 ...Most drivers seem to want this sort of ordering/grouping: group0.foo group0.bar ... group1.foo group1.bar ... but if we have a standard way of indicating groups of statistics then the user can choose whether they want to reorder by type name.
I mean, whether they want to reorder/regroup by the final part of the statistic name. Ben. -- Ben Hutchings, Staff Engineer, Solarflare Not speaking for my employer; that's the marketing department's job. They asked us to note that Solarflare product names are trademarked.