From patchwork Fri Jun 26 14:30:11 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: "Tahhan, Maryam" X-Patchwork-Id: 5851 Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id 00AD6C5C6; Fri, 26 Jun 2015 16:30:16 +0200 (CEST) Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id C7AA3C4B6 for ; Fri, 26 Jun 2015 16:30:14 +0200 (CEST) Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by orsmga102.jf.intel.com with ESMTP; 26 Jun 2015 07:30:13 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.13,685,1427785200"; d="scan'208,217";a="750997180" Received: from irsmsx152.ger.corp.intel.com ([163.33.192.66]) by fmsmga002.fm.intel.com with ESMTP; 26 Jun 2015 07:30:13 -0700 Received: from irsmsx109.ger.corp.intel.com ([169.254.13.200]) by IRSMSX152.ger.corp.intel.com ([169.254.6.90]) with mapi id 14.03.0224.002; Fri, 26 Jun 2015 15:30:12 +0100 From: "Tahhan, Maryam" To: Kyle Larose Thread-Topic: [dpdk-dev] [PATCH v3 4/7] ethdev: remove HW specific stats in stats structs Thread-Index: AQHQsA/4jSgP1CKxGECXRRr8DWhifJ2+wPWAgAAWRGA= Date: Fri, 26 Jun 2015 14:30:11 +0000 Message-ID: <1A27633A6DA49C4A92FCD5D4312DBF536A45182E@IRSMSX109.ger.corp.intel.com> References: <1435323558-169985-1-git-send-email-maryam.tahhan@intel.com> <1435323558-169985-5-git-send-email-maryam.tahhan@intel.com> In-Reply-To: Accept-Language: en-IE, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [163.33.239.182] MIME-Version: 1.0 X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [PATCH v3 4/7] ethdev: remove HW specific stats in stats structs X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Fri, Jun 26, 2015 at 8:59 AM, Maryam Tahhan > wrote: Remove non generic stats in rte_stats_strings and mark the relevant fields in struct rte_eth_stats as deprecated. Signed-off-by: Maryam Tahhan > --- doc/guides/rel_notes/abi.rst | 11 +++++++++++ lib/librte_ether/rte_ethdev.c | 9 --------- lib/librte_ether/rte_ethdev.h | 30 ++++++++++++++++++++---------- 3 files changed, 31 insertions(+), 19 deletions(-) Are CRC errors (ibadcrc) truly hardware specific? Which NIC (aside from purely virtual ones) does not have a MAC which does frame checksumming? Likewise, which NIC doesn't drop because the PCI bus/cpu/etc is too busy to pull packets off of it (imissed)? Debugging interactions with NICs is hard enough with only CRC errors and missed packets to go on. Without those it is close to impossible. CRC errors are almost guaranteed any time a bare-metal application is deployed: dirty fiber, bad SFPs, etc. How will users of the application be able to determine why their packets are dropping if they can only see "in errors"? I understand that we want to avoid placing too much useless information into these statistics structures. However, without a hardware-independent way of accessing fairly standard networking-equipment diagnostics, I feel like any real-world application using DPDK will be terribly cumbersome to build: every single one will need to develop an abstraction layer which detects the attached NICs, and loads an appropriate driver to integrate with the xstats api. Is there any plan for such an API? If not, is it really a good idea to deprecate these stats? Thanks, Kyle Hi Kyle If it’s just for debug/diagnostic purposes that this information is being used then I would recommend using the proc_info app which is already integrated with xstats will give you detailed error statistics. It runs as a DPDK secondary process. I’m not sure about crcerrors and imissed, I had taken feedback to my previous version of the patches onboard and made the changes based on that. Regards Maryam diff --git a/doc/guides/rel_notes/abi.rst b/doc/guides/rel_notes/abi.rst index f00a6ee..957b13f 100644 --- a/doc/guides/rel_notes/abi.rst +++ b/doc/guides/rel_notes/abi.rst @@ -38,3 +38,14 @@ Examples of Deprecation Notices Deprecation Notices ------------------- +* The following fields have been deprecated in rte_eth_stats: + * uint64_t imissed + * uint64_t ibadcrc + * uint64_t ibadlen + * uint64_t imcasts + * uint64_t fdirmatch + * uint64_t fdirmiss + * uint64_t tx_pause_xon + * uint64_t rx_pause_xon + * uint64_t tx_pause_xoff + * uint64_t rx_pause_xoff