mbox series

[v9,0/3] add basic ethdev stats with data object recursion

Message ID 20200923111228.6274-1-ciara.power@intel.com (mailing list archive)
Headers show
Series add basic ethdev stats with data object recursion | expand

Message

Power, Ciara Sept. 23, 2020, 11:12 a.m. UTC
v9: Rebased onto main to remove conflict.
v8: Rebased onto main.
v7:
  - Simplified connecting to socket by removing use of glob.
  - Fixed buffer overflow issue when reading from socket.
  - Split expected response strings over multiple lines.
v6:
  - Fixed FreeBSD build failure for unit tests.
  - Added comments and expanded commit log.
  - Add loop to call test cases stored in a list.
v5: Added unit tests for telemetry data to JSON conversion.
v4: Added missing param description to Doxygen comment.
v3:
  - Modified commit logs.
  - Changed function names to reference container.
  - Modified Doxygen comments to reference container.
v2:
  - Added support for arrays to have container values.
  - Added support for int and string arrays within dict/array.
  - Added APIs for internal container memory management.

This patchset adds support for basic ethdev statistics in Telemetry.
To do this, recursive data object support is needed to report the queue
statistics in a list. With this patch, an array or dictionary supports
uint64_t, int or string array type values, which is used for the ethdev
queue stats.

Ciara Power (2):
  telemetry: support array values in data objects
  ethdev: add common stats for telemetry

Louise Kilheeney (1):
  test/test_telemetry_data: add unit tests for data to JSON

 app/test/meson.build                          |   5 +-
 app/test/test_telemetry_data.c                | 369 ++++++++++++++++++
 lib/librte_ethdev/rte_ethdev.c                |  53 +++
 lib/librte_telemetry/rte_telemetry.h          |  70 ++++
 .../rte_telemetry_version.map                 |   4 +
 lib/librte_telemetry/telemetry.c              |  56 +++
 lib/librte_telemetry/telemetry_data.c         |  51 +++
 lib/librte_telemetry/telemetry_data.h         |   7 +
 lib/librte_telemetry/telemetry_json.h         |  33 ++
 9 files changed, 646 insertions(+), 2 deletions(-)
 create mode 100644 app/test/test_telemetry_data.c

Comments

Thomas Monjalon Oct. 6, 2020, 8:56 p.m. UTC | #1
> This patchset adds support for basic ethdev statistics in Telemetry.
> To do this, recursive data object support is needed to report the queue
> statistics in a list. With this patch, an array or dictionary supports
> uint64_t, int or string array type values, which is used for the ethdev
> queue stats.

Applied, thanks