[PATCH 4/5] net/mlx5: add assertions in counter get/put

[Michael Baum]

Add assertions to help debug in case of counter double alloc/free.

Signed-off-by: Michael Baum <michae...@nvidia.com>
Acked-by: Matan Azrad <ma...@nvidia.com>
Acked-by: Xiaoyu Min <jack...@nvidia.com>
---
 drivers/net/mlx5/mlx5_hws_cnt.h | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/net/mlx5/mlx5_hws_cnt.h b/drivers/net/mlx5/mlx5_hws_cnt.h
index 6e371f1929..338ee4d688 100644
--- a/drivers/net/mlx5/mlx5_hws_cnt.h
+++ b/drivers/net/mlx5/mlx5_hws_cnt.h
@@ -396,6 +396,7 @@ mlx5_hws_cnt_pool_put(struct mlx5_hws_cnt_pool *cpool,
        uint32_t iidx;
 
        iidx = mlx5_hws_cnt_iidx(cpool, *cnt_id);
+       MLX5_ASSERT(cpool->pool[iidx].in_used);
        cpool->pool[iidx].in_used = false;
        cpool->pool[iidx].query_gen_when_free =
                __atomic_load_n(&cpool->query_gen, __ATOMIC_RELAXED);
@@ -475,6 +476,7 @@ mlx5_hws_cnt_pool_get(struct mlx5_hws_cnt_pool *cpool, 
uint32_t *queue,
                __hws_cnt_query_raw(cpool, *cnt_id,
                                    &cpool->pool[iidx].reset.hits,
                                    &cpool->pool[iidx].reset.bytes);
+               MLX5_ASSERT(!cpool->pool[iidx].in_used);
                cpool->pool[iidx].in_used = true;
                cpool->pool[iidx].age_idx = age_idx;
                return 0;
@@ -511,6 +513,7 @@ mlx5_hws_cnt_pool_get(struct mlx5_hws_cnt_pool *cpool, 
uint32_t *queue,
                            &cpool->pool[iidx].reset.bytes);
        rte_ring_dequeue_zc_elem_finish(qcache, 1);
        cpool->pool[iidx].share = 0;
+       MLX5_ASSERT(!cpool->pool[iidx].in_used);
        cpool->pool[iidx].in_used = true;
        cpool->pool[iidx].age_idx = age_idx;
        return 0;
-- 
2.25.1