[dpdk-dev] [PATCH] ethdev: fix statistics description

Dai, Wei wei.dai at intel.com
Wed Nov 2 09:28:37 CET 2016


Hi, John & Greg

Would you please give any opinion for this patch ?

I have looked through all PMDs and found not all statistics items can be supported by some NIC.
For example,  rx_nombuf,  q_ipackets,  q_opackets,  q_ibytes and q_obytes are not supported by i40e.
But when the function rte_eth_stats_get(uint8_t port_id, struct rte_eth_stats *stats) is called for i40e PMD,
Above un-supported statistics item in output stats are zero, this is not real value.
So far, there is no way to know whether an item in struct rte_eth_stats is supported or not only from this structure definition.
Maybe some structure member can be added to indicate each of statistics item valid or not.
But this means ABI change.

In following list, I list statistics support details of all PMDs.
Hope it can be displayed in your screen.

Thanks
/Wei

NIC           ipackets   opackets  ibytes  obytes  imissed  ierrors  oerrors  rx_nombuf  q_ipackets   q_opacktes     q_ibytes    q_obytes  q_errors
af_packet      y          y         y       y       n        n        y        n          y            y            y         y         y
bnx2x         y          y         y       y       y        y        y        y          n            n            n         n         n
bnxt          y          y         y       y       y        y        y        n          y            y            y         y         y
bonding       y          y         y       y       y        y        y        y          y            y            y         y         y
cxgbe         y          y         y       y       y        y        y        n          y            y            y         y         y
e1000(igb)     y          y         y       y       y        y        y        n          n            n            n         n         n
e1000(igbvf)   y          y         y       y       n        n        n        n          n            n            n         n         n
ena           y          y         y       y       y        y        y        y          n            n            n         n         n
enic          y          y         y       y       y        y        y        y          n            n            n         n         n
fm10k         y          y         y       y       n        n        n        n          y            y            y         y         n
i40e          y          y         y       y       y        y        y        n          n            n            n         n         n
i40evf        y          y         y       y       n        y        y        n          n            n            n         n         n
ixgbe         y          y         y       y       y        y        y        n          y            y            y         y         y
ixgbevf       y          y         y       y       n        n        n        n          n            n            n         n         n
mlx4          y          y         y       y       n        y        y        y          y            y            y         y         y
mlx5          y          y         y       y       n        y        y        y          y            y            y         y         y
mpipe         y          y         y       y       n        y        y        y          y            y            y         y         y
nfp           y          y         y       y       y        y        y        y          y            y            y         y         n
null          y          y         n       n       n        n        y        n          y            y            n         n         y
pcap          y          y         y       y       n        n        y        n          y            y            y         y         y
qede          y          y         y       y       y        y        y        y          n            n            n         n         n
ring          y          y         n       n       n        n        y        n          y            y            n         n         y
szedata2      y          y         y       y       n        n        y        n          y            y            y         y         y
thunderx      y          y         y       y       y        y        y        n          y            y            y         y         n
vhost         y          y         y       y       n        n        y        n          y            y            y         y         n
virtio         y          y         y       y       n        y        y        y          y            y            y         y         n
vmxnet3      y          y         y       y       n        y        y        y          y            y            y         y         y
xenvirt       y          y         n       n       n        n        n        n          n            n            n         n         n

> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Sent: Tuesday, October 4, 2016 5:35 PM
> To: Dai, Wei <wei.dai at intel.com>; Mcnamara, John
> <john.mcnamara at intel.com>
> Cc: dev at dpdk.org
> Subject: Re: [dpdk-dev] [PATCH] ethdev: fix statistics description
> 
> 2016-08-26 18:08, Wei Dai:
> >  /**
> >   * A structure used to retrieve statistics for an Ethernet port.
> > + * Not all statistics fields in struct rte_eth_stats are supported
> > + * by any type of network interface card (NIC). If any statistics
> > + * field is not supported, its value is 0 .
> >   */
> >  struct rte_eth_stats {
> 
> I'm missing the point of this patch.
> Why do you think it is a fix?
> 
> John, any opinion?




More information about the dev mailing list