On 09/03/2014 01:04 AM, Francesco Romani wrote: >> >> Missing is virsh exposure of the new stat groups (Li's series gave an >> example for adding --block). > > Sure, will add with a followup patch. > One question, though. > Do we want default to expose nothing, and options to add groups, is this > correct? Correct - the API is already wired that way (stats == 0 implies list all stats; non-zero stats says list just the requested stats). > > So usage would be something like > > virsh domstats # equivalent to --all , I suppose Yes, except I don't think we need an actual --all flag. > virsh domstats --block # only block group > virsh domstats --interface --vcpu # only vcpu and interface Yep, looks right to me :) -- 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