summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFilip Tehlar <ftehlar@cisco.com>2019-06-04 01:22:55 +0000
committerDamjan Marion <dmarion@me.com>2019-06-06 12:07:09 +0000
commit831f4200cab2b363f2a8ea340331343c14407b7d (patch)
tree0c69ad2f1ad66cb0429dd7ca85b98eda7d07ab32
parenta342da22f35676114ca928c65b0219fac8009f7a (diff)
dpdk: fix clear hw stats
.. and remove helper stat struct for keeping last cleared stats. This is not needed anymore as dpdk lib provides rte_eth_dev_reset(). Change-Id: I78076e689aac7ca70836ce688dfa8e704f64cd84 Signed-off-by: Filip Tehlar <ftehlar@cisco.com>
-rw-r--r--src/plugins/dpdk/device/device.c1
-rw-r--r--src/plugins/dpdk/device/dpdk.h1
-rw-r--r--src/plugins/dpdk/device/format.c4
3 files changed, 3 insertions, 3 deletions
diff --git a/src/plugins/dpdk/device/device.c b/src/plugins/dpdk/device/device.c
index 836357f1c9e..270abe19190 100644
--- a/src/plugins/dpdk/device/device.c
+++ b/src/plugins/dpdk/device/device.c
@@ -397,6 +397,7 @@ dpdk_clear_hw_interface_counters (u32 instance)
dpdk_main_t *dm = &dpdk_main;
dpdk_device_t *xd = vec_elt_at_index (dm->devices, instance);
+ rte_eth_stats_reset (xd->port_id);
rte_eth_xstats_reset (xd->port_id);
}
diff --git a/src/plugins/dpdk/device/dpdk.h b/src/plugins/dpdk/device/dpdk.h
index a1c522490ab..0ec4ec3f28b 100644
--- a/src/plugins/dpdk/device/dpdk.h
+++ b/src/plugins/dpdk/device/dpdk.h
@@ -246,7 +246,6 @@ typedef struct
struct rte_eth_stats stats;
struct rte_eth_stats last_stats;
- struct rte_eth_stats last_cleared_stats;
struct rte_eth_xstat *xstats;
f64 time_last_stats_update;
dpdk_port_type_t port_type;
diff --git a/src/plugins/dpdk/device/format.c b/src/plugins/dpdk/device/format.c
index be2349d9e9d..7bba3eb4b6e 100644
--- a/src/plugins/dpdk/device/format.c
+++ b/src/plugins/dpdk/device/format.c
@@ -646,11 +646,11 @@ format_dpdk_device (u8 * s, va_list * args)
/* $$$ MIB counters */
{
#define _(N, V) \
- if ((xd->stats.V - xd->last_cleared_stats.V) != 0) { \
+ if (xd->stats.V != 0) { \
s = format (s, "\n%U%-40U%16Lu", \
format_white_space, indent + 2, \
format_c_identifier, #N, \
- xd->stats.V - xd->last_cleared_stats.V); \
+ xd->stats.V); \
} \
foreach_dpdk_counter