Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx> --- src/qemu/qemu_monitor.c | 33 --------------- src/qemu/qemu_monitor.h | 6 --- src/qemu/qemu_monitor_text.c | 82 ------------------------------------ src/qemu/qemu_monitor_text.h | 6 --- 4 files changed, 127 deletions(-) diff --git a/src/qemu/qemu_monitor.c b/src/qemu/qemu_monitor.c index 456c54aed3..93890c43be 100644 --- a/src/qemu/qemu_monitor.c +++ b/src/qemu/qemu_monitor.c @@ -2605,26 +2605,6 @@ qemuMonitorGetChardevInfo(qemuMonitor *mon, } -/** - * qemuMonitorDriveDel: - * @mon: monitor object - * @drivestr: identifier of drive to delete. - * - * Attempts to remove a host drive. - * Returns 1 if unsupported, 0 if ok, and -1 on other failure */ -int -qemuMonitorDriveDel(qemuMonitor *mon, - const char *drivestr) -{ - VIR_DEBUG("drivestr=%s", drivestr); - - QEMU_CHECK_MONITOR(mon); - - /* there won't be a direct replacement for drive_del in QMP */ - return qemuMonitorTextDriveDel(mon, drivestr); -} - - /** * @mon: monitor object * @devalias: alias of the device to detach @@ -2796,19 +2776,6 @@ qemuMonitorDelObject(qemuMonitor *mon, } -int -qemuMonitorAddDrive(qemuMonitor *mon, - const char *drivestr) -{ - VIR_DEBUG("drive=%s", drivestr); - - QEMU_CHECK_MONITOR(mon); - - /* there won't ever be a direct QMP replacement for this function */ - return qemuMonitorTextAddDrive(mon, drivestr); -} - - int qemuMonitorCreateSnapshot(qemuMonitor *mon, const char *name) { diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monitor.h index 163e12c0f6..830be5418c 100644 --- a/src/qemu/qemu_monitor.h +++ b/src/qemu/qemu_monitor.h @@ -955,12 +955,6 @@ int qemuMonitorDelObject(qemuMonitor *mon, const char *objalias, bool report_error); -int qemuMonitorAddDrive(qemuMonitor *mon, - const char *drivestr); - -int qemuMonitorDriveDel(qemuMonitor *mon, - const char *drivestr); - int qemuMonitorCreateSnapshot(qemuMonitor *mon, const char *name); int qemuMonitorDeleteSnapshot(qemuMonitor *mon, const char *name); diff --git a/src/qemu/qemu_monitor_text.c b/src/qemu/qemu_monitor_text.c index 65785f1dae..8e70b8f78d 100644 --- a/src/qemu/qemu_monitor_text.c +++ b/src/qemu/qemu_monitor_text.c @@ -31,88 +31,6 @@ VIR_LOG_INIT("qemu.qemu_monitor_text"); -int qemuMonitorTextAddDrive(qemuMonitor *mon, - const char *drivestr) -{ - g_autofree char *cmd = NULL; - g_autofree char *reply = NULL; - - /* 'dummy' here is just a placeholder since there is no PCI - * address required when attaching drives to a controller */ - cmd = g_strdup_printf("drive_add dummy %s", drivestr); - - if (qemuMonitorJSONHumanCommand(mon, cmd, -1, &reply) < 0) - return -1; - - if (strstr(reply, "unknown command:")) { - virReportError(VIR_ERR_OPERATION_FAILED, "%s", - _("drive hotplug is not supported")); - return -1; - } - - if (strstr(reply, "could not open disk image")) { - virReportError(VIR_ERR_OPERATION_FAILED, "%s", - _("open disk image file failed")); - return -1; - } - - if (strstr(reply, "Could not open")) { - size_t len = strlen(reply); - if (reply[len - 1] == '\n') - reply[len - 1] = '\0'; - - virReportError(VIR_ERR_OPERATION_FAILED, "%s", - reply); - return -1; - } - - if (strstr(reply, "Image is not in")) { - virReportError(VIR_ERR_OPERATION_FAILED, "%s", - _("Incorrect disk format")); - return -1; - } - - if (strstr(reply, "IOMMU") || - strstr(reply, "VFIO")) { - virReportError(VIR_ERR_OPERATION_FAILED, "%s", - reply); - return -1; - } - - return 0; -} - - -int qemuMonitorTextDriveDel(qemuMonitor *mon, - const char *drivestr) -{ - g_autofree char *cmd = NULL; - g_autofree char *reply = NULL; - - cmd = g_strdup_printf("drive_del %s", drivestr); - - if (qemuMonitorJSONHumanCommand(mon, cmd, -1, &reply) < 0) - return -1; - - if (strstr(reply, "unknown command:")) { - VIR_ERROR(_("deleting drive is not supported. " - "This may leak data if disk is reassigned")); - return 1; - - /* (qemu) drive_del wark - * Device 'wark' not found */ - } else if (strstr(reply, "Device '") && strstr(reply, "not found")) { - /* NB: device not found errors mean the drive was auto-deleted and we - * ignore the error */ - } else if (STRNEQ(reply, "")) { - virReportError(VIR_ERR_OPERATION_FAILED, - _("deleting %s drive failed: %s"), drivestr, reply); - return -1; - } - - return 0; -} - int qemuMonitorTextCreateSnapshot(qemuMonitor *mon, const char *name) diff --git a/src/qemu/qemu_monitor_text.h b/src/qemu/qemu_monitor_text.h index d959fc8889..27d0f061d3 100644 --- a/src/qemu/qemu_monitor_text.h +++ b/src/qemu/qemu_monitor_text.h @@ -25,11 +25,5 @@ #include "qemu_monitor.h" -int qemuMonitorTextAddDrive(qemuMonitor *mon, - const char *drivestr); - -int qemuMonitorTextDriveDel(qemuMonitor *mon, - const char *drivestr); - int qemuMonitorTextCreateSnapshot(qemuMonitor *mon, const char *name); int qemuMonitorTextDeleteSnapshot(qemuMonitor *mon, const char *name); -- 2.36.1