[libvirt] [PATCH 0/2] qemu: Fix incorrect qemuDomainGetBlockIoTune() results

Eric Blake eblake at redhat.com
Thu Nov 1 19:02:09 UTC 2012


On 11/01/2012 11:20 AM, Stefan Hajnoczi wrote:
> The qemuDomainGetBlockIoTune() function was returning incorrect results as I
> discovered on Fedora Virt Test Day:
> https://fedoraproject.org/wiki/QA:Testcase_Virtualization_IO_Throttling
> 
> The problem is that I/O limits can be set but I only see zeroes when displaying
> them with "virsh blkdeviotune <domain> <device>" on a domain with two disks.
> 
> First, we're comparing guest device names ("virtio-blk-0") instead of host
> device names ("drive-virtio-blk-0").  Second, the JSON monitor code has
> inverted string compare logic so we were skipping the device we were looking
> for.
> 
> These patches make "virsh blkdeviotune" report correct results.
> 
> Stefan Hajnoczi (2):
>   qemu: Keep QEMU host drive prefix in BlkIoTune
>   qemu: Fix device name comparison in
>     qemuMonitorJSONBlockIoThrottleInfo()

ACK series, and I'll push shortly.  Nice that the test day is just
before the 1.0.0 release!  I'll also push into the v0.10.2-maint branch,
so this will hit the next F18 build of libvirt.

-- 
Eric Blake   eblake at redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 617 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20121101/920ec2cd/attachment-0001.sig>


More information about the libvir-list mailing list