Hi Gowrishankar, >-----Original Message----- >From: Gowrishankar Muthukrishnan <gmuthukri...@marvell.com> >Sent: Tuesday 3 August 2021 09:06 >To: dev@dpdk.org >Cc: Richardson, Bruce <bruce.richard...@intel.com>; Power, Ciara ><ciara.po...@intel.com>; jer...@marvell.com; kirankum...@marvell.com; >ndabilpu...@marvell.com; sk...@marvell.com; skotesh...@marvell.com; >Gowrishankar Muthukrishnan <gmuthukri...@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
