mbox series

[v2,0/2] *failsafe: add xstats

Message ID 20190703172512.24909-1-stephen@networkplumber.org (mailing list archive)
Headers
Series *failsafe: add xstats |

Message

Stephen Hemminger July 3, 2019, 5:25 p.m. UTC 
  A useful feature of netvsc PMD is the ability to see how many packets
were processed through the VF device. This patch set adds a similar
(but more limited) capability to failsafe driver.

Since failsafe doesn't have top level xstats, this set uses the generic
xstats that exist already as a base then adds the sub-device xstats
to that.

v2 
    move generic xstats helpers to rte_ethdev_driver.h
    change prefix on subdevice stats to sub0_XXXX

Stephen Hemminger (2):
  ethdev: expose basic xstats for driver use
  failsafe: implement xstats

 drivers/net/failsafe/failsafe_ops.c      | 135 +++++++++++++++++++++++
 lib/librte_ethdev/rte_ethdev.c           |  17 ++-
 lib/librte_ethdev/rte_ethdev_driver.h    |  65 +++++++++++
 lib/librte_ethdev/rte_ethdev_version.map |   3 +
 4 files changed, 211 insertions(+), 9 deletions(-)

Comments

Gaëtan Rivet July 4, 2019, 9:56 a.m. UTC | #1 
On Wed, Jul 03, 2019 at 10:25:10AM -0700, Stephen Hemminger wrote:
> A useful feature of netvsc PMD is the ability to see how many packets
> were processed through the VF device. This patch set adds a similar
> (but more limited) capability to failsafe driver.
> 
> Since failsafe doesn't have top level xstats, this set uses the generic
> xstats that exist already as a base then adds the sub-device xstats
> to that.
> 
> v2 
>     move generic xstats helpers to rte_ethdev_driver.h
>     change prefix on subdevice stats to sub0_XXXX
> 

For the series:

Acked-by: Gaetan Rivet <gaetan.rivet@6wind.com>

> Stephen Hemminger (2):
>   ethdev: expose basic xstats for driver use
>   failsafe: implement xstats
> 
>  drivers/net/failsafe/failsafe_ops.c      | 135 +++++++++++++++++++++++
>  lib/librte_ethdev/rte_ethdev.c           |  17 ++-
>  lib/librte_ethdev/rte_ethdev_driver.h    |  65 +++++++++++
>  lib/librte_ethdev/rte_ethdev_version.map |   3 +
>  4 files changed, 211 insertions(+), 9 deletions(-)
> 
> -- 
> 2.20.1
>