]> git.ipfire.org Git - thirdparty/libvirt.git/commit
qemu: improve errors related to offline domains
authorEric Blake <eblake@redhat.com>
Thu, 26 Apr 2012 19:36:01 +0000 (13:36 -0600)
committerCole Robinson <crobinso@redhat.com>
Fri, 27 Apr 2012 18:24:30 +0000 (14:24 -0400)
commit05cee1a9abda860559ca8b8db54b56536e38ee72
tree8e5ad0e299e846f7924822dd4dd695c5f88c5d00
parent54c3a530c7751ac4b83ad233daa6ef9c80c05b62
qemu: improve errors related to offline domains

https://bugzilla.redhat.com/show_bug.cgi?id=816662 pointed out
that attempting 'virsh blockpull' on an offline domain gave a
misleading error message about qemu lacking support for the
operation, even when qemu was specifically updated to support it.
The real problem is that we have several capabilities that are
only determined when starting a domain, and therefore are still
clear when first working with an inactive domain (namely, any
capability set by qemuMonitorJSONCheckCommands).

While this patch was able to hoist an existing check in one of the
three culprits, it had to add redundant checks in the other two
places (because you always have to check for an active domain after
obtaining a VM job lock, but the capability bits were being checked
prior to obtaining the job lock).

Someday it would be nice to patch libvirt to cache the set of
capabilities per qemu binary (as determined by inode and timestamp),
rather than re-probing the binary every time a domain is started,
and to teach the cache how to query the monitor during the one
time the probe is made rather than having to wait until a guest
is started; then, a capability probe would succeed even for offline
guests because it just refers to the cache, and the single check for
an active domain after grabbing the job lock would be sufficient.
But since that will involve a lot more coding, I'm happy to go
with this simpler solution for an immediate solution.

* src/qemu/qemu_driver.c (qemuDomainPMSuspendForDuration)
(qemuDomainSnapshotCreateXML, qemuDomainBlockJobImpl): Check for
offline state before checking an online-only cap.

Conflicts:

src/qemu/qemu_driver.c
src/qemu/qemu_driver.c