[PATCH v2 0/2] qemu: fix and clean up resctrl monitor domstats

Jedrzej Wasiukiewicz posted 2 patches 1 week, 2 days ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20260904102322.583641-1-jedrzej.wasiukiewicz@intel.com
src/libvirt_private.syms |  1 +
src/qemu/qemu_driver.c   |  5 ++++-
src/util/virresctrl.c    | 10 ++++++++++
src/util/virresctrl.h    |  3 +++
4 files changed, 18 insertions(+), 1 deletion(-)
[PATCH v2 0/2] qemu: fix and clean up resctrl monitor domstats
Posted by Jedrzej Wasiukiewicz 1 week, 2 days ago
Fix two independent inconsistencies in resctrl monitor domstats
reporting. The first patch reports the actual resctrl monitor group name
instead of its internal ID. The second avoids reporting a zero cache
monitor count when no cache monitor data is available, matching the
memory bandwidth and energy collectors.

These patches were previously posted as part of the whole-process
resctrl series:

https://lists.libvirt.org/archives/list/devel@lists.libvirt.org/thread/QFXZZOHRSAUIUZYANHKRAXPBDQH2OSKM/

Changes since v1:
- Split the independent fixes from the whole-process feature series.
- Rebased onto current master.
- Added Christopher's Reviewed-by.
- Limited the domstats cleanup to the independent cache monitor fix.
  whole-process vCPU reporting remains in the feature series.

Jedrzej Wasiukiewicz (2):
  qemu: fix resctrl monitor stats reporting the wrong group name
  qemu: omit empty cache monitor domstats

 src/libvirt_private.syms |  1 +
 src/qemu/qemu_driver.c   |  5 ++++-
 src/util/virresctrl.c    | 10 ++++++++++
 src/util/virresctrl.h    |  3 +++
 4 files changed, 18 insertions(+), 1 deletion(-)

--
2.43.0
---------------------------------------------------------------------
Intel Technology Poland sp. z o.o.
ul. Slowackiego 173 | 80-298 Gdansk | Sad Rejonowy Gdansk Polnoc | VII Wydzial Gospodarczy Krajowego Rejestru Sadowego - KRS 101882 | NIP 957-07-52-316 | Kapital zakladowy 200.000 PLN.
Spolka oswiadcza, ze posiada status duzego przedsiebiorcy w rozumieniu ustawy z dnia 8 marca 2013 r. o przeciwdzialaniu nadmiernym opoznieniom w transakcjach handlowych.

Ta wiadomosc wraz z zalacznikami jest przeznaczona dla okreslonego adresata i moze zawierac informacje poufne. W razie przypadkowego otrzymania tej wiadomosci, prosimy o powiadomienie nadawcy oraz trwale jej usuniecie; jakiekolwiek przegladanie lub rozpowszechnianie jest zabronione.
This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are not the intended recipient, please contact the sender and delete all copies; any review or distribution by others is strictly prohibited.
Re: [PATCH v2 0/2] qemu: fix and clean up resctrl monitor domstats
Posted by Michal Prívozník via Devel 1 week, 2 days ago
On 9/4/26 12:23, Jedrzej Wasiukiewicz wrote:
> Fix two independent inconsistencies in resctrl monitor domstats
> reporting. The first patch reports the actual resctrl monitor group name
> instead of its internal ID. The second avoids reporting a zero cache
> monitor count when no cache monitor data is available, matching the
> memory bandwidth and energy collectors.
> 
> These patches were previously posted as part of the whole-process
> resctrl series:
> 
> https://lists.libvirt.org/archives/list/devel@lists.libvirt.org/thread/QFXZZOHRSAUIUZYANHKRAXPBDQH2OSKM/
> 
> Changes since v1:
> - Split the independent fixes from the whole-process feature series.
> - Rebased onto current master.
> - Added Christopher's Reviewed-by.
> - Limited the domstats cleanup to the independent cache monitor fix.
>   whole-process vCPU reporting remains in the feature series.
> 
> Jedrzej Wasiukiewicz (2):
>   qemu: fix resctrl monitor stats reporting the wrong group name
>   qemu: omit empty cache monitor domstats
> 
>  src/libvirt_private.syms |  1 +
>  src/qemu/qemu_driver.c   |  5 ++++-
>  src/util/virresctrl.c    | 10 ++++++++++
>  src/util/virresctrl.h    |  3 +++
>  4 files changed, 18 insertions(+), 1 deletion(-)
> 

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

and merged.

Michal