On 09/15/2014 09:42 AM, Peter Krempa wrote: > From: Francesco Romani <fromani@xxxxxxxxxx> > > This patch implements the VIR_DOMAIN_STATS_CPU_TOTAL group of > statistics. > > Signed-off-by: Francesco Romani <fromani@xxxxxxxxxx> > Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx> > --- > > Notes: > Version 6: > - added a check that skips the stats group in case cgroups aren't available for the given domain > > include/libvirt/libvirt.h.in | 1 + > src/libvirt.c | 7 +++++++ > src/qemu/qemu_driver.c | 44 ++++++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 52 insertions(+) > > +++ b/src/libvirt.c > @@ -21596,6 +21596,13 @@ virConnectGetDomainCapabilities(virConnectPtr conn, > * "state.reason" - reason for entering given state, returned as int from > * virDomain*Reason enum corresponding to given state. > * > + * VIR_DOMAIN_STATS_CPU_TOTAL: Return CPU statistics and usage information. > + * The typed parameter keys are in this format: > + * "cpu.time" - total cpu time spent for this domain as unsigned long long. > + * "cpu.user" - user cpu time spent as unsigned long long. > + * "cpu.system" - system cpu time spent as unsigned long long. Missing the word "nanoseconds" on each of these three (at least, I'm assuming that's the unit, based on VIR_DOMAIN_CPU_STATS_CPUTIME). ACK with that fixed. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list