Thread (17 messages) 17 messages, 6 authors, 2012-06-08

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 +0100
quoted
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.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help