[v2,1/2] ring: add ring list telemetry cmd
Checks
Commit Message
This patch supports the list of rings with telemetry cmd.
An example using this command is shown below:
--> /ring/list
{
"/ring/list": [
"HT_0000:7d:00.2",
"MP_mb_pool_0"
]
}
Signed-off-by: Jie Hai <haijie1@huawei.com>
---
lib/ring/meson.build | 1 +
lib/ring/rte_ring.c | 40 ++++++++++++++++++++++++++++++++++++++++
2 files changed, 41 insertions(+)
Comments
Hi Jie,
> This patch supports the list of rings with telemetry cmd.
> An example using this command is shown below:
>
> --> /ring/list
> {
> "/ring/list": [
> "HT_0000:7d:00.2",
> "MP_mb_pool_0"
> ]
> }
>
> Signed-off-by: Jie Hai <haijie1@huawei.com>
> ---
> lib/ring/meson.build | 1 +
> lib/ring/rte_ring.c | 40 ++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 41 insertions(+)
>
> diff --git a/lib/ring/meson.build b/lib/ring/meson.build
> index c20685c689ac..7fca958ed7fa 100644
> --- a/lib/ring/meson.build
> +++ b/lib/ring/meson.build
> @@ -18,3 +18,4 @@ indirect_headers += files (
> 'rte_ring_rts.h',
> 'rte_ring_rts_elem_pvt.h',
> )
> +deps += ['telemetry']
> diff --git a/lib/ring/rte_ring.c b/lib/ring/rte_ring.c
> index cddaf6b2876f..bb1dafd4d1ca 100644
> --- a/lib/ring/rte_ring.c
> +++ b/lib/ring/rte_ring.c
> @@ -22,6 +22,7 @@
> #include <rte_errno.h>
> #include <rte_string_fns.h>
> #include <rte_tailq.h>
> +#include <rte_telemetry.h>
>
> #include "rte_ring.h"
> #include "rte_ring_elem.h"
> @@ -419,3 +420,42 @@ rte_ring_lookup(const char *name)
>
> return r;
> }
> +
> +static void
> +rte_ring_walk(void (*func)(struct rte_ring *, void *), void *arg)
As a nit: it is a static function, so I think we can skip 'rte_' prefix
for it.
Apart from that:
Acked-by: Konstantin Ananyev <konstantin.v.ananyev@yandex.ru>
> +{
> + struct rte_ring_list *ring_list;
> + struct rte_tailq_entry *te;
> +
> + ring_list = RTE_TAILQ_CAST(rte_ring_tailq.head, rte_ring_list);
> + rte_mcfg_tailq_read_lock();
> +
> + TAILQ_FOREACH(te, ring_list, next) {
> + (*func)((struct rte_ring *) te->data, arg);
> + }
> +
> + rte_mcfg_tailq_read_unlock();
> +}
> +
> +static void
> +ring_list_cb(struct rte_ring *r, void *arg)
> +{
> + struct rte_tel_data *d = (struct rte_tel_data *)arg;
> +
> + rte_tel_data_add_array_string(d, r->name);
> +}
> +
> +static int
> +ring_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_ring_walk(ring_list_cb, d);
> + return 0;
> +}
> +
> +RTE_INIT(ring_init_telemetry)
> +{
> + rte_telemetry_register_cmd("/ring/list", ring_handle_list,
> + "Returns list of available ring. Takes no parameters");
> +}
On 2023/1/23 0:40, Konstantin Ananyev wrote:
> Hi Jie,
>
>> This patch supports the list of rings with telemetry cmd.
>> An example using this command is shown below:
>>
>> --> /ring/list
>> {
>> "/ring/list": [
>> "HT_0000:7d:00.2",
>> "MP_mb_pool_0"
>> ]
>> }
>>
>> Signed-off-by: Jie Hai <haijie1@huawei.com>
>> ---
>> lib/ring/meson.build | 1 +
>> lib/ring/rte_ring.c | 40 ++++++++++++++++++++++++++++++++++++++++
>> 2 files changed, 41 insertions(+)
>>
>> diff --git a/lib/ring/meson.build b/lib/ring/meson.build
>> index c20685c689ac..7fca958ed7fa 100644
>> --- a/lib/ring/meson.build
>> +++ b/lib/ring/meson.build
>> @@ -18,3 +18,4 @@ indirect_headers += files (
>> 'rte_ring_rts.h',
>> 'rte_ring_rts_elem_pvt.h',
>> )
>> +deps += ['telemetry']
>> diff --git a/lib/ring/rte_ring.c b/lib/ring/rte_ring.c
>> index cddaf6b2876f..bb1dafd4d1ca 100644
>> --- a/lib/ring/rte_ring.c
>> +++ b/lib/ring/rte_ring.c
>> @@ -22,6 +22,7 @@
>> #include <rte_errno.h>
>> #include <rte_string_fns.h>
>> #include <rte_tailq.h>
>> +#include <rte_telemetry.h>
>> #include "rte_ring.h"
>> #include "rte_ring_elem.h"
>> @@ -419,3 +420,42 @@ rte_ring_lookup(const char *name)
>> return r;
>> }
>> +
>> +static void
>> +rte_ring_walk(void (*func)(struct rte_ring *, void *), void *arg)
>
> As a nit: it is a static function, so I think we can skip 'rte_' prefix
> for it.
> Apart from that:
> Acked-by: Konstantin Ananyev <konstantin.v.ananyev@yandex.ru>
>
Hi, Konstantin,
Thanks for your review. Accepted and fixed in v3.
Jie Hai
>> +{
>> + struct rte_ring_list *ring_list;
>> + struct rte_tailq_entry *te;
>> +
>> + ring_list = RTE_TAILQ_CAST(rte_ring_tailq.head, rte_ring_list);
>> + rte_mcfg_tailq_read_lock();
>> +
>> + TAILQ_FOREACH(te, ring_list, next) {
>> + (*func)((struct rte_ring *) te->data, arg);
>> + }
>> +
>> + rte_mcfg_tailq_read_unlock();
>> +}
>> +
>> +static void
>> +ring_list_cb(struct rte_ring *r, void *arg)
>> +{
>> + struct rte_tel_data *d = (struct rte_tel_data *)arg;
>> +
>> + rte_tel_data_add_array_string(d, r->name);
>> +}
>> +
>> +static int
>> +ring_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_ring_walk(ring_list_cb, d);
>> + return 0;
>> +}
>> +
>> +RTE_INIT(ring_init_telemetry)
>> +{
>> + rte_telemetry_register_cmd("/ring/list", ring_handle_list,
>> + "Returns list of available ring. Takes no parameters");
>> +}
>
> .
@@ -18,3 +18,4 @@ indirect_headers += files (
'rte_ring_rts.h',
'rte_ring_rts_elem_pvt.h',
)
+deps += ['telemetry']
@@ -22,6 +22,7 @@
#include <rte_errno.h>
#include <rte_string_fns.h>
#include <rte_tailq.h>
+#include <rte_telemetry.h>
#include "rte_ring.h"
#include "rte_ring_elem.h"
@@ -419,3 +420,42 @@ rte_ring_lookup(const char *name)
return r;
}
+
+static void
+rte_ring_walk(void (*func)(struct rte_ring *, void *), void *arg)
+{
+ struct rte_ring_list *ring_list;
+ struct rte_tailq_entry *te;
+
+ ring_list = RTE_TAILQ_CAST(rte_ring_tailq.head, rte_ring_list);
+ rte_mcfg_tailq_read_lock();
+
+ TAILQ_FOREACH(te, ring_list, next) {
+ (*func)((struct rte_ring *) te->data, arg);
+ }
+
+ rte_mcfg_tailq_read_unlock();
+}
+
+static void
+ring_list_cb(struct rte_ring *r, void *arg)
+{
+ struct rte_tel_data *d = (struct rte_tel_data *)arg;
+
+ rte_tel_data_add_array_string(d, r->name);
+}
+
+static int
+ring_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_ring_walk(ring_list_cb, d);
+ return 0;
+}
+
+RTE_INIT(ring_init_telemetry)
+{
+ rte_telemetry_register_cmd("/ring/list", ring_handle_list,
+ "Returns list of available ring. Takes no parameters");
+}