Skip to content

stats: add support for ring buffer type#345

Merged
ondrej-fabry merged 1 commit intoFDio:masterfrom
hsandid:ring-buffer-stats
Mar 19, 2026
Merged

stats: add support for ring buffer type#345
ondrej-fabry merged 1 commit intoFDio:masterfrom
hsandid:ring-buffer-stats

Conversation

@hsandid
Copy link
Copy Markdown
Contributor

@hsandid hsandid commented Mar 5, 2026

Hello all,

This PR adds support for ring buffers entries recently introduced as part of VPP stats-segment (https://gerrit.fd.io/r/c/vpp/+/43471).

This updates statseg_v2 to be able to fetch ring-buffer information (config, metadata), as well as return ring-buffer entries in raw bytes format. The stats-client can also be now be used to poll/dump ring-buffer information.

➜  bin/stats-client dump /test/ring-buffer
Dumping stats.. /test/ring-buffer
 - /test/ring-buffer                                               RingBuffer
  config: entry_size=258, ring_size=4096, threads=3, schema_version=1, schema_size=45
  thread[0]: head=29 seq=29 schema_version=1
  thread[1]: head=13 seq=13 schema_version=1
  thread[2]: head=8 seq=8 schema_version=1
Dumped 1 (1) stats

@ondrej-fabry ondrej-fabry added the feature New Feature label Mar 5, 2026
@ondrej-fabry
Copy link
Copy Markdown
Member

Can you update this PR by reabasing on master - to resolve conflicts.

@hsandid hsandid force-pushed the ring-buffer-stats branch from ef91daf to 6e875d8 Compare March 19, 2026 14:25
* Support for ring buffers entries in the VPP stats-segment
was recently introduced (https://gerrit.fd.io/r/c/vpp/+/43471)

* This PR introduces support in the statseg_v2 for consuming
ring-buffer entries (config, metadata, data), and also
displaying ring-buffer information when dumping through
the stats-client.

Signed-off-by: Hadi Sandid <hadisandid@gmail.com>
@hsandid hsandid force-pushed the ring-buffer-stats branch from 6e875d8 to ff8cf37 Compare March 19, 2026 14:36
@hsandid
Copy link
Copy Markdown
Contributor Author

hsandid commented Mar 19, 2026

@ondrej-fabry Done - thanks for the heads-up

@ondrej-fabry ondrej-fabry merged commit 7dce601 into FDio:master Mar 19, 2026
11 checks passed
@ondrej-fabry
Copy link
Copy Markdown
Member

Thanks for the contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New Feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants