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 <olivier.matz at 6wind.com>