[PATCH 00/11] Fix vm usage after ExitMonitor

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

 



Check if the domain died while we were in the monitor
in most of the places (I didn't look at the Snapshot and Blockjob
APIs yet).
In some places, we've been accessing the data after it could've been
freed by qemuProcessStop (if the monitor API itself returned success,
but the domain was cleaned up because somehow qemuProcessStop
got the vm lock before the ExitMonitor call).

A more likely crash happens if qemu crashes in one of the APIs
with multiple Monitor calls, like the DetachDevice APIs
and AttachCharDevice.

Also, some occurences of writing the status XML for a dead domain
are fixed.

https://bugzilla.redhat.com/show_bug.cgi?id=1161024

Ján Tomko (11):
  Introduce qemuDomainObjExitMonitorAlive
  Mark the domain as active in qemuhotplugtest
  Fix vm usage after ExitMonitor in UpdateDeviceList
  Fix vm usage after ExitMonitor on device removal
  Fix error message on redirdev caps detection
  Fix vm usage after ExitMonitor on AttachDevice
  Fix vm usage after ExitMonitor in DetachDevice
  Fix monitor usage in qemuDomainHotplugVcpus
  Fix vm usage after ExitMonitor in qemu driver
  Fix vm usage after ExitMonitor in qemu migration
  Fix vm usage after ExitMonitor in qemu process

 src/qemu/qemu_command.c   |   6 +-
 src/qemu/qemu_domain.c    |  20 +++++-
 src/qemu/qemu_domain.h    |   3 +
 src/qemu/qemu_driver.c    |  49 ++++++++++----
 src/qemu/qemu_hotplug.c   | 169 +++++++++++++++++++++++++++++++---------------
 src/qemu/qemu_migration.c |  41 +++++++----
 src/qemu/qemu_process.c   |  65 ++++++++++++------
 tests/qemuhotplugtest.c   |   4 ++
 8 files changed, 251 insertions(+), 106 deletions(-)

-- 
2.0.4

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