External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2025-01-16 17:45:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

qemuDomainChangeEjectableMedia: Unlock domain while waiting for event

Browse Source 
In 84c59ffa I've tried to fix changing ejectable media process. The
process should go like this:

1) we need to call 'eject' on the monitor
2) we should wait for 'DEVICE_TRAY_MOVED' event
3) now we can issue 'change' command

However, while waiting in step 2) the domain monitor was locked. So
even if qemu reported the desired event, the proper callback was not
called immediately. The monitor handling code needs to lock the
monitor in order to read the event. So that's the first lock we must
not hold while waiting. The second one is the domain lock. When
monitor handling code reads an event, the appropriate callback is
called then. The first thing that each callback does is locking the
corresponding domain as a domain or its device is about to change
state. So we need to unlock both monitor and VM lock. Well, holding
any lock while sleep()-ing is not the best thing to do anyway.
(cherry picked from commit 543af79a14f06cd16844c28887210bbb93a455fa)

Conflicts:
	src/qemu/qemu_hotplug.c
This commit is contained in:
Michal Privoznik 2013-05-20 19:26:14 +02:00 committed by Cole Robinson
parent 7c4b6833bb
commit a93f2757e2
1 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/qemu/qemu_hotplug.c
View File

@ -105,15 +105,20 @@ int qemuDomainChangeEjectableMedia(struct qemud_driver *driver,
qemuDomainObjEnterMonitorWithDriver(driver, vm);
ret = qemuMonitorEjectMedia(priv->mon, driveAlias, force);
qemuDomainObjExitMonitor(driver, vm);
virObjectRef(vm);
/* we don't want to report errors from media tray_open polling */
while (retries--) {
if (origdisk->tray_status == VIR_DOMAIN_DISK_TRAY_OPEN)
break;
virDomainObjUnlock(vm);
VIR_DEBUG("Waiting 500ms for tray to open. Retries left %d", retries);
usleep(500 * 1000); /* sleep 500ms */
virDomainObjLock(vm);
}
virObjectUnref(vm);
if (disk->src) {
/* deliberately don't depend on 'ret' as 'eject' may have failed for the
@ -126,7 +131,7 @@ int qemuDomainChangeEjectableMedia(struct qemud_driver *driver,
if (retries <= 0) {
virReportError(VIR_ERR_OPERATION_FAILED, "%s",
_("Unable to eject media before changing it"));
goto exit_monitor;
goto audit;
}
if (disk->type != VIR_DOMAIN_DISK_TYPE_DIR) {
@ -135,13 +140,14 @@ int qemuDomainChangeEjectableMedia(struct qemud_driver *driver,
else if (origdisk->driverType)
format = origdisk->driverType;
}
qemuDomainObjEnterMonitor(driver, vm);
ret = qemuMonitorChangeMedia(priv->mon,
driveAlias,
disk->src, format);
qemuDomainObjExitMonitor(driver, vm);
}
exit_monitor:
qemuDomainObjExitMonitorWithDriver(driver, vm);
audit:
virDomainAuditDisk(vm, origdisk->src, disk->src, "update", ret >= 0);
if (ret < 0)