mbox series

[v2,0/2] net/cxgbe: add support for xstats API

Message ID cover.1625157360.git.rahul.lakkireddy@chelsio.com (mailing list archive)
Headers
Series net/cxgbe: add support for xstats API |

Message

Rahul Lakkireddy July 1, 2021, 4:56 p.m. UTC 
  This series of patches add support to fetch port and queue stats via
xstats API.

Patch 1 adds support to fetch port and queue stats via xstats API.

Patch 2 removes queue stats from basic stats since they're available
via xstats API. Also removes RTE_ETH_DEV_AUTOFILL_QUEUE_XSTATS flag.

---
v2:
- Perform explicit checks for NULL in conditions.
- Use rte_strlcpy() instead of strcpy().

Rahul Lakkireddy (2):
  net/cxgbe: add support for xstats API
  net/cxgbe: remove queue stats from basic stats

 drivers/net/cxgbe/cxgbe_ethdev.c | 321 ++++++++++++++++++++++++++++---
 1 file changed, 297 insertions(+), 24 deletions(-)

Comments

Andrew Rybchenko July 2, 2021, 9:27 a.m. UTC | #1 
On 7/1/21 7:56 PM, Rahul Lakkireddy wrote:
> This series of patches add support to fetch port and queue stats via
> xstats API.
> 
> Patch 1 adds support to fetch port and queue stats via xstats API.

Fixed mulit-line comments to following DPDK coding style.
Just made it single-line.

> Patch 2 removes queue stats from basic stats since they're available
> via xstats API. Also removes RTE_ETH_DEV_AUTOFILL_QUEUE_XSTATS flag.
> 
> ---
> v2:
> - Perform explicit checks for NULL in conditions.
> - Use rte_strlcpy() instead of strcpy().
> 
> Rahul Lakkireddy (2):
>   net/cxgbe: add support for xstats API
>   net/cxgbe: remove queue stats from basic stats
> 
>  drivers/net/cxgbe/cxgbe_ethdev.c | 321 ++++++++++++++++++++++++++++---
>  1 file changed, 297 insertions(+), 24 deletions(-)
> 

Applied, thanks.