>-----Original Message----- >From: Thomas Monjalon <tho...@monjalon.net> >Sent: Monday, July 31, 2023 2:33 PM >To: Tomasz Duszynski <tduszyn...@marvell.com> >Cc: dev@dpdk.org; roret...@linux.microsoft.com; ruifeng.w...@arm.com; >bruce.richard...@intel.com; >Jerin Jacob Kollanukkaran <jer...@marvell.com>; mattias.ronnb...@ericsson.com; >m...@smartsharesystems.com; zhou...@loongson.cn; Konstantin Ananyev ><konstantin.anan...@huawei.com>; David Marchand <david.march...@redhat.com> >Subject: [EXT] Re: [PATCH v11 0/4] add support for self monitoring > >External Email > >---------------------------------------------------------------------- >Ping for update >What is the status of this feature? > >
Hi Thomas, I'll re-spin the series soon. >04/05/2023 10:02, David Marchand: >> Hello Tomasz, >> >> On Thu, Feb 16, 2023 at 6:55 PM Tomasz Duszynski <tduszyn...@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? > > > >
