Re: [PATCH] Fix error reporting with virDomainBlockStats on inactive guest

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

 



On 06/02/2011 07:58 AM, Daniel P. Berrange wrote:
> From: "Daniel P. Berrange" <berrange@xxxxxxxxxx>
> 
> Pull the check on virDomainIsActive upto top of the QEMU

s/upto/up to/

> impl of virDomainBlockStats so that is is reported promptly
> 
> * src/qemu/qemu_driver.c: Check if guest is active immediately
> ---
>  src/qemu/qemu_driver.c |    6 ++++++
>  1 files changed, 6 insertions(+), 0 deletions(-)

This means that we are now checking multiple times (once on entry for
immediate effect even if migration is under way, and again after
acquiring job condition if migration is not under way), but that
shouldn't hurt.

However, you also need to do the same thing for qemuDomainGetBlockInfo,
since both methods were given the same control flow during commit 18c2a592.

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-801-349-2682
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

[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]