From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga17.intel.com ([192.55.52.151]:64422 "EHLO mga17.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752716AbeCMVOQ (ORCPT ); Tue, 13 Mar 2018 17:14:16 -0400 Date: Tue, 13 Mar 2018 14:14:14 -0700 From: Jesse Brandeburg To: Eric Dumazet , davem@davemloft.net Cc: "Venkataramanan, Anirudh" , "kubakici@wp.pl" , "netdev@vger.kernel.org" , "intel-wired-lan@lists.osuosl.org" , jesse.brandeburg@intel.com Subject: Re: [Intel-wired-lan] [PATCH 12/15] ice: Add stats and ethtool support Message-ID: <20180313141414.00007d11@intel.com> In-Reply-To: References: <20180309172136.9073-1-anirudh.venkataramanan@intel.com> <20180309172136.9073-13-anirudh.venkataramanan@intel.com> <20180309151428.4d73358c@cakuba.netronome.com> <1520967916.696.21.camel@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: netdev-owner@vger.kernel.org List-ID: On Tue, 13 Mar 2018 12:17:10 -0700 Eric Dumazet wrote: > > Yes, this is a recurring mistake > > See commit > bf909456f6a89654cb65c01fe83a4f9b133bf978 Revert "net: hns3: Add packet > statistics of netdev" Thanks for the pointer, that was a useful thread to review. I understand the point that was made about not having the netdev stats shown in ethtool -S. We definitely do provide per-queue stats as well as these regular stats in ethtool -S. I do remember from the past discussions that it *is* useless for the driver to keep internally any stats that were already stored via the get_stats NDO, and we missed it in the internal review that this driver was doing that, so that will be fixed. Maybe it's just that I've been doing this too long, but I regularly (and many other customers/users do as well) depend on the ethtool stats being atomically updated w.r.t. each other. This means that if I'm getting the over rx_packets, as well as the per-queue rx_packets, and I read them all at once from the driver with ethtool, then I can check that things are working as expected. If I have to gather the netdev stats from /proc/net/dev (which iproute2 tool shows the /proc/net/dev stats?) and somehow atomically gather the ethtool -S stats. What's the user to do in this brave new world where ethtool doesn't at least have rx/tx bytes and packets?