[PATCH] ethdev: remove telemetry Rx mbuf alloc failed field

Thomas Monjalon thomas at monjalon.net
Wed Mar 1 12:37:56 CET 2023


28/02/2023 19:02, Ferruh Yigit:
> 'eth_dev->data->rx_mbuf_alloc_failed' field is not directly exposed to
> user via ethdev APIs but it is used internally to set "stats->rx_nombuf'
> which is exposed via ehtdev stat APIs.
> 
> But telemetry exposes this field to user via "/ethdev/info",
> instead user can get 'rx_nombuf' value from stats via "/ethdev/stats".
> 
> Removing 'rx_mbuf_alloc_failed' from telemetry to align with ethdev APIs
> 
> Fixes: 58b43c1ddfd1 ("ethdev: add telemetry endpoint for device info")
> Cc: stable at dpdk.org
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit at amd.com>
> ---
> -	rte_tel_data_add_dict_uint(d, "rx_mbuf_alloc_fail",
> -			eth_dev->data->rx_mbuf_alloc_failed);

Good catch, thanks.

Acked-by: Thomas Monjalon <thomas at monjalon.net>




More information about the stable mailing list