Thread (121 messages) 121 messages, 10 authors, 2021-11-04

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

From: Power, Ciara <hidden>
Date: 2021-08-11 15:59:42

Hi Gowrishankar,
-----Original Message-----
From: Gowrishankar Muthukrishnan <redacted>
Sent: Tuesday 3 August 2021 09:06
To: dev@dpdk.org
Cc: Richardson, Bruce <redacted>; Power, Ciara
[off-list ref]; jerinj@marvell.com; kirankumark@marvell.com;
ndabilpuram@marvell.com; skori@marvell.com; skoteshwar@marvell.com;
Gowrishankar Muthukrishnan [off-list ref]
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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help