Not sure review from who we should wait.
Cc'ing telemetry people
29/09/2021 08:40, Gowrishankar Muthukrishnan:
> Add telemetry endpoint for mempool info.
>
> Signed-off-by: Gowrishankar Muthukrishnan <gmuthukri...@marvell.com>
> ---
> lib/mempool/rte_mempool.c | 84 +++++++++++++++++++++++++++++++++++++++
> 1 file changed, 84 insertions(+)
>
> diff --git a/lib/mempool/rte_mempool.c b/lib/mempool/rte_mempool.c
> index 59a588425b..b8dad2997a 100644
> --- a/lib/mempool/rte_mempool.c
> +++ b/lib/mempool/rte_mempool.c
> @@ -31,6 +31,7 @@
> #include <rte_spinlock.h>
> #include <rte_tailq.h>
> #include <rte_eal_paging.h>
> +#include <rte_telemetry.h>
>
> #include "rte_mempool.h"
> #include "rte_mempool_trace.h"
> @@ -1343,3 +1344,86 @@ void rte_mempool_walk(void (*func)(struct rte_mempool
> *, void *),
>
> rte_mcfg_mempool_read_unlock();
> }
> +
> +static void
> +mempool_list_cb(struct rte_mempool *mp, void *arg)
> +{
> + struct rte_tel_data *d = (struct rte_tel_data *)arg;
> +
> + rte_tel_data_add_array_string(d, mp->name);
> +}
> +
> +static int
> +mempool_handle_list(const char *cmd __rte_unused,
> + const char *params __rte_unused, struct rte_tel_data *d)
> +{
> + rte_tel_data_start_array(d, RTE_TEL_STRING_VAL);
> + rte_mempool_walk(mempool_list_cb, d);
> + return 0;
> +}
> +
> +struct mempool_info_cb_arg {
> + char *pool_name;
> + struct rte_tel_data *d;
> +};
> +
> +static void
> +mempool_info_cb(struct rte_mempool *mp, void *arg)
> +{
> + struct mempool_info_cb_arg *info = (struct mempool_info_cb_arg *)arg;
> + const struct rte_memzone *mz;
> +
> + if (strncmp(mp->name, info->pool_name, RTE_MEMZONE_NAMESIZE))
> + return;
> +
> + rte_tel_data_add_dict_string(info->d, "name", mp->name);
> + rte_tel_data_add_dict_int(info->d, "pool_id", mp->pool_id);
> + rte_tel_data_add_dict_int(info->d, "flags", mp->flags);
> + rte_tel_data_add_dict_int(info->d, "socket_id", mp->socket_id);
> + rte_tel_data_add_dict_int(info->d, "size", mp->size);
> + rte_tel_data_add_dict_int(info->d, "cache_size", mp->cache_size);
> + rte_tel_data_add_dict_int(info->d, "elt_size", mp->elt_size);
> + rte_tel_data_add_dict_int(info->d, "header_size", mp->header_size);
> + rte_tel_data_add_dict_int(info->d, "trailer_size", mp->trailer_size);
> + rte_tel_data_add_dict_int(info->d, "private_data_size",
> + mp->private_data_size);
> + rte_tel_data_add_dict_int(info->d, "ops_index", mp->ops_index);
> + rte_tel_data_add_dict_int(info->d, "populated_size",
> + mp->populated_size);
> +
> + mz = mp->mz;
> + rte_tel_data_add_dict_string(info->d, "mz_name", mz->name);
> + rte_tel_data_add_dict_int(info->d, "mz_len", mz->len);
> + rte_tel_data_add_dict_int(info->d, "mz_hugepage_sz",
> + mz->hugepage_sz);
> + rte_tel_data_add_dict_int(info->d, "mz_socket_id", mz->socket_id);
> + rte_tel_data_add_dict_int(info->d, "mz_flags", mz->flags);
> +}
> +
> +static int
> +mempool_handle_info(const char *cmd __rte_unused, const char *params,
> + struct rte_tel_data *d)
> +{
> + struct mempool_info_cb_arg mp_arg;
> + char name[RTE_MEMZONE_NAMESIZE];
> +
> + if (params == NULL || strlen(params) == 0)
> + return -EINVAL;
> +
> + rte_strlcpy(name, params, RTE_MEMZONE_NAMESIZE);
> +
> + rte_tel_data_start_dict(d);
> + mp_arg.pool_name = name;
> + mp_arg.d = d;
> + rte_mempool_walk(mempool_info_cb, &mp_arg);
> +
> + return 0;
> +}
> +
> +RTE_INIT(mempool_init_telemetry)
> +{
> + rte_telemetry_register_cmd("/mempool/list", mempool_handle_list,
> + "Returns list of available mempool. Takes no parameters");
> + rte_telemetry_register_cmd("/mempool/info", mempool_handle_info,
> + "Returns mempool info. Parameters: pool_name");
> +}
>
