Re: [dpdk-dev] [v3, 0/3] common/cnxk: enable npa telemetry

["Power, Ciara" <ciara.power@intel.com>]

Hi Gowrishankar,

>-----Original Message-----
>From: Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com>
>Sent: Tuesday 3 August 2021 09:06
>To: dev@dpdk.org
>Cc: Richardson, Bruce <bruce.richardson@intel.com>; Power, Ciara
><ciara.power@intel.com>; jerinj@marvell.com; kirankumark@marvell.com;
>ndabilpuram@marvell.com; skori@marvell.com; skoteshwar@marvell.com;
>Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com>
>Subject: [v3, 0/3] common/cnxk: enable npa telemetry
>
>This patch series enables telemetry in NPA LF of cnxk.
>
>v3:
> - fixed format specifier for uintptr_t
>
>Gowrishankar Muthukrishnan (3):
>  telemetry: enable storing pointer value
>  test/telemetry: add unit tests for pointer value
>  common/cnxk: add telemetry endpoints to npa
>
> app/test/test_telemetry_data.c           | 125 +++++++++++++
> app/test/test_telemetry_json.c           |  29 ++-
> drivers/common/cnxk/cnxk_telemetry.h     |  26 +++
> drivers/common/cnxk/cnxk_telemetry_npa.c | 227
>+++++++++++++++++++++++
> drivers/common/cnxk/meson.build          |   4 +
> drivers/common/cnxk/roc_platform.h       |   8 +
> lib/telemetry/rte_telemetry.h            |  37 +++-
> lib/telemetry/telemetry.c                |  21 ++-
> lib/telemetry/telemetry_data.c           |  40 +++-
> lib/telemetry/telemetry_data.h           |   2 +
> lib/telemetry/telemetry_json.h           |  32 ++++
> lib/telemetry/version.map                |   2 +
> 12 files changed, 539 insertions(+), 14 deletions(-)  create mode 100644
>drivers/common/cnxk/cnxk_telemetry.h
> create mode 100644 drivers/common/cnxk/cnxk_telemetry_npa.c
>
>--
>2.25.1

I am still unsure exactly what the use case is here - why are we choosing to publish the pointer values through telemetry rather than using a debug log for example?
Maybe I am missing something here.

Thanks,
Ciara