[dpdk-dev,v3,01/11] doc: add extended statistics notes

Message ID 1445528914-27636-2-git-send-email-harry.van.haaren@intel.com (mailing list archive)
State Superseded, archived
Headers

Commit Message

Van Haaren, Harry Oct. 22, 2015, 3:48 p.m. UTC
  Add release notes for this patchset.

Signed-off-by: Harry van Haaren <harry.van.haaren@intel.com>
---
 doc/guides/rel_notes/release_2_2.rst | 12 ++++++++++++
 1 file changed, 12 insertions(+)
  

Comments

Van Haaren, Harry Oct. 30, 2015, 11:36 a.m. UTC | #1
This patchset adds an implementation of the xstats_get() and xstats_reset() API
to the following PMDs: virtio, igb, igbvf, ixgbe, ixgbevf, i40e, i40evf and
fm10k.

The xstats API allows DPDK apps to gain access to extended statistics from each
port on a NIC. These statistics are structured as per a scheme detailed in the
patch for the doc/prog_guide.

v4: Consistency of names, refactored Q stat code
v3: Added more stats to PMDs
v2: Send correct patchset

Harry van Haaren (10):
  doc: add extended statistics to prog_guide
  ethdev: update xstats_get() strings and Q handling
  virtio: add xstats() implementation
  igb: add xstats() implementation
  igbvf: add xstats() implementation
  ixgbe: add extended statistic strings
  ixgbevf: add xstats() implementation
  i40e: add xstats() implementation
  i40evf: add xstats() implementation
  fm10k: add xstats() implementation

 doc/guides/prog_guide/poll_mode_drv.rst |  51 ++++++-
 doc/guides/rel_notes/release_2_2.rst    |  14 ++
 drivers/net/e1000/igb_ethdev.c          | 193 ++++++++++++++++++++++--
 drivers/net/fm10k/fm10k_ethdev.c        |  87 +++++++++++
 drivers/net/i40e/i40e_ethdev.c          | 217 ++++++++++++++++++++++++++-
 drivers/net/i40e/i40e_ethdev_vf.c       |  89 ++++++++++-
 drivers/net/ixgbe/ixgbe_ethdev.c        | 254 +++++++++++++++++++++++++++-----
 drivers/net/virtio/virtio_ethdev.c      |  98 +++++++++++-
 drivers/net/virtio/virtio_rxtx.c        |  32 ++++
 drivers/net/virtio/virtqueue.h          |   4 +
 lib/librte_ether/rte_ethdev.c           |  38 +++--
 11 files changed, 1006 insertions(+), 71 deletions(-)
  
Tahhan, Maryam Oct. 30, 2015, 1:21 p.m. UTC | #2
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Harry van Haaren
> Sent: Friday, October 30, 2015 11:36 AM
> To: dev@dpdk.org
> Subject: [dpdk-dev] [PATCH v4 00/10] Port XStats
> 
> This patchset adds an implementation of the xstats_get() and xstats_reset()
> API to the following PMDs: virtio, igb, igbvf, ixgbe, ixgbevf, i40e, i40evf and
> fm10k.
> 
> The xstats API allows DPDK apps to gain access to extended statistics from
> each port on a NIC. These statistics are structured as per a scheme detailed in
> the patch for the doc/prog_guide.
> 
> v4: Consistency of names, refactored Q stat code
> v3: Added more stats to PMDs
> v2: Send correct patchset
> 
> Harry van Haaren (10):
>   doc: add extended statistics to prog_guide
>   ethdev: update xstats_get() strings and Q handling
>   virtio: add xstats() implementation
>   igb: add xstats() implementation
>   igbvf: add xstats() implementation
>   ixgbe: add extended statistic strings
>   ixgbevf: add xstats() implementation
>   i40e: add xstats() implementation
>   i40evf: add xstats() implementation
>   fm10k: add xstats() implementation
> 
>  doc/guides/prog_guide/poll_mode_drv.rst |  51 ++++++-
>  doc/guides/rel_notes/release_2_2.rst    |  14 ++
>  drivers/net/e1000/igb_ethdev.c          | 193 ++++++++++++++++++++++--
>  drivers/net/fm10k/fm10k_ethdev.c        |  87 +++++++++++
>  drivers/net/i40e/i40e_ethdev.c          | 217
> ++++++++++++++++++++++++++-
>  drivers/net/i40e/i40e_ethdev_vf.c       |  89 ++++++++++-
>  drivers/net/ixgbe/ixgbe_ethdev.c        | 254
> +++++++++++++++++++++++++++-----
>  drivers/net/virtio/virtio_ethdev.c      |  98 +++++++++++-
>  drivers/net/virtio/virtio_rxtx.c        |  32 ++++
>  drivers/net/virtio/virtqueue.h          |   4 +
>  lib/librte_ether/rte_ethdev.c           |  38 +++--
>  11 files changed, 1006 insertions(+), 71 deletions(-)
> 
> --
> 1.9.1

Series-acked-by: Maryam Tahhan <maryam.tahhan@intel.com>
  
Thomas Monjalon Nov. 2, 2015, 8:04 a.m. UTC | #3
Harry,
Checkpatch is not happy because you missed the space after the keywords
"if" and "for".
  

Patch

diff --git a/doc/guides/rel_notes/release_2_2.rst b/doc/guides/rel_notes/release_2_2.rst
index 4f75cff..fe93629 100644
--- a/doc/guides/rel_notes/release_2_2.rst
+++ b/doc/guides/rel_notes/release_2_2.rst
@@ -9,6 +9,18 @@  New Features
   *  Added support for Jumbo Frames.
   *  Optimize forwarding performance for Chelsio T5 40GbE cards.
 
+* **Extended Statistics**
+
+  Define extended statistics naming scheme to store metadata in the name
+  string name of each statistic. Implemented the extended stats API for
+  these PMDs:
+
+  * igb
+  * igbvf
+  * i40e
+  * i40evf
+  * fm10k
+  * virtio
 
 Resolved Issues
 ---------------