Re: [PATCH v11 0/4] add support for self monitoring

[Thomas Monjalon <thomas@monjalon.net>]

Ping for update
What is the status of this feature?


04/05/2023 10:02, David Marchand:
> Hello Tomasz,
> 
> On Thu, Feb 16, 2023 at 6:55 PM Tomasz Duszynski <tduszynski@marvell.com> wrote:
> >
> > This series adds self monitoring support i.e allows to configure and
> > read performance measurement unit (PMU) counters in runtime without
> > using perf utility. This has certain advantages when application runs on
> > isolated cores running dedicated tasks.
> >
> > Events can be read directly using rte_pmu_read() or using dedicated
> > tracepoint rte_eal_trace_pmu_read(). The latter will cause events to be
> > stored inside CTF file.
> >
> > By design, all enabled events are grouped together and the same group
> > is attached to lcores that use self monitoring funtionality.
> >
> > Events are enabled by names, which need to be read from standard
> > location under sysfs i.e
> >
> > /sys/bus/event_source/devices/PMU/events
> >
> > where PMU is a core pmu i.e one measuring cpu events. As of today
> > raw events are not supported.
> >
> > Tomasz Duszynski (4):
> >   lib: add generic support for reading PMU events
> >   pmu: support reading ARM PMU events in runtime
> >   pmu: support reading Intel x86_64 PMU events in runtime
> >   eal: add PMU support to tracing library
> 
> There are still some pending comments on this series and it can't be
> merged until they get sorted out.
> 
> I noted two points :
> - Konstantin asked for better explanations in the implementation.
> - He also pointed out at using this feature with non EAL lcores.
> 
> Could you work on this so we can consider this series for v23.07?