RE: [PATCH 20.11] net/mlx5: fix typo in queue counter query

["Xueming(Steven) Li" <xuemingl@nvidia.com>]

Hi Matan,

Slava sent another backport patch for the same issue, I'll use that version:
https://mails.dpdk.org/archives/stable/2022-August/039873.html

Thanks,
Xueming

> -----Original Message-----
> From: Matan Azrad <matan@nvidia.com>
> Sent: Sunday, July 31, 2022 10:34 PM
> To: stable@dpdk.org
> Cc: Luca Boccassi <bluca@debian.org>; Kevin Traynor <ktraynor@redhat.com>; Christian Ehrhardt <christian.ehrhardt@canonical.com>;
> Xueming(Steven) Li <xuemingl@nvidia.com>
> Subject: [PATCH 20.11] net/mlx5: fix typo in queue counter query
> 
> The backport commit of the imissed statistics fix for 20.11 includes a typo issue and use a wrong pointer in DevX command to query queue
> counters.
> 
> Set the correct pointer points to the created queue counter object instead of the context pointer used wrongly in this query API.
> 
> Fixes: ede02cfc4783 ("net/mlx5: fix imissed statistics")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Matan Azrad <matan@nvidia.com>
> ---
>  drivers/net/mlx5/linux/mlx5_os.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/mlx5/linux/mlx5_os.c b/drivers/net/mlx5/linux/mlx5_os.c
> index f98cf90dd8..05f47127c9 100644
> --- a/drivers/net/mlx5/linux/mlx5_os.c
> +++ b/drivers/net/mlx5/linux/mlx5_os.c
> @@ -2589,7 +2589,7 @@ mlx5_os_read_dev_stat(struct mlx5_priv *priv, const char *ctr_name,
>  	if (priv->sh) {
>  		if (priv->q_counters != NULL &&
>  		    strcmp(ctr_name, "out_of_buffer") == 0)
> -			return mlx5_devx_cmd_queue_counter_query(priv->sh->ctx,
> +			return mlx5_devx_cmd_queue_counter_query(priv->q_counters,
>  							   0, (uint32_t *)stat);
>  		MKSTR(path, "%s/ports/%d/hw_counters/%s",
>  		      priv->sh->ibdev_path,
> --
> 2.25.1