提交 fde060b8 编写于 作者: D Daniel P. Berrange

Rename parameter in qemuMonitorDeviceDel

The parameter for the qemuMonitorDeviceDel() is a device alias,
not a device config string. Rename the parameter reflect this
and avoid confusion to readers.

* src/qemu/qemu_monitor.c, src/qemu/qemu_monitor.h,
  src/qemu/qemu_monitor_json.c, src/qemu/qemu_monitor_json.h,
  src/qemu/qemu_monitor_text.c, src/qemu/qemu_monitor_text.h:
  Rename devicestr to devalias in qemuMonitorDeviceDel()
上级 efd4ee78
...@@ -1453,15 +1453,15 @@ int qemuMonitorGetAllPCIAddresses(qemuMonitorPtr mon, ...@@ -1453,15 +1453,15 @@ int qemuMonitorGetAllPCIAddresses(qemuMonitorPtr mon,
} }
int qemuMonitorDelDevice(qemuMonitorPtr mon, int qemuMonitorDelDevice(qemuMonitorPtr mon,
const char *devicestr) const char *devalias)
{ {
DEBUG("mon=%p, fd=%d device(del)=%s", mon, mon->fd, devicestr); DEBUG("mon=%p, fd=%d devalias=%s", mon, mon->fd, devalias);
int ret; int ret;
if (mon->json) if (mon->json)
ret = qemuMonitorJSONDelDevice(mon, devicestr); ret = qemuMonitorJSONDelDevice(mon, devalias);
else else
ret = qemuMonitorTextDelDevice(mon, devicestr); ret = qemuMonitorTextDelDevice(mon, devalias);
return ret; return ret;
} }
......
...@@ -338,7 +338,7 @@ int qemuMonitorAddDevice(qemuMonitorPtr mon, ...@@ -338,7 +338,7 @@ int qemuMonitorAddDevice(qemuMonitorPtr mon,
const char *devicestr); const char *devicestr);
int qemuMonitorDelDevice(qemuMonitorPtr mon, int qemuMonitorDelDevice(qemuMonitorPtr mon,
const char *devicestr); const char *devalias);
int qemuMonitorAddDrive(qemuMonitorPtr mon, int qemuMonitorAddDrive(qemuMonitorPtr mon,
const char *drivestr); const char *drivestr);
......
...@@ -1938,14 +1938,14 @@ int qemuMonitorJSONGetAllPCIAddresses(qemuMonitorPtr mon ATTRIBUTE_UNUSED, ...@@ -1938,14 +1938,14 @@ int qemuMonitorJSONGetAllPCIAddresses(qemuMonitorPtr mon ATTRIBUTE_UNUSED,
int qemuMonitorJSONDelDevice(qemuMonitorPtr mon, int qemuMonitorJSONDelDevice(qemuMonitorPtr mon,
const char *devicestr) const char *devalias)
{ {
int ret; int ret;
virJSONValuePtr cmd; virJSONValuePtr cmd;
virJSONValuePtr reply = NULL; virJSONValuePtr reply = NULL;
cmd = qemuMonitorJSONMakeCommand("device_del", cmd = qemuMonitorJSONMakeCommand("device_del",
"s:config", devicestr, "s:config", devalias,
NULL); NULL);
if (!cmd) if (!cmd)
return -1; return -1;
......
...@@ -169,7 +169,7 @@ int qemuMonitorJSONAddDevice(qemuMonitorPtr mon, ...@@ -169,7 +169,7 @@ int qemuMonitorJSONAddDevice(qemuMonitorPtr mon,
const char *devicestr); const char *devicestr);
int qemuMonitorJSONDelDevice(qemuMonitorPtr mon, int qemuMonitorJSONDelDevice(qemuMonitorPtr mon,
const char *devicestr); const char *devalias);
int qemuMonitorJSONAddDrive(qemuMonitorPtr mon, int qemuMonitorJSONAddDrive(qemuMonitorPtr mon,
const char *drivestr); const char *drivestr);
......
...@@ -2133,14 +2133,14 @@ error: ...@@ -2133,14 +2133,14 @@ error:
int qemuMonitorTextDelDevice(qemuMonitorPtr mon, int qemuMonitorTextDelDevice(qemuMonitorPtr mon,
const char *devicestr) const char *devalias)
{ {
char *cmd = NULL; char *cmd = NULL;
char *reply = NULL; char *reply = NULL;
char *safedev; char *safedev;
int ret = -1; int ret = -1;
if (!(safedev = qemuMonitorEscapeArg(devicestr))) { if (!(safedev = qemuMonitorEscapeArg(devalias))) {
virReportOOMError(); virReportOOMError();
goto cleanup; goto cleanup;
} }
...@@ -2152,13 +2152,13 @@ int qemuMonitorTextDelDevice(qemuMonitorPtr mon, ...@@ -2152,13 +2152,13 @@ int qemuMonitorTextDelDevice(qemuMonitorPtr mon,
if (qemuMonitorCommand(mon, cmd, &reply) < 0) { if (qemuMonitorCommand(mon, cmd, &reply) < 0) {
qemuReportError(VIR_ERR_OPERATION_FAILED, qemuReportError(VIR_ERR_OPERATION_FAILED,
_("cannot detach %s device"), devicestr); _("cannot detach %s device"), devalias);
goto cleanup; goto cleanup;
} }
if (STRNEQ(reply, "")) { if (STRNEQ(reply, "")) {
qemuReportError(VIR_ERR_OPERATION_FAILED, qemuReportError(VIR_ERR_OPERATION_FAILED,
_("detaching %s device failed: %s"), devicestr, reply); _("detaching %s device failed: %s"), devalias, reply);
goto cleanup; goto cleanup;
} }
......
...@@ -168,7 +168,7 @@ int qemuMonitorTextAddDevice(qemuMonitorPtr mon, ...@@ -168,7 +168,7 @@ int qemuMonitorTextAddDevice(qemuMonitorPtr mon,
const char *devicestr); const char *devicestr);
int qemuMonitorTextDelDevice(qemuMonitorPtr mon, int qemuMonitorTextDelDevice(qemuMonitorPtr mon,
const char *devicestr); const char *devalias);
int qemuMonitorTextAddDrive(qemuMonitorPtr mon, int qemuMonitorTextAddDrive(qemuMonitorPtr mon,
const char *drivestr); const char *drivestr);
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册