[dpdk-dev] [v3, 0/3] common/cnxk: enable npa telemetry
Power, Ciara
ciara.power at intel.com
Wed Aug 11 17:59:33 CEST 2021
Hi Gowrishankar,
>-----Original Message-----
>From: Gowrishankar Muthukrishnan <gmuthukrishn at marvell.com>
>Sent: Tuesday 3 August 2021 09:06
>To: dev at dpdk.org
>Cc: Richardson, Bruce <bruce.richardson at intel.com>; Power, Ciara
><ciara.power at intel.com>; jerinj at marvell.com; kirankumark at marvell.com;
>ndabilpuram at marvell.com; skori at marvell.com; skoteshwar at marvell.com;
>Gowrishankar Muthukrishnan <gmuthukrishn at 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
More information about the dev
mailing list