[Intel-wired-lan] [PATCH v3] fm10k: cleanup fm10k stats and remove debug-statistics
Jeff Kirsher
jeffrey.t.kirsher at intel.com
Wed Mar 2 22:48:14 UTC 2016
On Wed, 2016-03-02 at 12:45 -0800, Jacob Keller wrote:
> This change fixes up subtle issues with the current fm10k ethtool
> stats.
> Primarily, support of debug-statistics and per-queue length
> statistics
> is not something the current API can handle. Due to the way that
> ethtool
> works, the number of statistics needs to be static for the life time
> of
> a given device. Our use of debug-statistics does not really allow for
> this, so this patch drops its use.
>
> Finish this cleanup by reworking the per-queue stats to use the new
> helper functions which reduce the duplicate and error prone code.
>
> Signed-off-by: Jacob Keller <jacob.e.keller at intel.com>
> ---
>
> Notes:
> - v3
> * based on comments upstream, make queues static
> * remove debug-statistics support since this can break the
> ethtool API
>
> - v4
> * remove extra debug stats strings forgotten in previous rework
>
> This is based on the fact that the ethtool API cannot really support
> variable statistics lengths.
>
> drivers/net/ethernet/intel/fm10k/fm10k_ethtool.c | 127 ++++++-------
> ----------
> 1 file changed, 33 insertions(+), 94 deletions(-)
So is this really v4? So much fun when I see multiple patches with the
same version number. :-(
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <http://lists.osuosl.org/pipermail/intel-wired-lan/attachments/20160302/8dab42b6/attachment.asc>
More information about the Intel-wired-lan
mailing list