]> git.ipfire.org Git - thirdparty/libvirt.git/commit
Fix race condition reconnecting to vms & loading configs
authorDaniel P. Berrange <berrange@redhat.com>
Mon, 28 Oct 2013 11:49:18 +0000 (11:49 +0000)
committerCole Robinson <crobinso@redhat.com>
Wed, 6 Nov 2013 16:45:03 +0000 (11:45 -0500)
commitb044210ed236d74fe4c41a42fecf51173811daa7
tree686f6be97221954dc1234357a5823850a33aa225
parent5ddb57e01ef710f93dfe4ac782a83b74c5bff74c
Fix race condition reconnecting to vms & loading configs

The following sequence

 1. Define a persistent QMEU guest
 2. Start the QEMU guest
 3. Stop libvirtd
 4. Kill the QEMU process
 5. Start libvirtd
 6. List persistent guests

At the last step, the previously running persistent guest
will be missing. This is because of a race condition in the
QEMU driver startup code. It does

 1. Load all VM state files
 2. Spawn thread to reconnect to each VM
 3. Load all VM config files

Only at the end of step 3, does the 'virDomainObjPtr' get
marked as "persistent". There is therefore a window where
the thread reconnecting to the VM will remove the persistent
VM from the list.

The easy fix is to simply switch the order of steps 2 & 3.

In addition to this though, we must only attempt to reconnect
to a VM which had a non-zero PID loaded from its state file.

Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
(cherry picked from commit f26701f565525dd402df021d8923489e62412158)
src/qemu/qemu_driver.c
src/qemu/qemu_process.c