On 12/08/2014 07:17 AM, Peter Krempa wrote:
On 12/06/14 09:14, Eric Blake wrote:
> I'm about to make block stats optionally more complex to cover
> backing chains, where block.count will no longer equal the number
> of <disks> for a domain. For these reasons, it is nicer if the
> statistics output includes the source path (for local files).
> This patch doesn't add anything for network disks, although we
> may decide to add that later.
>
> With this patch, I now see the following for the same domain as
> in earlier patches (one qcow2 file, and an empty cdrom drive):
> $ virsh domstats --block foo
> Domain: 'foo'
> block.count=2
> block.0.name=hda
> block.0.path=/var/lib/libvirt/images/foo.qcow2
> block.0.allocation=200704
> block.0.capacity=42949672960
> block.0.physical=200704
> block.1.name=hdc
ACK,
2, 6, and 8 now pushed. Of course, the commit message for 8 had to be
tweaked a bit (since the output is waiting on the offline stats of patch
7, which in turn are waiting on the split of offline vs. online in the
helper to blockinfo).
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library
http://libvirt.org