Re: [PATCH v3 0/2] qemu: expand domain memory statistics

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



26-Jul-16 18:32, Pavel Hrdina пишет:

On Sat, Jun 04, 2016 at 06:41:35PM +0300, Maxim Nestratov wrote:
01.06.2016 20:07, Derbyshev Dmitriy пишет:

From: Derbyshev Dmitry <dderbyshev@xxxxxxxxxxxxx>

QEMU reports timestamp and available along with other memory statistics.
This information was not saved into domain statistics.

Changes since v1:
   * Enum numeration fixed
   * Macro getting "usage" field fixed
Changes since v2:
   * previous patches were on wrong branch
   * qemu's stat name was "stat-available-memory"

Derbyshev Dmitry (2):
    qemu: expand domain memory statistics with 'usable'
    qemu: expand domain memory statistics with 'last-update' timestamp

   include/libvirt/libvirt-domain.h | 11 ++++++++++-
   src/libvirt-domain.c             |  5 +++++
   src/qemu/qemu_monitor_json.c     | 22 +++++++++++++---------
   tools/virsh-domain-monitor.c     |  4 ++++
   4 files changed, 32 insertions(+), 10 deletions(-)

Makes sense, looks good to me.
ACK the series.
Pushed now.  I've just found out that this wasn't pushed so far.

Pavel

Yeah, and in fact the work has been continued in another patchset version though no one replied to this one. It even reached v7 https://www.redhat.com/archives/libvir-list/2016-July/msg00428.html and should be adapted now because these two are pushed.
Anyway, thanks for pushing.

Maxim

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]