DPDK patches and discussions
 help / color / mirror / Atom feed
From: Michal Jastrzebski <michalx.k.jastrzebski@intel.com>
To: dev@dpdk.org
Cc: harry.van.haaren@intel.com, deepak.k.jain@intel.com,
	Kuba Kozak <kubax.kozak@intel.com>
Subject: [dpdk-dev] [PATCH v1 0/6] Extended xstats API in ethdev library to allow grouping of stats
Date: Thu, 27 Apr 2017 16:42:34 +0200	[thread overview]
Message-ID: <20170427144240.10188-1-michalx.k.jastrzebski@intel.com> (raw)

From: Kuba Kozak <kubax.kozak@intel.com>

This patchset fixes following patches:

    commit ea85e7d711b6 ("ethdev: retrieve xstats by ID")
    commit a954495245c4 ("ethdev: get xstats ID by name")
    commit 1223608adb9b ("app/proc-info: support xstats by ID")
    commit 25e38f09af9c ("net/e1000: support xstats by ID")
    commit 923419333f5a ("net/ixgbe: support xstats by ID")

This patch addresses some API concerns with the xstat patchset
applied in DPDK 17.05 RC2.

For clarity the first patch reverts all changes from the above
commits.

The subsequent patches extends the xstats API in ethdev library
to allow grouping of stats logically so they can be retrieved per
logical grouping managed by the application.

The patchset adds new functions rte_eth_xstats_get_names_by_id()
and rte_eth_xstats_get_by_id() to use a new list of arguments:
array of ids and array of values. It also introduces a new
function, rte_eth_xstats_get_id_by_name(), to retrieve xstats ids
by their names.

It also extends the functionality of the proc_info application:
    --xstats-name NAME: to display single xstat value by NAME

Finally the test-pmd application is updated to use new API.


Kuba Kozak (6):
  ethdev: revert patches extending xstats API in ethdev
  ethdev: retrieve xstats by ID
  ethdev: get xstats ID by name
  app/proc-info: support xstats by ID and by name
  net/e1000: support xstats by ID
  net/ixgbe: support xstats by ID

 app/proc_info/main.c                    |  12 +-
 app/test-pmd/config.c                   |  19 ++-
 doc/guides/prog_guide/poll_mode_drv.rst |  22 ++--
 doc/guides/rel_notes/release_17_05.rst  |   5 +-
 drivers/net/e1000/igb_ethdev.c          |  26 ++--
 drivers/net/ixgbe/ixgbe_ethdev.c        |  25 ++--
 lib/librte_ether/rte_ethdev.c           | 224 +++++++++++++++++++++-----------
 lib/librte_ether/rte_ethdev.h           | 175 ++++++++-----------------
 lib/librte_ether/rte_ether_version.map  |   6 +-
 9 files changed, 259 insertions(+), 255 deletions(-)

-- 
2.7.4

             reply	other threads:[~2017-04-27 14:43 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-04-27 14:42 Michal Jastrzebski [this message]
2017-04-27 14:42 ` [dpdk-dev] [PATCH v1 1/6] ethdev: revert patches extending xstats API in ethdev Michal Jastrzebski
2017-04-28 11:09   ` Van Haaren, Harry
2017-04-27 14:42 ` [dpdk-dev] [PATCH v1 2/6] ethdev: retrieve xstats by ID Michal Jastrzebski
2017-04-28 11:09   ` Van Haaren, Harry
2017-04-27 14:42 ` [dpdk-dev] [PATCH v1 3/6] ethdev: get xstats ID by name Michal Jastrzebski
2017-04-28 11:10   ` Van Haaren, Harry
2017-04-27 14:42 ` [dpdk-dev] [PATCH v1 4/6] app/proc-info: support xstats by ID and " Michal Jastrzebski
2017-04-28 11:10   ` Van Haaren, Harry
2017-04-27 14:42 ` [dpdk-dev] [PATCH v1 5/6] net/e1000: support xstats by ID Michal Jastrzebski
2017-04-28 11:10   ` Van Haaren, Harry
2017-04-27 14:42 ` [dpdk-dev] [PATCH v1 6/6] net/ixgbe: " Michal Jastrzebski
2017-04-28 11:10   ` Van Haaren, Harry
2017-05-01 21:34 ` [dpdk-dev] [PATCH v1 0/6] Extended xstats API in ethdev library to allow grouping of stats Thomas Monjalon
2017-05-01 21:49   ` Thomas Monjalon
2017-05-01 21:50 ` Thomas Monjalon

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170427144240.10188-1-michalx.k.jastrzebski@intel.com \
    --to=michalx.k.jastrzebski@intel.com \
    --cc=deepak.k.jain@intel.com \
    --cc=dev@dpdk.org \
    --cc=harry.van.haaren@intel.com \
    --cc=kubax.kozak@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).