提交 db37f3cc 编写于 作者: P Peter Krempa

qemu: Clean up old leftovers in qemuMonitorDrivePivot

There are two leftover unused variables. Remove them and clean up the
fallout of the change.
上级 3eab2f64
...@@ -16044,7 +16044,6 @@ qemuDomainBlockPivot(virConnectPtr conn, ...@@ -16044,7 +16044,6 @@ qemuDomainBlockPivot(virConnectPtr conn,
int ret = -1, rc; int ret = -1, rc;
qemuDomainObjPrivatePtr priv = vm->privateData; qemuDomainObjPrivatePtr priv = vm->privateData;
virDomainBlockJobInfo info; virDomainBlockJobInfo info;
const char *format = NULL;
bool resume = false; bool resume = false;
virStorageSourcePtr oldsrc = NULL; virStorageSourcePtr oldsrc = NULL;
virQEMUDriverConfigPtr cfg = virQEMUDriverGetConfig(driver); virQEMUDriverConfigPtr cfg = virQEMUDriverGetConfig(driver);
...@@ -16056,8 +16055,6 @@ qemuDomainBlockPivot(virConnectPtr conn, ...@@ -16056,8 +16055,6 @@ qemuDomainBlockPivot(virConnectPtr conn,
goto cleanup; goto cleanup;
} }
format = virStorageFileFormatTypeToString(disk->mirror->format);
/* Probe the status, if needed. */ /* Probe the status, if needed. */
if (!disk->mirrorState) { if (!disk->mirrorState) {
qemuDomainObjEnterMonitor(driver, vm); qemuDomainObjEnterMonitor(driver, vm);
...@@ -16141,7 +16138,7 @@ qemuDomainBlockPivot(virConnectPtr conn, ...@@ -16141,7 +16138,7 @@ qemuDomainBlockPivot(virConnectPtr conn,
* overall return value. */ * overall return value. */
disk->mirrorState = VIR_DOMAIN_DISK_MIRROR_STATE_PIVOT; disk->mirrorState = VIR_DOMAIN_DISK_MIRROR_STATE_PIVOT;
qemuDomainObjEnterMonitor(driver, vm); qemuDomainObjEnterMonitor(driver, vm);
ret = qemuMonitorDrivePivot(priv->mon, device, disk->mirror->path, format); ret = qemuMonitorDrivePivot(priv->mon, device);
if (qemuDomainObjExitMonitor(driver, vm) < 0) { if (qemuDomainObjExitMonitor(driver, vm) < 0) {
ret = -1; ret = -1;
goto cleanup; goto cleanup;
......
...@@ -3492,23 +3492,20 @@ qemuMonitorDiskNameLookup(qemuMonitorPtr mon, ...@@ -3492,23 +3492,20 @@ qemuMonitorDiskNameLookup(qemuMonitorPtr mon,
} }
/* Use the block-job-complete monitor command to pivot a block copy /* Use the block-job-complete monitor command to pivot a block copy job. */
* job. */
int int
qemuMonitorDrivePivot(qemuMonitorPtr mon, const char *device, qemuMonitorDrivePivot(qemuMonitorPtr mon,
const char *file, const char *format) const char *device)
{ {
int ret = -1; VIR_DEBUG("mon=%p, device=%s", mon, device);
VIR_DEBUG("mon=%p, device=%s, file=%s, format=%s",
mon, device, file, NULLSTR(format));
if (mon->json) if (!mon->json) {
ret = qemuMonitorJSONDrivePivot(mon, device, file, format);
else
virReportError(VIR_ERR_OPERATION_UNSUPPORTED, "%s", virReportError(VIR_ERR_OPERATION_UNSUPPORTED, "%s",
_("drive pivot requires JSON monitor")); _("drive pivot requires JSON monitor"));
return ret; return -1;
}
return qemuMonitorJSONDrivePivot(mon, device);
} }
int qemuMonitorArbitraryCommand(qemuMonitorPtr mon, int qemuMonitorArbitraryCommand(qemuMonitorPtr mon,
......
...@@ -721,10 +721,8 @@ int qemuMonitorDriveMirror(qemuMonitorPtr mon, ...@@ -721,10 +721,8 @@ int qemuMonitorDriveMirror(qemuMonitorPtr mon,
unsigned int flags) unsigned int flags)
ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2) ATTRIBUTE_NONNULL(3); ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2) ATTRIBUTE_NONNULL(3);
int qemuMonitorDrivePivot(qemuMonitorPtr mon, int qemuMonitorDrivePivot(qemuMonitorPtr mon,
const char *device, const char *device)
const char *file, ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2);
const char *format)
ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2) ATTRIBUTE_NONNULL(3);
int qemuMonitorBlockCommit(qemuMonitorPtr mon, int qemuMonitorBlockCommit(qemuMonitorPtr mon,
const char *device, const char *device,
......
...@@ -3859,9 +3859,8 @@ qemuMonitorJSONBlockCommit(qemuMonitorPtr mon, const char *device, ...@@ -3859,9 +3859,8 @@ qemuMonitorJSONBlockCommit(qemuMonitorPtr mon, const char *device,
} }
int int
qemuMonitorJSONDrivePivot(qemuMonitorPtr mon, const char *device, qemuMonitorJSONDrivePivot(qemuMonitorPtr mon,
const char *file ATTRIBUTE_UNUSED, const char *device)
const char *format ATTRIBUTE_UNUSED)
{ {
int ret; int ret;
virJSONValuePtr cmd; virJSONValuePtr cmd;
......
...@@ -264,9 +264,7 @@ int qemuMonitorJSONDriveMirror(qemuMonitorPtr mon, ...@@ -264,9 +264,7 @@ int qemuMonitorJSONDriveMirror(qemuMonitorPtr mon,
unsigned int flags) unsigned int flags)
ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2) ATTRIBUTE_NONNULL(3); ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2) ATTRIBUTE_NONNULL(3);
int qemuMonitorJSONDrivePivot(qemuMonitorPtr mon, int qemuMonitorJSONDrivePivot(qemuMonitorPtr mon,
const char *device, const char *device)
const char *file,
const char *format)
ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2); ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2);
int qemuMonitorJSONBlockCommit(qemuMonitorPtr mon, int qemuMonitorJSONBlockCommit(qemuMonitorPtr mon,
......
...@@ -1186,7 +1186,7 @@ GEN_TEST_FUNC(qemuMonitorJSONSetDrivePassphrase, "vda", "secret_passhprase") ...@@ -1186,7 +1186,7 @@ GEN_TEST_FUNC(qemuMonitorJSONSetDrivePassphrase, "vda", "secret_passhprase")
GEN_TEST_FUNC(qemuMonitorJSONDriveMirror, "vdb", "/foo/bar", NULL, 1024, 0, 0, GEN_TEST_FUNC(qemuMonitorJSONDriveMirror, "vdb", "/foo/bar", NULL, 1024, 0, 0,
VIR_DOMAIN_BLOCK_REBASE_SHALLOW | VIR_DOMAIN_BLOCK_REBASE_REUSE_EXT) VIR_DOMAIN_BLOCK_REBASE_SHALLOW | VIR_DOMAIN_BLOCK_REBASE_REUSE_EXT)
GEN_TEST_FUNC(qemuMonitorJSONBlockCommit, "vdb", "/foo/bar1", "/foo/bar2", NULL, 1024) GEN_TEST_FUNC(qemuMonitorJSONBlockCommit, "vdb", "/foo/bar1", "/foo/bar2", NULL, 1024)
GEN_TEST_FUNC(qemuMonitorJSONDrivePivot, "vdb", NULL, NULL) GEN_TEST_FUNC(qemuMonitorJSONDrivePivot, "vdb")
GEN_TEST_FUNC(qemuMonitorJSONScreendump, "/foo/bar") GEN_TEST_FUNC(qemuMonitorJSONScreendump, "/foo/bar")
GEN_TEST_FUNC(qemuMonitorJSONOpenGraphics, "spice", "spicefd", false) GEN_TEST_FUNC(qemuMonitorJSONOpenGraphics, "spice", "spicefd", false)
GEN_TEST_FUNC(qemuMonitorJSONNBDServerStart, "localhost", 12345) GEN_TEST_FUNC(qemuMonitorJSONNBDServerStart, "localhost", 12345)
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册