Re: [dpdk-stable] [dpdk-dev] [PATCH v1 1/2] net/i40e: fix incorrect xstats value mapping

Re: [dpdk-stable] [dpdk-dev] [PATCH v1 1/2] net/i40e: fix incorrect xstats value mapping

[Kevin Traynor]

On 11/14/2016 06:14 AM, Remy Horton wrote:
> The offsets used in rte_i40evf_stats_strings for transmission
> statistics were wrong, returning the total byte count rather than
> the respective (unicast, multicast, broadcast, drop, & error)
> packet counts.
> 
> Fixes: da61cd084976 ("i40evf: add extended stats")

This should go to the LTS branch also.

Acked-by: Kevin Traynor <ktraynor@redhat.com>

> 
> Signed-off-by: Remy Horton <remy.horton@intel.com>
> ---
>  drivers/net/i40e/i40e_ethdev_vf.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/net/i40e/i40e_ethdev_vf.c b/drivers/net/i40e/i40e_ethdev_vf.c
> index aa306d6..afae2ec 100644
> --- a/drivers/net/i40e/i40e_ethdev_vf.c
> +++ b/drivers/net/i40e/i40e_ethdev_vf.c
> @@ -176,11 +176,11 @@ static const struct rte_i40evf_xstats_name_off rte_i40evf_stats_strings[] = {
>  	{"rx_unknown_protocol_packets", offsetof(struct i40e_eth_stats,
>  		rx_unknown_protocol)},
>  	{"tx_bytes", offsetof(struct i40e_eth_stats, tx_bytes)},
> -	{"tx_unicast_packets", offsetof(struct i40e_eth_stats, tx_bytes)},
> -	{"tx_multicast_packets", offsetof(struct i40e_eth_stats, tx_bytes)},
> -	{"tx_broadcast_packets", offsetof(struct i40e_eth_stats, tx_bytes)},
> -	{"tx_dropped_packets", offsetof(struct i40e_eth_stats, tx_bytes)},
> -	{"tx_error_packets", offsetof(struct i40e_eth_stats, tx_bytes)},
> +	{"tx_unicast_packets", offsetof(struct i40e_eth_stats, tx_unicast)},
> +	{"tx_multicast_packets", offsetof(struct i40e_eth_stats, tx_multicast)},
> +	{"tx_broadcast_packets", offsetof(struct i40e_eth_stats, tx_broadcast)},
> +	{"tx_dropped_packets", offsetof(struct i40e_eth_stats, tx_discards)},
> +	{"tx_error_packets", offsetof(struct i40e_eth_stats, tx_errors)},
>  };
>  
>  #define I40EVF_NB_XSTATS (sizeof(rte_i40evf_stats_strings) / \
>