Hi Gowrishankar, >-----Original Message----- >From: Gowrishankar Muthukrishnan <[email protected]> >Sent: Tuesday 3 August 2021 09:06 >To: [email protected] >Cc: Richardson, Bruce <[email protected]>; Power, Ciara ><[email protected]>; [email protected]; [email protected]; >[email protected]; [email protected]; [email protected]; >Gowrishankar Muthukrishnan <[email protected]> >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
