From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olivier MATZ Subject: Re: [PATCH v6 0/9] Expose IXGBE extended stats to DPDK apps Date: Thu, 16 Jul 2015 09:54:04 +0200 Message-ID: <55A7631C.3020408@6wind.com> References: <1436965894-130019-1-git-send-email-maryam.tahhan@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit To: Maryam Tahhan , dev@dpdk.org Return-path: Received: from mail-wi0-f171.google.com (mail-wi0-f171.google.com [209.85.212.171]) by dpdk.org (Postfix) with ESMTP id AC6C55A7A for ; Thu, 16 Jul 2015 09:54:11 +0200 (CEST) Received: by widic2 with SMTP id ic2so8005645wid.0 for ; Thu, 16 Jul 2015 00:54:11 -0700 (PDT) In-Reply-To: <1436965894-130019-1-git-send-email-maryam.tahhan@intel.com> 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" Hi Maryam, On 07/15/2015 03:11 PM, Maryam Tahhan wrote: > This patch set implements xstats_get() and xstats_reset() in dev_ops for > ixgbe to expose detailed error statistics to DPDK applications. The > dump_cfg application was extended to demonstrate the usage of > retrieving statistics for DPDK interfaces and renamed to proc_info > in order reflect this new functionality. This patch set also removes non > generic statistics from the statistics strings at the ethdev level and > marks the relevant registers as depricated in struct rte_eth_stats. > > v2: > - Fixed patch dependencies. > - Broke down patches into smaller logical changes. > > v3: > - Removes non-generic stats fields in rte_stats_strings and deprecates > the fields related to them in struct rte_eth_stats. > - Modifies rte_eth_xstats_get() to return generic stats and extended > stats. > > v4: > - Replace count use in the loop in ixgbe_dev_xstats_get() function > definition with i. > - Breakdown "ixgbe: add NIC specific stats removed from ethdev" into > two patches, one that adds the stats and another that extends > ierrors to include more error stats. > - Remove second call to ixgbe_dev_xstats_get() from > rte_eth_xstats_get(). > > v5: > - Added documentation for proc_info. > - Fixed proc_info copyright year. > - Display queue stats for all devices in proc_info. > > v6: > - Modified the driver implementation of ixgbe_dev_xstats_get() so that > it doesn't worry about the generic stats written by the generic layer. > > Maryam Tahhan (9): > ixgbe: move stats register reads to a new function > ixgbe: add functions to get and reset xstats > ethdev: expose extended error stats > ethdev: remove HW specific stats in stats structs > ixgbe: add NIC specific stats removed from ethdev > ixgbe: return more errors in ierrors > app: remove dump_cfg > app: add a new app proc_info > doc: Add documentation for proc_info > > MAINTAINERS | 4 + > app/Makefile | 2 +- > app/dump_cfg/Makefile | 45 ----- > app/dump_cfg/main.c | 92 --------- > app/proc_info/Makefile | 45 +++++ > app/proc_info/main.c | 354 +++++++++++++++++++++++++++++++++ > doc/guides/rel_notes/abi.rst | 12 ++ > doc/guides/sample_app_ug/index.rst | 1 + > doc/guides/sample_app_ug/proc_info.rst | 71 +++++++ > drivers/net/ixgbe/ixgbe_ethdev.c | 193 ++++++++++++++---- > lib/librte_ether/rte_ethdev.c | 40 ++-- > lib/librte_ether/rte_ethdev.h | 30 ++- > mk/rte.sdktest.mk | 4 +- > 13 files changed, 685 insertions(+), 208 deletions(-) > delete mode 100644 app/dump_cfg/Makefile > delete mode 100644 app/dump_cfg/main.c > create mode 100644 app/proc_info/Makefile > create mode 100644 app/proc_info/main.c > create mode 100644 doc/guides/sample_app_ug/proc_info.rst > mode change 100644 => 100755 lib/librte_ether/rte_ethdev.c Acked-by: Olivier Matz