External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2024-12-29 00:55:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

qemu: Don't report error on successful media eject

Browse Source 
If we are just ejecting media, ret == -1 even after the retry loop
determines that the tray is open, as requested. This means media
disconnect always report's error.

Fix it, and fix some other mini issues:

- Don't overwrite the 'eject' error message if the retry loop fails
- Move the retries decrement inside the loop, otherwise the final loop
  might succeed, yet retries == 0 and we will raise error
- Setting ret = -1 in the disk->src check is unneeded
- Fix comment typos

cc: mprivozn@redhat.com
(cherry picked from commit 406d8a9809)
This commit is contained in:
Cole Robinson 2013-05-28 10:23:00 -04:00
parent a93f2757e2
commit f8085a5471
1 changed files with 14 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
src/qemu/qemu_hotplug.c
View File

@ -109,10 +109,11 @@ int qemuDomainChangeEjectableMedia(struct qemud_driver *driver,
virObjectRef(vm);
/* we don't want to report errors from media tray_open polling */
while (retries--) {
while (retries) {
if (origdisk->tray_status == VIR_DOMAIN_DISK_TRAY_OPEN)
break;
retries--;
virDomainObjUnlock(vm);
VIR_DEBUG("Waiting 500ms for tray to open. Retries left %d", retries);
usleep(500 * 1000); /* sleep 500ms */
@ -120,19 +121,20 @@ int qemuDomainChangeEjectableMedia(struct qemud_driver *driver,
}
virObjectUnref(vm);
if (disk->src) {
/* deliberately don't depend on 'ret' as 'eject' may have failed for the
* fist time and we are gonna check the drive state anyway */
const char *format = NULL;
/* We haven't succeeded yet */
ret = -1;
if (retries <= 0) {
if (retries <= 0) {
if (ret == 0) {
/* If ret == -1, EjectMedia already set an error message */
virReportError(VIR_ERR_OPERATION_FAILED, "%s",
_("Unable to eject media before changing it"));
goto audit;
_("Unable to eject media"));
}
goto audit;
}
ret = 0;
if (disk->src) {
/* deliberately don't depend on 'ret' as 'eject' may have failed the
* first time and we are going to check the drive state anyway */
const char *format = NULL;
if (disk->type != VIR_DOMAIN_DISK_TYPE_DIR) {
if (disk->driverType)