On 25/08/2022 08:47, Morten Brørup wrote:
From: Kevin Laatz [mailto:kevin.la...@intel.com]
Sent: Wednesday, 24 August 2022 18.25

Currently, there is no way to measure lcore busyness in a passive way,
without any modifications to the application. This patchset adds a new
EAL
API that will be able to passively track core busyness. As part of the
set,
new telemetry endpoints are added to read the generate metrics.

Anatoly Burakov (2):
   eal: add lcore poll busyness telemetry
   eal: add cpuset lcore telemetry entries

Kevin Laatz (1):
   doc: add howto guide for lcore poll busyness

At first glance, reading the commit messages, I thought you had ignored the 
feedback.
However, I see that V2 adds "poll" in front of "busyness".

I'll make changes to the message to clarify this a bit better.

It's just missing in many locations in the code, e.g. in eal_common_lcore_telemetry.c, 
the telemetry command is still registered as "/eal/lcore/busyness".

Seems some builds are failing in the CI so I'll send a v3 with fixes shortly - I'll do another sweep before sending to make sure there are no omissions.

-Kevin

Reply via email to