[PATCH v4 0/3] add telemetry cmds for ring

fengchengwen fengchengwen at huawei.com
Wed Feb 15 04:04:21 CET 2023


LGTM
Series-acked-by: Chengwen Feng <fengchengwen at huawei.com>

On 2023/2/10 10:48, Jie Hai wrote:
> This patch set supports telemetry cmd to list rings and dump information
> of a ring by its name.
> 
> v1->v2:
> 1. Add space after "switch".
> 2. Fix wrong strlen parameter.
> 
> v2->v3:
> 1. Remove prefix "rte_" for static function.
> 2. Add Acked-by Konstantin Ananyev for PATCH 1.
> 3. Introduce functions to return strings instead copy strings.
> 4. Check pointer to memzone of ring.
> 5. Remove redundant variable.
> 6. Hold lock when access ring data.
> 
> v3->v4:
> 1. Update changelog according to reviews of Honnappa Nagarahalli.
> 2. Add Reviewed-by Honnappa Nagarahalli.
> 3. Correct grammar in help information.
> 4. Correct spell warning on "te" reported by checkpatch.pl.
> 5. Use ring_walk() to query ring info instead of rte_ring_lookup().
> 6. Fix that type definition the flag field of rte_ring does not match the usage.
> 7. Use rte_tel_data_add_dict_uint_hex instead of rte_tel_data_add_dict_u64
>    for mask and flags.
> 
> Jie Hai (3):
>   ring: fix unmatched type definition and usage
>   ring: add telemetry cmd to list rings
>   ring: add telemetry cmd for ring info
> 
>  lib/ring/meson.build     |   1 +
>  lib/ring/rte_ring.c      | 139 +++++++++++++++++++++++++++++++++++++++
>  lib/ring/rte_ring_core.h |   2 +-
>  3 files changed, 141 insertions(+), 1 deletion(-)
> 


More information about the dev mailing list