From 09deb982228ce70439976c8d4c2b92b0d0fb4172 Mon Sep 17 00:00:00 2001 From: Ido Barnea Date: Thu, 9 Mar 2017 16:59:28 +0000 Subject: [PATCH] net/ixgbevf: set xstats id values Without setting the id, calling xstats_get twice with same array causes memory corruption. Also, if IXGBEVF_NB_XSTATS will be different than 1 in the future, this will cause issues. Fixes: 156712ba40d3 ("ixgbevf: add extending stats") Signed-off-by: Ido Barnea --- drivers/net/ixgbe/ixgbe_ethdev.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c b/drivers/net/ixgbe/ixgbe_ethdev.c index 19b971dc86..4fd22c5ee3 100644 --- a/drivers/net/ixgbe/ixgbe_ethdev.c +++ b/drivers/net/ixgbe/ixgbe_ethdev.c @@ -3247,6 +3247,7 @@ ixgbevf_dev_xstats_get(struct rte_eth_dev *dev, struct rte_eth_xstat *xstats, /* Extended stats */ for (i = 0; i < IXGBEVF_NB_XSTATS; i++) { + xstats[i].id = i; xstats[i].value = *(uint64_t *)(((char *)hw_stats) + rte_ixgbevf_stats_strings[i].offset); } -- 2.20.1