From nobody Tue May 14 04:11:24 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) client-ip=170.10.133.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1647598719; cv=none; d=zohomail.com; s=zohoarc; b=hWMHvlpuYLe4I7cgFOSwD65iyN7JqOH3rBEWOc9iQC84fCBJKphZ8IazCWTob/9j1lVDRd9emvd9IEHxLYFLbqhjMCuyBJsZzs2rwFzLCwFrVJTScTK/jPrDaBnUacLTh4zduP/DQ9Cjvq/cY26ZY9dwspHvZlfKNqVS5R7MvWM= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1647598719; h=Content-Type:Content-Transfer-Encoding:Date:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Sender:Subject:To; bh=2EyG8iU1EA5AIFUJaZf6qGwMA1s7ecoDxsj2B5rV25c=; b=Vrdeuw6rrbqL+hrYv3goyuhgkWF109/g23WztOgMG588SYBt+IchOkz0YALzh64YbAq77Ubabn3nnwOZ8uXebv2+3Ty3DjKsrvlR9tZEhO8qrwKn4l8z9b9Egbvxao63o+y18FsT7h8lMY+mWeUqyXDejtuqt+ACnuxcnZNcwiQ= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mx.zohomail.com with SMTPS id 1647598719742898.0309934120511; Fri, 18 Mar 2022 03:18:39 -0700 (PDT) Received: from mimecast-mx02.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-246-m6E_kVa8Mi2u4SBZti249A-1; Fri, 18 Mar 2022 06:18:35 -0400 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.rdu2.redhat.com [10.11.54.7]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id D1E82296A61D; Fri, 18 Mar 2022 10:18:32 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com [10.30.29.100]) by smtp.corp.redhat.com (Postfix) with ESMTP id 75C101400E73; Fri, 18 Mar 2022 10:18:31 +0000 (UTC) Received: from mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (localhost [IPv6:::1]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id DA07D1949763; Fri, 18 Mar 2022 10:18:28 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.rdu2.redhat.com [10.11.54.4]) by mm-prod-listman-01.mail-001.prod.us-east-1.aws.redhat.com (Postfix) with ESMTP id 5A1FC1949761 for ; Fri, 18 Mar 2022 10:17:41 +0000 (UTC) Received: by smtp.corp.redhat.com (Postfix) id 605092023A1F; Fri, 18 Mar 2022 10:17:41 +0000 (UTC) Received: from fedora.redhat.com (unknown [10.43.2.78]) by smtp.corp.redhat.com (Postfix) with ESMTP id 5199A2014F87 for ; Fri, 18 Mar 2022 10:17:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1647598719; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=2EyG8iU1EA5AIFUJaZf6qGwMA1s7ecoDxsj2B5rV25c=; b=Jd/0o0LDtGquSn101r1lq/2pcWQJCbBukBWdBs5oKO4iORmMqwH0OCOyKXwQDJihHPdIjK +8VZRynjBgeZFEeUPIjTm9XKh/LdpeSH0s5tD5Qtn4gkfSphHJ5VeD3688mpaGlzn8VbWN XR4URyCVRaHVYJ6QsRcGGYGbBnEUexI= X-MC-Unique: m6E_kVa8Mi2u4SBZti249A-1 X-Original-To: libvir-list@listman.corp.redhat.com From: Kristina Hanicova To: libvir-list@redhat.com Subject: [PATCH] qemu: use qemuDomainSaveStatus() and remove qemuDomainObjSaveStatus() Date: Fri, 18 Mar 2022 11:17:28 +0100 Message-Id: <9a2e35ceb1a299c1babf7744e3eb72a75178475a.1647598623.git.khanicov@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.78 on 10.11.54.4 X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: libvir-list-bounces@redhat.com Sender: "libvir-list" X-Scanned-By: MIMEDefang 2.85 on 10.11.54.7 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=libvir-list-bounces@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1647598721586100001 Content-Type: text/plain; charset="utf-8" It does not make sense to have both of these, since one of them is only a wrapper for the other one. I decided to preserve the more general one, which requires only virDomainObj and rewrote it a bit, so that it pulls the qemu driver from privateData. Signed-off-by: Kristina Hanicova Reviewed-by: Jiri Denemark --- src/qemu/THREADS.txt | 4 +- src/qemu/qemu_backup.c | 20 +-- src/qemu/qemu_block.c | 18 +- src/qemu/qemu_blockjob.c | 20 +-- src/qemu/qemu_checkpoint.c | 14 +- src/qemu/qemu_domain.c | 35 ++-- src/qemu/qemu_domain.h | 7 +- src/qemu/qemu_domainjob.c | 19 +- src/qemu/qemu_domainjob.h | 12 +- src/qemu/qemu_driver.c | 296 +++++++++++++++---------------- src/qemu/qemu_hotplug.c | 136 +++++++------- src/qemu/qemu_migration.c | 120 ++++++------- src/qemu/qemu_migration_cookie.c | 2 +- src/qemu/qemu_migration_params.c | 8 +- src/qemu/qemu_process.c | 65 ++++--- src/qemu/qemu_process.h | 3 +- src/qemu/qemu_snapshot.c | 14 +- 17 files changed, 384 insertions(+), 409 deletions(-) diff --git a/src/qemu/THREADS.txt b/src/qemu/THREADS.txt index 98aa3165e3..30cf3ce210 100644 --- a/src/qemu/THREADS.txt +++ b/src/qemu/THREADS.txt @@ -294,7 +294,7 @@ Design patterns goto error; } ...start qemu job... - qemuDomainObjExitMonitor(driver, obj); + qemuDomainObjExitMonitor(obj); =20 while (!finished) { if (qemuDomainObjEnterMonitorAsync(driver, obj, @@ -303,7 +303,7 @@ Design patterns goto error; } ...monitor job progress... - qemuDomainObjExitMonitor(driver, obj); + qemuDomainObjExitMonitor(obj); =20 virObjectUnlock(obj); sleep(aWhile); diff --git a/src/qemu/qemu_backup.c b/src/qemu/qemu_backup.c index 2471242e60..f31b840617 100644 --- a/src/qemu/qemu_backup.c +++ b/src/qemu/qemu_backup.c @@ -132,7 +132,7 @@ qemuBackupDiskDataCleanupOne(virDomainObj *vm, if (dd->added) { qemuDomainObjEnterMonitor(priv->driver, vm); qemuBlockStorageSourceAttachRollback(priv->mon, dd->crdata->sr= cdata[0]); - qemuDomainObjExitMonitor(priv->driver, vm); + qemuDomainObjExitMonitor(vm); } =20 if (dd->created) { @@ -474,7 +474,7 @@ qemuBackupDiskPrepareOneStorage(virDomainObj *vm, =20 rc =3D qemuBlockStorageSourceAttachApply(priv->mon, dd->crdata->sr= cdata[0]); =20 - qemuDomainObjExitMonitor(priv->driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) return -1; } @@ -623,7 +623,7 @@ qemuBackupJobTerminate(virDomainObj *vm, g_clear_pointer(&priv->backup, virDomainBackupDefFree); =20 if (priv->job.asyncJob =3D=3D QEMU_ASYNC_JOB_BACKUP) - qemuDomainObjEndAsyncJob(priv->driver, vm); + qemuDomainObjEndAsyncJob(vm); } =20 =20 @@ -681,7 +681,7 @@ qemuBackupJobCancelBlockjobs(virDomainObj *vm, =20 rc =3D qemuMonitorBlockJobCancel(priv->mon, job->name, true); =20 - qemuDomainObjExitMonitor(priv->driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rc =3D=3D 0) { backupdisk->state =3D VIR_DOMAIN_BACKUP_DISK_STATE_CANCELLING; @@ -893,7 +893,7 @@ qemuBackupBegin(virDomainObj *vm, if (rc =3D=3D 0) rc =3D qemuMonitorTransaction(priv->mon, &actions); =20 - qemuDomainObjExitMonitor(priv->driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) goto endjob; =20 @@ -915,7 +915,7 @@ qemuBackupBegin(virDomainObj *vm, /* note that if the export fails we've already created the checkpo= int * and we will not delete it */ rc =3D qemuBackupBeginPullExportDisks(vm, dd, ndd); - qemuDomainObjExitMonitor(priv->driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rc < 0) { qemuBackupJobCancelBlockjobs(vm, priv->backup, false, QEMU_ASY= NC_JOB_BACKUP); @@ -939,7 +939,7 @@ qemuBackupBegin(virDomainObj *vm, ignore_value(qemuMonitorDelObject(priv->mon, tlsAlias, false)); if (tlsSecretAlias) ignore_value(qemuMonitorDelObject(priv->mon, tlsSecretAlias, f= alse)); - qemuDomainObjExitMonitor(priv->driver, vm); + qemuDomainObjExitMonitor(vm); } =20 if (ret < 0 && !job_started && priv->backup) @@ -948,7 +948,7 @@ qemuBackupBegin(virDomainObj *vm, if (ret =3D=3D 0) qemuDomainObjReleaseAsyncJob(vm); else - qemuDomainObjEndAsyncJob(priv->driver, vm); + qemuDomainObjEndAsyncJob(vm); =20 return ret; } @@ -1007,7 +1007,7 @@ qemuBackupNotifyBlockjobEnd(virDomainObj *vm, ignore_value(qemuMonitorDelObject(priv->mon, backup->tlsAlias,= false)); if (backup->tlsSecretAlias) ignore_value(qemuMonitorDelObject(priv->mon, backup->tlsSecret= Alias, false)); - qemuDomainObjExitMonitor(priv->driver, vm); + qemuDomainObjExitMonitor(vm); =20 /* update the final statistics with the current job's data */ backup->pull_tmp_used +=3D cur; @@ -1165,7 +1165,7 @@ qemuBackupGetJobInfoStats(virQEMUDriver *driver, =20 rc =3D qemuMonitorGetJobInfo(priv->mon, &blockjobs, &nblockjobs); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) goto cleanup; =20 diff --git a/src/qemu/qemu_block.c b/src/qemu/qemu_block.c index 4195883a1e..f70b6d3e63 100644 --- a/src/qemu/qemu_block.c +++ b/src/qemu/qemu_block.c @@ -326,7 +326,7 @@ qemuBlockNodeNamesDetect(virQEMUDriver *driver, data =3D qemuMonitorQueryNamedBlockNodes(qemuDomainGetMonitor(vm)); blockstats =3D qemuMonitorQueryBlockstats(qemuDomainGetMonitor(vm)); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (!data || !blockstats) return -1; @@ -2133,7 +2133,7 @@ qemuBlockStorageSourceDetachOneBlockdev(virQEMUDriver= *driver, if (ret =3D=3D 0) ret =3D qemuMonitorBlockdevDel(qemuDomainGetMonitor(vm), src->node= storage); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 return ret; } @@ -2712,7 +2712,7 @@ qemuBlockStorageSourceCreateGeneric(virDomainObj *vm, =20 rc =3D qemuMonitorBlockdevCreate(priv->mon, job->name, &props); =20 - qemuDomainObjExitMonitor(priv->driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) goto cleanup; =20 @@ -2859,7 +2859,7 @@ qemuBlockStorageSourceCreate(virDomainObj *vm, =20 rc =3D qemuBlockStorageSourceAttachApplyStorageDeps(priv->mon, data); =20 - qemuDomainObjExitMonitor(priv->driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) goto cleanup; =20 @@ -2874,7 +2874,7 @@ qemuBlockStorageSourceCreate(virDomainObj *vm, if (rc =3D=3D 0) rc =3D qemuBlockStorageSourceAttachApplyFormatDeps(priv->mon, data= ); =20 - qemuDomainObjExitMonitor(priv->driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) goto cleanup; =20 @@ -2893,7 +2893,7 @@ qemuBlockStorageSourceCreate(virDomainObj *vm, =20 rc =3D qemuBlockStorageSourceAttachApplyFormat(priv->mon, data); =20 - qemuDomainObjExitMonitor(priv->driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) goto cleanup; =20 @@ -2905,7 +2905,7 @@ qemuBlockStorageSourceCreate(virDomainObj *vm, qemuDomainObjEnterMonitorAsync(priv->driver, vm, asyncJob) =3D=3D = 0) { =20 qemuBlockStorageSourceAttachRollback(priv->mon, data); - qemuDomainObjExitMonitor(priv->driver, vm); + qemuDomainObjExitMonitor(vm); } =20 return ret; @@ -3033,7 +3033,7 @@ qemuBlockGetNamedNodeData(virDomainObj *vm, =20 blockNamedNodeData =3D qemuMonitorBlockGetNamedNodeData(priv->mon, sup= ports_flat); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (!blockNamedNodeData) return NULL; @@ -3391,7 +3391,7 @@ qemuBlockReopenFormat(virDomainObj *vm, =20 rc =3D qemuBlockReopenFormatMon(priv->mon, src); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) return -1; =20 diff --git a/src/qemu/qemu_blockjob.c b/src/qemu/qemu_blockjob.c index 7032814d21..87f8ae7b52 100644 --- a/src/qemu/qemu_blockjob.c +++ b/src/qemu/qemu_blockjob.c @@ -504,7 +504,7 @@ qemuBlockJobRefreshJobs(virQEMUDriver *driver, =20 rc =3D qemuMonitorGetJobInfo(priv->mon, &jobinfo, &njobinfo); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) goto cleanup; =20 @@ -528,7 +528,7 @@ qemuBlockJobRefreshJobs(virQEMUDriver *driver, if (rc =3D=3D -1 && jobinfo[i]->status =3D=3D QEMU_MONITOR_JOB= _STATUS_CONCLUDED) VIR_WARN("can't cancel job '%s' with invalid data", job->n= ame); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rc < 0) qemuBlockJobUnregister(job, vm); @@ -852,7 +852,7 @@ qemuBlockJobEventProcessConcludedRemoveChain(virQEMUDri= ver *driver, =20 qemuBlockStorageSourceChainDetach(qemuDomainGetMonitor(vm), data); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 qemuDomainStorageSourceChainAccessRevoke(driver, vm, chain); } @@ -967,7 +967,7 @@ qemuBlockJobProcessEventCompletedPullBitmaps(virDomainO= bj *vm, =20 qemuMonitorTransaction(priv->mon, &actions); =20 - qemuDomainObjExitMonitor(priv->driver, vm); + qemuDomainObjExitMonitor(vm); =20 return 0; } @@ -1140,7 +1140,7 @@ qemuBlockJobProcessEventCompletedCommitBitmaps(virDom= ainObj *vm, =20 qemuMonitorTransaction(priv->mon, &actions); =20 - qemuDomainObjExitMonitor(priv->driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (!active) { if (qemuBlockReopenReadOnly(vm, job->data.commit.base, asyncJob) <= 0) @@ -1357,7 +1357,7 @@ qemuBlockJobProcessEventCompletedCopyBitmaps(virDomai= nObj *vm, =20 qemuMonitorTransaction(priv->mon, &actions); =20 - qemuDomainObjExitMonitor(priv->driver, vm); + qemuDomainObjExitMonitor(vm); =20 return 0; } @@ -1455,7 +1455,7 @@ qemuBlockJobProcessEventFailedActiveCommit(virQEMUDri= ver *driver, disk->mirror->nodeformat, "libvirt-tmp-activewrite"); =20 - qemuDomainObjExitMonitor(priv->driver, vm); + qemuDomainObjExitMonitor(vm); =20 /* Ideally, we would make the backing chain read only again (yes, SELi= nux * can do that using different labels). But that is not implemented ye= t and @@ -1501,7 +1501,7 @@ qemuBlockJobProcessEventConcludedCreate(virQEMUDriver= *driver, =20 qemuBlockStorageSourceAttachRollback(qemuDomainGetMonitor(vm), backend= ); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 qemuDomainStorageSourceAccessRevoke(driver, vm, job->data.create.src); } @@ -1536,7 +1536,7 @@ qemuBlockJobProcessEventConcludedBackup(virQEMUDriver= *driver, job->disk->src->nodeformat, job->data.backup.bitmap); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (job->data.backup.store) qemuDomainStorageSourceAccessRevoke(driver, vm, job->data.backup.s= tore); @@ -1649,7 +1649,7 @@ qemuBlockJobEventProcessConcluded(qemuBlockJobData *j= ob, /* dismiss job in qemu */ ignore_value(qemuMonitorJobDismiss(qemuDomainGetMonitor(vm), job->name= )); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if ((job->newstate =3D=3D QEMU_BLOCKJOB_STATE_COMPLETED || job->newstate =3D=3D QEMU_BLOCKJOB_STATE_FAILED) && diff --git a/src/qemu/qemu_checkpoint.c b/src/qemu/qemu_checkpoint.c index 69f287399b..2a495dfe08 100644 --- a/src/qemu/qemu_checkpoint.c +++ b/src/qemu/qemu_checkpoint.c @@ -237,7 +237,7 @@ qemuCheckpointDiscardBitmaps(virDomainObj *vm, =20 qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorTransaction(priv->mon, &actions); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 relabel: for (next =3D relabelimages; next; next =3D next->next) { @@ -524,7 +524,7 @@ qemuCheckpointCreate(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorTransaction(qemuDomainGetMonitor(vm), &actions); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) { qemuCheckpointRollbackMetadata(vm, chk); return NULL; @@ -629,7 +629,7 @@ qemuCheckpointCreateXML(virDomainPtr domain, checkpoint =3D virGetDomainCheckpoint(domain, chk->def->name); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 return checkpoint; } @@ -736,7 +736,7 @@ qemuCheckpointGetXMLDescUpdateSize(virDomainObj *vm, if (rc =3D=3D 0) rc =3D qemuMonitorTransaction(priv->mon, &mergeactions); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) goto endjob; =20 @@ -748,7 +748,7 @@ qemuCheckpointGetXMLDescUpdateSize(virDomainObj *vm, =20 rc =3D qemuMonitorTransaction(priv->mon, &cleanupactions); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) goto endjob; =20 @@ -768,7 +768,7 @@ qemuCheckpointGetXMLDescUpdateSize(virDomainObj *vm, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); return ret; } =20 @@ -921,6 +921,6 @@ qemuCheckpointDelete(virDomainObj *vm, } =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); return ret; } diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c index b4184285bf..7180ae616b 100644 --- a/src/qemu/qemu_domain.c +++ b/src/qemu/qemu_domain.c @@ -5822,9 +5822,10 @@ virDomainDefParserConfig virQEMUDriverDomainDefParse= rConfig =3D { =20 =20 void -qemuDomainObjSaveStatus(virQEMUDriver *driver, - virDomainObj *obj) +qemuDomainSaveStatus(virDomainObj *obj) { + qemuDomainObjPrivate *priv =3D obj->privateData; + virQEMUDriver *driver =3D priv->driver; g_autoptr(virQEMUDriverConfig) cfg =3D virQEMUDriverGetConfig(driver); =20 if (virDomainObjIsActive(obj)) { @@ -5834,13 +5835,6 @@ qemuDomainObjSaveStatus(virQEMUDriver *driver, } =20 =20 -void -qemuDomainSaveStatus(virDomainObj *obj) -{ - qemuDomainObjSaveStatus(QEMU_DOMAIN_PRIVATE(obj)->driver, obj); -} - - void qemuDomainSaveConfig(virDomainObj *obj) { @@ -5887,7 +5881,7 @@ qemuDomainObjEnterMonitorInternal(virQEMUDriver *driv= er, if (!virDomainObjIsActive(obj)) { virReportError(VIR_ERR_OPERATION_FAILED, "%s", _("domain is no longer running")); - qemuDomainObjEndJob(driver, obj); + qemuDomainObjEndJob(obj); return -1; } } else if (priv->job.asyncOwner =3D=3D virThreadSelfID()) { @@ -5916,8 +5910,7 @@ qemuDomainObjEnterMonitorInternal(virQEMUDriver *driv= er, * */ void -qemuDomainObjExitMonitor(virQEMUDriver *driver, - virDomainObj *obj) +qemuDomainObjExitMonitor(virDomainObj *obj) { qemuDomainObjPrivate *priv =3D obj->privateData; bool hasRefs; @@ -5938,7 +5931,7 @@ qemuDomainObjExitMonitor(virQEMUDriver *driver, priv->mon =3D NULL; =20 if (priv->job.active =3D=3D QEMU_JOB_ASYNC_NESTED) - qemuDomainObjEndJob(driver, obj); + qemuDomainObjEndJob(obj); } =20 void qemuDomainObjEnterMonitor(virQEMUDriver *driver, @@ -7004,7 +6997,7 @@ qemuDomainSnapshotDiscard(virQEMUDriver *driver, qemuDomainObjEnterMonitor(driver, vm); /* we continue on even in the face of error */ qemuMonitorDeleteSnapshot(priv->mon, snap->def->name); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); } } =20 @@ -7169,7 +7162,7 @@ qemuDomainRemoveInactiveJob(virQEMUDriver *driver, qemuDomainRemoveInactive(driver, vm); =20 if (haveJob) - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); } =20 =20 @@ -7190,7 +7183,7 @@ qemuDomainRemoveInactiveJobLocked(virQEMUDriver *driv= er, qemuDomainRemoveInactiveLocked(driver, vm); =20 if (haveJob) - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); } =20 =20 @@ -8213,7 +8206,7 @@ qemuDomainUpdateDeviceList(virQEMUDriver *driver, if (qemuDomainObjEnterMonitorAsync(driver, vm, asyncJob) < 0) return -1; rc =3D qemuMonitorGetDeviceAliases(priv->mon, &aliases); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) return -1; =20 @@ -8241,7 +8234,7 @@ qemuDomainUpdateMemoryDeviceInfo(virQEMUDriver *drive= r, =20 rc =3D qemuMonitorGetMemoryDeviceInfo(priv->mon, &meminfo); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rc < 0) return -1; @@ -9515,7 +9508,7 @@ qemuDomainRefreshVcpuInfo(virQEMUDriver *driver, rc =3D qemuMonitorGetCPUInfo(qemuDomainGetMonitor(vm), &info, maxvcpus, hotplug, fast); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rc < 0) goto cleanup; @@ -9669,7 +9662,7 @@ qemuDomainRefreshVcpuHalted(virQEMUDriver *driver, QEMU_CAPS_QUERY_CPUS_FAST); haltedmap =3D qemuMonitorGetCpuHalted(qemuDomainGetMonitor(vm), maxvcp= us, fast); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (!haltedmap) return -1; @@ -10100,7 +10093,7 @@ qemuDomainCheckMonitor(virQEMUDriver *driver, =20 ret =3D qemuMonitorCheck(priv->mon); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 return ret; } diff --git a/src/qemu/qemu_domain.h b/src/qemu/qemu_domain.h index be56b5dbb3..edafb585b3 100644 --- a/src/qemu/qemu_domain.h +++ b/src/qemu/qemu_domain.h @@ -61,10 +61,6 @@ typedef void (*qemuDomainCleanupCallback)(virQEMUDriver = *driver, =20 #define QEMU_DOMAIN_MASTER_KEY_LEN 32 /* 32 bytes for 256 bit random key = */ =20 -void -qemuDomainObjSaveStatus(virQEMUDriver *driver, - virDomainObj *obj); - void qemuDomainSaveStatus(virDomainObj *obj); void qemuDomainSaveConfig(virDomainObj *obj); =20 @@ -500,8 +496,7 @@ qemuMonitor *qemuDomainGetMonitor(virDomainObj *vm) void qemuDomainObjEnterMonitor(virQEMUDriver *driver, virDomainObj *obj) ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2); -void qemuDomainObjExitMonitor(virQEMUDriver *driver, - virDomainObj *obj) +void qemuDomainObjExitMonitor(virDomainObj *obj) ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2); int qemuDomainObjEnterMonitorAsync(virQEMUDriver *driver, virDomainObj *obj, diff --git a/src/qemu/qemu_domainjob.c b/src/qemu/qemu_domainjob.c index 28c09ae179..cf1e093e22 100644 --- a/src/qemu/qemu_domainjob.c +++ b/src/qemu/qemu_domainjob.c @@ -703,8 +703,7 @@ qemuDomainJobDataToParams(virDomainJobData *jobData, =20 =20 void -qemuDomainObjSetJobPhase(virQEMUDriver *driver, - virDomainObj *obj, +qemuDomainObjSetJobPhase(virDomainObj *obj, int phase) { qemuDomainObjPrivate *priv =3D obj->privateData; @@ -727,7 +726,7 @@ qemuDomainObjSetJobPhase(virQEMUDriver *driver, =20 priv->job.phase =3D phase; priv->job.asyncOwner =3D me; - qemuDomainObjSaveStatus(driver, obj); + qemuDomainSaveStatus(obj); } =20 void @@ -743,14 +742,14 @@ qemuDomainObjSetAsyncJobMask(virDomainObj *obj, } =20 void -qemuDomainObjDiscardAsyncJob(virQEMUDriver *driver, virDomainObj *obj) +qemuDomainObjDiscardAsyncJob(virDomainObj *obj) { qemuDomainObjPrivate *priv =3D obj->privateData; =20 if (priv->job.active =3D=3D QEMU_JOB_ASYNC_NESTED) qemuDomainObjResetJob(&priv->job); qemuDomainObjResetAsyncJob(&priv->job); - qemuDomainObjSaveStatus(driver, obj); + qemuDomainSaveStatus(obj); } =20 void @@ -926,7 +925,7 @@ qemuDomainObjBeginJobInternal(virQEMUDriver *driver, } =20 if (qemuDomainTrackJob(job)) - qemuDomainObjSaveStatus(driver, obj); + qemuDomainSaveStatus(obj); =20 return 0; =20 @@ -1134,7 +1133,7 @@ qemuDomainObjBeginJobNowait(virQEMUDriver *driver, * earlier qemuDomainBeginJob() call */ void -qemuDomainObjEndJob(virQEMUDriver *driver, virDomainObj *obj) +qemuDomainObjEndJob(virDomainObj *obj) { qemuDomainObjPrivate *priv =3D obj->privateData; qemuDomainJob job =3D priv->job.active; @@ -1148,7 +1147,7 @@ qemuDomainObjEndJob(virQEMUDriver *driver, virDomainO= bj *obj) =20 qemuDomainObjResetJob(&priv->job); if (qemuDomainTrackJob(job)) - qemuDomainObjSaveStatus(driver, obj); + qemuDomainSaveStatus(obj); /* We indeed need to wake up ALL threads waiting because * grabbing a job requires checking more variables. */ virCondBroadcast(&priv->job.cond); @@ -1174,7 +1173,7 @@ qemuDomainObjEndAgentJob(virDomainObj *obj) } =20 void -qemuDomainObjEndAsyncJob(virQEMUDriver *driver, virDomainObj *obj) +qemuDomainObjEndAsyncJob(virDomainObj *obj) { qemuDomainObjPrivate *priv =3D obj->privateData; =20 @@ -1185,7 +1184,7 @@ qemuDomainObjEndAsyncJob(virQEMUDriver *driver, virDo= mainObj *obj) obj, obj->def->name); =20 qemuDomainObjResetAsyncJob(&priv->job); - qemuDomainObjSaveStatus(driver, obj); + qemuDomainSaveStatus(obj); virCondBroadcast(&priv->job.asyncCond); } =20 diff --git a/src/qemu/qemu_domainjob.h b/src/qemu/qemu_domainjob.h index a078e62a1f..bec6e3a61c 100644 --- a/src/qemu/qemu_domainjob.h +++ b/src/qemu/qemu_domainjob.h @@ -208,21 +208,17 @@ int qemuDomainObjBeginJobNowait(virQEMUDriver *driver, qemuDomainJob job) G_GNUC_WARN_UNUSED_RESULT; =20 -void qemuDomainObjEndJob(virQEMUDriver *driver, - virDomainObj *obj); +void qemuDomainObjEndJob(virDomainObj *obj); void qemuDomainObjEndAgentJob(virDomainObj *obj); -void qemuDomainObjEndAsyncJob(virQEMUDriver *driver, - virDomainObj *obj); +void qemuDomainObjEndAsyncJob(virDomainObj *obj); void qemuDomainObjAbortAsyncJob(virDomainObj *obj); -void qemuDomainObjSetJobPhase(virQEMUDriver *driver, - virDomainObj *obj, +void qemuDomainObjSetJobPhase(virDomainObj *obj, int phase); void qemuDomainObjSetAsyncJobMask(virDomainObj *obj, unsigned long long allowedJobs); int qemuDomainObjRestoreJob(virDomainObj *obj, qemuDomainJobObj *job); -void qemuDomainObjDiscardAsyncJob(virQEMUDriver *driver, - virDomainObj *obj); +void qemuDomainObjDiscardAsyncJob(virDomainObj *obj); void qemuDomainObjReleaseAsyncJob(virDomainObj *obj); =20 int qemuDomainJobDataUpdateTime(virDomainJobData *jobData) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index de7d2b93b9..b7e83c769a 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -208,7 +208,7 @@ qemuAutostartDomain(virDomainObj *vm, vm->def->name, virGetLastErrorMessage()); } =20 - qemuProcessEndJob(driver, vm); + qemuProcessEndJob(vm); } =20 ret =3D 0; @@ -1631,7 +1631,7 @@ static virDomainPtr qemuDomainCreateXML(virConnectPtr= conn, start_flags) < 0) { virDomainAuditStart(vm, "booted", false); qemuDomainRemoveInactive(driver, vm); - qemuProcessEndJob(driver, vm); + qemuProcessEndJob(vm); goto cleanup; } =20 @@ -1652,7 +1652,7 @@ static virDomainPtr qemuDomainCreateXML(virConnectPtr= conn, =20 dom =3D virGetDomain(conn, vm->def->name, vm->def->uuid, vm->def->id); =20 - qemuProcessEndJob(driver, vm); + qemuProcessEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -1706,7 +1706,7 @@ static int qemuDomainSuspend(virDomainPtr dom) ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -1762,7 +1762,7 @@ static int qemuDomainResume(virDomainPtr dom) ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -1827,10 +1827,10 @@ qemuDomainShutdownFlagsMonitor(virQEMUDriver *drive= r, qemuDomainSetFakeReboot(vm, isReboot); qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorSystemPowerdown(priv->mon); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); return ret; } =20 @@ -1952,10 +1952,10 @@ qemuDomainRebootMonitor(virQEMUDriver *driver, qemuDomainSetFakeReboot(vm, isReboot); qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorSystemPowerdown(priv->mon); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); return ret; } =20 @@ -2041,7 +2041,7 @@ qemuDomainReset(virDomainPtr dom, unsigned int flags) priv =3D vm->privateData; qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorSystemReset(priv->mon); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 priv->fakeReboot =3D false; =20 @@ -2050,7 +2050,7 @@ qemuDomainReset(virDomainPtr dom, unsigned int flags) virDomainObjSetState(vm, VIR_DOMAIN_PAUSED, VIR_DOMAIN_PAUSED_CRAS= HED); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -2121,7 +2121,7 @@ qemuDomainDestroyFlags(virDomainPtr dom, endjob: if (ret =3D=3D 0) qemuDomainRemoveInactive(driver, vm); - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -2276,7 +2276,7 @@ static int qemuDomainSetMemoryFlags(virDomainPtr dom,= unsigned long newmem, priv =3D vm->privateData; qemuDomainObjEnterMonitor(driver, vm); r =3D qemuMonitorSetBalloon(priv->mon, newmem); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (r < 0) goto endjob; =20 @@ -2299,7 +2299,7 @@ static int qemuDomainSetMemoryFlags(virDomainPtr dom,= unsigned long newmem, =20 ret =3D 0; endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -2357,7 +2357,7 @@ static int qemuDomainSetMemoryStatsPeriod(virDomainPt= r dom, int period, =20 qemuDomainObjEnterMonitor(driver, vm); r =3D qemuMonitorSetMemoryStatsPeriod(priv->mon, def->memballoon, = period); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (r < 0) { virReportError(VIR_ERR_OPERATION_INVALID, "%s", _("unable to set balloon driver collection peri= od")); @@ -2382,7 +2382,7 @@ static int qemuDomainSetMemoryStatsPeriod(virDomainPt= r dom, int period, =20 ret =3D 0; endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -2414,10 +2414,10 @@ static int qemuDomainInjectNMI(virDomainPtr domain,= unsigned int flags) =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorInjectNMI(priv->mon); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -2473,10 +2473,10 @@ static int qemuDomainSendKey(virDomainPtr domain, =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorSendKey(priv->mon, holdtime, keycodes, nkeycodes); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -2739,7 +2739,7 @@ qemuDomainSaveInternal(virQEMUDriver *driver, virErrorRestore(&save_err); } } - qemuDomainObjEndAsyncJob(driver, vm); + qemuDomainObjEndAsyncJob(vm); if (ret =3D=3D 0) qemuDomainRemoveInactiveJob(driver, vm); =20 @@ -3013,14 +3013,14 @@ qemuDumpToFd(virQEMUDriver *driver, _("unsupported dumpformat '%s' " "for this QEMU binary"), dumpformat); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); return -1; } } =20 ret =3D qemuMonitorDumpToFd(priv->mon, fd, dumpformat, detach); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (ret < 0) return -1; =20 @@ -3199,7 +3199,7 @@ qemuDomainCoreDumpWithFormat(virDomainPtr dom, if ((ret =3D=3D 0) && (flags & VIR_DUMP_RESET)) { qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorSystemReset(priv->mon); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); } =20 if (resume && virDomainObjIsActive(vm)) { @@ -3216,7 +3216,7 @@ qemuDomainCoreDumpWithFormat(virDomainPtr dom, } } =20 - qemuDomainObjEndAsyncJob(driver, vm); + qemuDomainObjEndAsyncJob(vm); if (ret =3D=3D 0 && flags & VIR_DUMP_CRASH) qemuDomainRemoveInactiveJob(driver, vm); =20 @@ -3313,10 +3313,10 @@ qemuDomainScreenshot(virDomainPtr dom, =20 qemuDomainObjEnterMonitor(driver, vm); if (qemuMonitorScreendump(priv->mon, videoAlias, screen, tmp) < 0) { - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); goto endjob; } - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (VIR_CLOSE(tmp_fd) < 0) { virReportSystemError(errno, _("unable to close %s"), tmp); @@ -3336,7 +3336,7 @@ qemuDomainScreenshot(virDomainPtr dom, if (unlink_tmp) unlink(tmp); =20 - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -3412,7 +3412,7 @@ processWatchdogEvent(virQEMUDriver *driver, } =20 endjob: - qemuDomainObjEndAsyncJob(driver, vm); + qemuDomainObjEndAsyncJob(vm); } =20 static int @@ -3524,7 +3524,7 @@ processGuestPanicEvent(virQEMUDriver *driver, } =20 endjob: - qemuDomainObjEndAsyncJob(driver, vm); + qemuDomainObjEndAsyncJob(vm); if (removeInactive) qemuDomainRemoveInactiveJob(driver, vm); } @@ -3561,7 +3561,7 @@ processDeviceDeletedEvent(virQEMUDriver *driver, qemuDomainSaveStatus(vm); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); } =20 =20 @@ -3816,7 +3816,7 @@ processNicRxFilterChangedEvent(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorQueryRxFilter(priv->mon, devAlias, &guestFilter); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (ret < 0) goto endjob; =20 @@ -3858,7 +3858,7 @@ processNicRxFilterChangedEvent(virQEMUDriver *driver, } =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virNetDevRxFilterFree(hostFilter); @@ -3941,7 +3941,7 @@ processSerialChangedEvent(virQEMUDriver *driver, } =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); } =20 =20 @@ -3980,7 +3980,7 @@ processBlockJobEvent(virQEMUDriver *driver, qemuBlockJobUpdate(vm, job, QEMU_ASYNC_JOB_NONE); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); } =20 =20 @@ -4000,7 +4000,7 @@ processJobStatusChangeEvent(virQEMUDriver *driver, qemuBlockJobUpdate(vm, job, QEMU_ASYNC_JOB_NONE); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); } =20 =20 @@ -4046,7 +4046,7 @@ processMonitorEOFEvent(virQEMUDriver *driver, =20 endjob: qemuDomainRemoveInactive(driver, vm); - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); } =20 =20 @@ -4160,7 +4160,7 @@ processMemoryDeviceSizeChange(virQEMUDriver *driver, mem->currentsiz= e); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); virObjectEventStateQueue(driver->domainEventState, event); } =20 @@ -4369,7 +4369,7 @@ qemuDomainSetVcpusFlags(virDomainPtr dom, if (useAgent) qemuDomainObjEndAgentJob(vm); else - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -4525,7 +4525,7 @@ qemuDomainPinVcpuFlags(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -4673,7 +4673,7 @@ qemuDomainPinEmulator(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virObjectEventStateQueue(driver->domainEventState, event); @@ -4856,7 +4856,7 @@ qemuDomainGetIOThreadsMon(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorGetIOThreads(priv->mon, iothreads, niothreads); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 return ret; } @@ -4918,7 +4918,7 @@ qemuDomainGetIOThreadsLive(virQEMUDriver *driver, ret =3D niothreads; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: if (info_ret) { @@ -5083,7 +5083,7 @@ qemuDomainPinIOThread(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virObjectEventStateQueue(driver->domainEventState, event); @@ -5130,7 +5130,7 @@ qemuDomainHotplugAddIOThread(virQEMUDriver *driver, if (qemuMonitorGetIOThreads(priv->mon, &new_iothreads, &new_niothreads= ) < 0) goto exit_monitor; =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (new_niothreads !=3D exp_niothreads) { virReportError(VIR_ERR_INTERNAL_ERROR, @@ -5177,7 +5177,7 @@ qemuDomainHotplugAddIOThread(virQEMUDriver *driver, if (qemuMonitorDelObject(priv->mon, alias, true) < 0) VIR_WARN("deletion of iothread object %d of domain %s fail= ed when cleanup", iothread_id, vm->def->name); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); } } =20 @@ -5191,7 +5191,7 @@ qemuDomainHotplugAddIOThread(virQEMUDriver *driver, return ret; =20 exit_monitor: - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); goto cleanup; } =20 @@ -5214,7 +5214,7 @@ qemuDomainHotplugModIOThread(virQEMUDriver *driver, =20 rc =3D qemuMonitorSetIOThread(priv->mon, &iothread); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rc < 0) return -1; @@ -5250,7 +5250,7 @@ qemuDomainHotplugDelIOThread(virQEMUDriver *driver, if (qemuMonitorGetIOThreads(priv->mon, &new_iothreads, &new_niothreads= ) < 0) goto exit_monitor; =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (new_niothreads !=3D exp_niothreads) { virReportError(VIR_ERR_INTERNAL_ERROR, @@ -5279,7 +5279,7 @@ qemuDomainHotplugDelIOThread(virQEMUDriver *driver, return ret; =20 exit_monitor: - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); goto cleanup; } =20 @@ -5488,7 +5488,7 @@ qemuDomainChgIOThread(virQEMUDriver *driver, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 return ret; } @@ -5832,7 +5832,7 @@ qemuDomainRestoreFlags(virConnectPtr conn, ret =3D qemuSaveImageStartVM(conn, driver, vm, &fd, data, path, false, reset_nvram, QEMU_ASYNC_JOB_START); =20 - qemuProcessEndJob(driver, vm); + qemuProcessEndJob(vm); =20 cleanup: VIR_FORCE_CLOSE(fd); @@ -6420,7 +6420,7 @@ qemuDomainCreateWithFlags(virDomainPtr dom, unsigned = int flags) ret =3D 0; =20 endjob: - qemuProcessEndJob(driver, vm); + qemuProcessEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -6645,7 +6645,7 @@ qemuDomainUndefineFlags(virDomainPtr dom, =20 ret =3D 0; endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -7823,7 +7823,7 @@ qemuDomainAttachDeviceFlags(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -7926,7 +7926,7 @@ static int qemuDomainUpdateDeviceFlags(virDomainPtr d= om, } =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: if (dev !=3D dev_copy) @@ -8108,7 +8108,7 @@ qemuDomainDetachDeviceFlags(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -8143,7 +8143,7 @@ qemuDomainDetachDeviceAlias(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -8242,7 +8242,7 @@ static int qemuDomainSetAutostart(virDomainPtr dom, vm->autostart =3D autostart; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); } ret =3D 0; =20 @@ -8382,7 +8382,7 @@ qemuDomainSetBlkioParameters(virDomainPtr dom, } =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -8554,7 +8554,7 @@ qemuDomainSetMemoryParameters(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -8835,7 +8835,7 @@ qemuDomainSetNumaParameters(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -9028,7 +9028,7 @@ qemuDomainSetPerfEvents(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -9090,7 +9090,7 @@ qemuDomainGetPerfEvents(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -9479,7 +9479,7 @@ qemuDomainSetSchedulerParametersFlags(virDomainPtr do= m, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -9793,15 +9793,15 @@ qemuDomainBlockResize(virDomainPtr dom, =20 qemuDomainObjEnterMonitor(driver, vm); if (qemuMonitorBlockResize(priv->mon, device, nodename, size) < 0) { - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); goto endjob; } - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -9888,7 +9888,7 @@ qemuDomainBlocksStatsGather(virQEMUDriver *driver, rc =3D qemuMonitorBlockStatsUpdateCapacity(priv->mon, blocksta= ts); } =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (nstats < 0 || rc < 0) return -1; @@ -9985,7 +9985,7 @@ qemuDomainBlockStats(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -10073,7 +10073,7 @@ qemuDomainBlockStatsFlags(virDomainPtr dom, *nparams =3D nstats; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: VIR_FREE(blockstats); @@ -10376,7 +10376,7 @@ qemuDomainSetInterfaceParameters(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -10509,7 +10509,7 @@ qemuDomainMemoryStatsInternal(virQEMUDriver *driver, qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorGetMemoryStats(qemuDomainGetMonitor(vm), vm->def->memballoon, stats, nr_sta= ts); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (ret < 0 || ret >=3D nr_stats) return ret; @@ -10552,7 +10552,7 @@ qemuDomainMemoryStats(virDomainPtr dom, =20 ret =3D qemuDomainMemoryStatsInternal(driver, vm, stats, nr_stats); =20 - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -10677,16 +10677,16 @@ qemuDomainMemoryPeek(virDomainPtr dom, qemuDomainObjEnterMonitor(driver, vm); if (flags =3D=3D VIR_MEMORY_VIRTUAL) { if (qemuMonitorSaveVirtualMemory(priv->mon, offset, size, tmp) < 0= ) { - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); goto endjob; } } else { if (qemuMonitorSavePhysicalMemory(priv->mon, offset, size, tmp) < = 0) { - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); goto endjob; } } - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 /* Read the memory file into buffer. */ if (saferead(fd, buffer, size) =3D=3D (ssize_t)-1) { @@ -10699,7 +10699,7 @@ qemuDomainMemoryPeek(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: VIR_FORCE_CLOSE(fd); @@ -11012,7 +11012,7 @@ qemuDomainGetBlockInfo(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); cleanup: VIR_FREE(entry); virDomainObjEndAPI(&vm); @@ -12461,7 +12461,7 @@ qemuDomainGetJobInfoDumpStats(virQEMUDriver *driver, =20 rc =3D qemuMonitorQueryDump(priv->mon, &stats); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) return -1; =20 @@ -12563,7 +12563,7 @@ qemuDomainGetJobStatsInternal(virQEMUDriver *driver, ret =3D 0; =20 cleanup: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); return ret; } =20 @@ -12660,7 +12660,7 @@ qemuDomainAbortJobMigration(virDomainObj *vm) qemuDomainObjAbortAsyncJob(vm); qemuDomainObjEnterMonitor(priv->driver, vm); ret =3D qemuMonitorMigrateCancel(priv->mon); - qemuDomainObjExitMonitor(priv->driver, vm); + qemuDomainObjExitMonitor(vm); =20 return ret; } @@ -12748,7 +12748,7 @@ static int qemuDomainAbortJob(virDomainPtr dom) } =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -12801,7 +12801,7 @@ qemuDomainMigrateSetMaxDowntime(virDomainPtr dom, } else { qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorSetMigrationDowntime(priv->mon, downtime); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) goto endjob; } @@ -12809,7 +12809,7 @@ qemuDomainMigrateSetMaxDowntime(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -12862,7 +12862,7 @@ qemuDomainMigrateGetMaxDowntime(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -12917,7 +12917,7 @@ qemuDomainMigrateGetCompressionCache(virDomainPtr d= om, } else { qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorGetMigrationCacheSize(priv->mon, cacheSize); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) goto endjob; } @@ -12925,7 +12925,7 @@ qemuDomainMigrateGetCompressionCache(virDomainPtr d= om, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -12983,7 +12983,7 @@ qemuDomainMigrateSetCompressionCache(virDomainPtr d= om, } else { qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorSetMigrationCacheSize(priv->mon, cacheSize); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) goto endjob; } @@ -12991,7 +12991,7 @@ qemuDomainMigrateSetCompressionCache(virDomainPtr d= om, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -13072,7 +13072,7 @@ qemuDomainMigrateSetMaxSpeed(virDomainPtr dom, =20 qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorSetMigrationSpeed(priv->mon, bandwidth); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) goto endjob; } @@ -13083,7 +13083,7 @@ qemuDomainMigrateSetMaxSpeed(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -13138,7 +13138,7 @@ qemuDomainMigrationGetPostcopyBandwidth(virQEMUDriv= er *driver, ret =3D 0; =20 cleanup: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); return ret; } =20 @@ -13221,10 +13221,10 @@ qemuDomainMigrateStartPostCopy(virDomainPtr dom, VIR_DEBUG("Starting post-copy"); qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorMigrateStartPostCopy(priv->mon); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -13955,10 +13955,10 @@ qemuDomainQemuMonitorCommandWithFiles(virDomainPt= r domain, =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorArbitraryCommand(priv->mon, cmd, fd, result, hmp); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -14240,7 +14240,7 @@ qemuDomainBlockPivot(virQEMUDriver *driver, } else { ret =3D qemuMonitorDrivePivot(priv->mon, job->name); } - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 /* The pivot failed. The block job in QEMU remains in the synchronised= state */ if (ret < 0) @@ -14371,7 +14371,7 @@ qemuDomainBlockPullCommon(virDomainObj *vm, (!baseSource || basePath)) ret =3D qemuMonitorBlockStream(priv->mon, device, jobname, persist= job, basePath, nodebase, backingPath, speed); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (ret < 0) goto endjob; @@ -14379,7 +14379,7 @@ qemuDomainBlockPullCommon(virDomainObj *vm, qemuBlockJobStarted(job, vm); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: qemuBlockJobStartupFinalize(vm, job); @@ -14445,7 +14445,7 @@ qemuDomainBlockJobAbort(virDomainPtr dom, } else { qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorBlockJobCancel(priv->mon, job->name, false); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (ret < 0) goto endjob; @@ -14487,7 +14487,7 @@ qemuDomainBlockJobAbort(virDomainPtr dom, endjob: if (job && !async) qemuBlockJobSyncEnd(vm, job, QEMU_ASYNC_JOB_NONE); - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -14589,7 +14589,7 @@ qemuDomainGetBlockJobInfo(virDomainPtr dom, =20 qemuDomainObjEnterMonitor(driver, vm); blockjobstats =3D qemuMonitorGetAllBlockJobInfo(qemuDomainGetMonitor(v= m), true); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (!blockjobstats) goto endjob; @@ -14603,7 +14603,7 @@ qemuDomainGetBlockJobInfo(virDomainPtr dom, ret =3D 1; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -14662,10 +14662,10 @@ qemuDomainBlockJobSetSpeed(virDomainPtr dom, ret =3D qemuMonitorBlockJobSetSpeed(qemuDomainGetMonitor(vm), job->name, speed); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -15061,7 +15061,7 @@ qemuDomainBlockCopyCommon(virDomainObj *vm, if (data) { qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuBlockStorageSourceChainAttach(priv->mon, data); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rc < 0) goto endjob; @@ -15096,7 +15096,7 @@ qemuDomainBlockCopyCommon(virDomainObj *vm, } =20 virDomainAuditDisk(vm, NULL, mirror, "mirror", ret >=3D 0); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (ret < 0) { qemuDomainStorageSourceChainAccessRevoke(driver, vm, mirror); goto endjob; @@ -15118,7 +15118,7 @@ qemuDomainBlockCopyCommon(virDomainObj *vm, qemuBlockStorageSourceChainDetach(priv->mon, data); if (crdata) qemuBlockStorageSourceAttachRollback(priv->mon, crdata->sr= cdata[0]); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); } if (need_revoke) qemuDomainStorageSourceChainAccessRevoke(driver, vm, mirror); @@ -15126,7 +15126,7 @@ qemuDomainBlockCopyCommon(virDomainObj *vm, if (need_unlink && virStorageSourceUnlink(mirror) < 0) VIR_WARN("%s", _("unable to remove just-created copy target")); virStorageSourceDeinit(mirror); - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); qemuBlockJobStartupFinalize(vm, job); =20 return ret; @@ -15534,7 +15534,7 @@ qemuDomainBlockCommit(virDomainPtr dom, topPath, nodetop, basePath, nodebase, backingPath, speed); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (ret < 0) goto endjob; @@ -15559,7 +15559,7 @@ qemuDomainBlockCommit(virDomainPtr dom, virErrorRestore(&orig_err); } qemuBlockJobStartupFinalize(vm, job); - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -15627,10 +15627,10 @@ qemuDomainOpenGraphics(virDomainPtr dom, qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorOpenGraphics(priv->mon, protocol, fd, "graphicsfd", (flags & VIR_DOMAIN_OPEN_GRAPHICS_SKIPAU= TH) !=3D 0); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -15703,8 +15703,8 @@ qemuDomainOpenGraphicsFD(virDomainPtr dom, qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorOpenGraphics(priv->mon, protocol, pair[1], "graphic= sfd", (flags & VIR_DOMAIN_OPEN_GRAPHICS_SKIPAU= TH)); - qemuDomainObjExitMonitor(driver, vm); - qemuDomainObjEndJob(driver, vm); + qemuDomainObjExitMonitor(vm); + qemuDomainObjEndJob(vm); if (ret < 0) goto cleanup; =20 @@ -16139,7 +16139,7 @@ qemuDomainSetBlockIoTune(virDomainPtr dom, =20 qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorSetBlockIoThrottle(priv->mon, drivealias, qd= evid, &info); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rc < 0) goto endjob; @@ -16189,7 +16189,7 @@ qemuDomainSetBlockIoTune(virDomainPtr dom, =20 ret =3D 0; endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: VIR_FREE(info.group_name); @@ -16271,7 +16271,7 @@ qemuDomainGetBlockIoTune(virDomainPtr dom, } qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorGetBlockIoThrottle(priv->mon, drivealias, qdevid= , &reply); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rc < 0) goto endjob; @@ -16344,7 +16344,7 @@ qemuDomainGetBlockIoTune(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: VIR_FREE(reply.group_name); @@ -16391,7 +16391,7 @@ qemuDomainGetDiskErrors(virDomainPtr dom, =20 qemuDomainObjEnterMonitor(driver, vm); table =3D qemuMonitorGetBlockInfo(priv->mon); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (!table) goto endjob; =20 @@ -16418,7 +16418,7 @@ qemuDomainGetDiskErrors(virDomainPtr dom, ret =3D n; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -16466,7 +16466,7 @@ qemuDomainSetMetadata(virDomainPtr dom, virObjectEventStateQueue(driver->domainEventState, ev); } =20 - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -16555,7 +16555,7 @@ qemuDomainProbeQMPCurrentMachine(virQEMUDriver *dri= ver, =20 qemuDomainObjEnterMonitor(driver, vm); rv =3D qemuMonitorGetCurrentMachineInfo(priv->mon, &info); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rv < 0) return -1; =20 @@ -16585,7 +16585,7 @@ qemuDomainQueryWakeupSuspendSupport(virQEMUDriver *= driver, ret =3D qemuDomainProbeQMPCurrentMachine(driver, vm, wakeupSupported); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); return ret; } =20 @@ -16720,10 +16720,10 @@ qemuDomainPMWakeup(virDomainPtr dom, =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorSystemWakeup(priv->mon); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -17106,7 +17106,7 @@ qemuDomainGetHostnameLease(virQEMUDriver *driver, =20 ret =3D 0; endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); return ret; } =20 @@ -17280,7 +17280,7 @@ qemuDomainSetTime(virDomainPtr dom, if (virQEMUCapsGet(priv->qemuCaps, QEMU_CAPS_RTC_RESET_REINJECTION)) { qemuDomainObjEnterMonitor(driver, vm); rv =3D qemuMonitorRTCResetReinjection(priv->mon); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rv < 0) goto endjob; @@ -17289,7 +17289,7 @@ qemuDomainSetTime(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -18412,7 +18412,7 @@ qemuDomainGetStatsBlock(virQEMUDriver *driver, if (fetchnodedata) nodedata =3D qemuMonitorQueryNamedBlockNodes(priv->mon); =20 - qemuDomainObjExitMonitor(driver, dom); + qemuDomainObjExitMonitor(dom); =20 /* failure to retrieve stats is fine at this point */ if (rc < 0 || (fetchnodedata && !nodedata)) @@ -18546,7 +18546,7 @@ qemuDomainGetStatsDirtyRateMon(virQEMUDriver *drive= r, =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorQueryDirtyRate(priv->mon, info); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 return ret; } @@ -18814,7 +18814,7 @@ qemuConnectGetAllDomainStats(virConnectPtr conn, rc =3D qemuDomainGetStats(conn, vm, requestedStats, &tmp, domflags= ); =20 if (HAVE_JOB(domflags)) - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 virObjectUnlock(vm); =20 @@ -18995,7 +18995,7 @@ qemuDomainGetFSInfo(virDomainPtr dom, ret =3D virDomainFSInfoFormat(agentinfo, nfs, vm->def, info); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: if (agentinfo) { @@ -19311,7 +19311,7 @@ static int qemuDomainRename(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -19571,7 +19571,7 @@ qemuDomainSetVcpu(virDomainPtr dom, ret =3D qemuDomainSetVcpuInternal(driver, vm, def, persistentDef, map,= !!state); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -19640,7 +19640,7 @@ qemuDomainSetBlockThreshold(virDomainPtr dom, =20 qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorSetBlockThreshold(priv->mon, nodename, threshold); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) goto endjob; =20 @@ -19651,7 +19651,7 @@ qemuDomainSetBlockThreshold(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -19756,7 +19756,7 @@ qemuDomainModifyLifecycleActionLive(virDomainObj *v= m, QEMU_MONITOR_ACTION_WATCHDOG_KEEP, QEMU_MONITOR_ACTION_PANIC_KEEP); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) return -1; =20 @@ -19834,7 +19834,7 @@ qemuDomainSetLifecycleAction(virDomainPtr dom, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -19954,13 +19954,13 @@ qemuDomainGetSEVInfo(virQEMUDriver *driver, =20 =20 if (!tmp) { - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); goto endjob; } =20 rv =3D qemuMonitorGetSEVInfo(QEMU_DOMAIN_PRIVATE(vm)->mon, &apiMajor, &apiMinor, &buildID, &policy); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rv < 0) goto endjob; @@ -19989,7 +19989,7 @@ qemuDomainGetSEVInfo(virQEMUDriver *driver, ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); return ret; } =20 @@ -20109,14 +20109,14 @@ qemuDomainSetLaunchSecurityState(virDomainPtr dom= ain, qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorSetLaunchSecurityState(QEMU_DOMAIN_PRIVATE(vm)->mon, secrethdr, secret, setaddr, has= Setaddr); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) goto endjob; =20 ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); @@ -20509,7 +20509,7 @@ qemuDomainGetGuestInfo(virDomainPtr dom, qemuAgentDiskInfoFormatParams(agentdiskinfo, ndisks, vm->def, = params, nparams, &maxparams); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); } =20 if (nifaces > 0) { @@ -20772,10 +20772,10 @@ qemuDomainStartDirtyRateCalc(virDomainPtr dom, qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorStartDirtyRateCalc(priv->mon, seconds, mode); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virDomainObjEndAPI(&vm); diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c index c70bc361fd..8ea95406c7 100644 --- a/src/qemu/qemu_hotplug.c +++ b/src/qemu/qemu_hotplug.c @@ -110,7 +110,7 @@ qemuDomainDeleteDevice(virDomainObj *vm, =20 rc =3D qemuMonitorDelDevice(priv->mon, alias); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rc < 0) { /* Deleting device failed. Let's check if DEVICE_DELETED @@ -313,7 +313,7 @@ qemuDomainChangeMediaLegacy(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorEjectMedia(priv->mon, driveAlias, force); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 /* If the tray is present wait for it to open. */ if (!force && diskPriv->tray) { @@ -324,7 +324,7 @@ qemuDomainChangeMediaLegacy(virQEMUDriver *driver, /* re-issue ejection command to pop out the media */ qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorEjectMedia(priv->mon, driveAlias, false); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) return -1; =20 @@ -346,7 +346,7 @@ qemuDomainChangeMediaLegacy(virQEMUDriver *driver, driveAlias, sourcestr, format); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); } =20 if (rc < 0) @@ -394,7 +394,7 @@ qemuHotplugAttachDBusVMState(virQEMUDriver *driver, if (ret =3D=3D 0) priv->dbusVMState =3D true; =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 return ret; } @@ -430,7 +430,7 @@ qemuHotplugRemoveDBusVMState(virQEMUDriver *driver, if (ret =3D=3D 0) priv->dbusVMState =3D false; =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 return ret; } @@ -477,7 +477,7 @@ qemuHotplugAttachManagedPR(virQEMUDriver *driver, =20 rc =3D qemuMonitorAddObject(priv->mon, &props, NULL); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) goto cleanup; =20 @@ -517,7 +517,7 @@ qemuHotplugRemoveManagedPR(virQEMUDriver *driver, goto cleanup; ignore_value(qemuMonitorDelObject(priv->mon, qemuDomainGetManagedPRAli= as(), false)); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 qemuProcessKillManagedPRDaemon(vm); =20 @@ -574,7 +574,7 @@ qemuDomainChangeMediaBlockdev(virQEMUDriver *driver, if (diskPriv->tray && disk->tray_status !=3D VIR_DOMAIN_DISK_TRAY_OPEN= ) { qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorBlockdevTrayOpen(priv->mon, diskPriv->qomName, f= orce); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) return -1; =20 @@ -613,7 +613,7 @@ qemuDomainChangeMediaBlockdev(virQEMUDriver *driver, if (rc < 0 && newbackend) qemuBlockStorageSourceChainDetach(priv->mon, newbackend); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) return -1; =20 @@ -782,7 +782,7 @@ qemuDomainAttachDiskGeneric(virQEMUDriver *driver, =20 rc =3D qemuBlockStorageSourceChainAttach(priv->mon, data); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rc < 0) goto rollback; @@ -832,7 +832,7 @@ qemuDomainAttachDiskGeneric(virQEMUDriver *driver, VIR_WARN("failed to set blkdeviotune for '%s' of '%s'", disk->= dst, vm->def->name); } =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rc < 0) goto rollback; @@ -848,7 +848,7 @@ qemuDomainAttachDiskGeneric(virQEMUDriver *driver, =20 qemuBlockStorageSourceChainDetach(priv->mon, data); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 return -1; } @@ -912,7 +912,7 @@ int qemuDomainAttachControllerDevice(virQEMUDriver *dri= ver, ignore_value(qemuDomainDetachExtensionDevice(priv->mon, &controlle= r->info)); =20 exit_monitor: - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (ret =3D=3D 0) virDomainControllerInsertPreAlloced(vm->def, controller); @@ -1461,7 +1461,7 @@ qemuDomainAttachNetDevice(virQEMUDriver *driver, if (qemuMonitorAddFileHandleToSet(priv->mon, vdpafd, -1, net->data.vdpa.devicepath, &fdinfo) < 0) { - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); goto cleanup; } vdpafdName =3D g_strdup_printf("/dev/fdset/%d", fdinfo.fdset); @@ -1471,13 +1471,13 @@ qemuDomainAttachNetDevice(virQEMUDriver *driver, tapfdName, tapfdSize, vhostfdName, vhostfdSize, slirpfdName, vdpafdName))) { - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); goto cleanup; } =20 if (actualType =3D=3D VIR_DOMAIN_NET_TYPE_VHOSTUSER) { if (qemuHotplugChardevAttach(priv->mon, charDevAlias, net->data.vh= ostuser) < 0) { - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); virDomainAuditNet(vm, NULL, net, "attach", false); goto cleanup; } @@ -1488,13 +1488,13 @@ qemuDomainAttachNetDevice(virQEMUDriver *driver, tapfd, tapfdName, tapfdSize, vhostfd, vhostfdName, vhostfdSize, slirpfd, slirpfdName) < 0) { - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); virDomainAuditNet(vm, NULL, net, "attach", false); goto try_remove; } netdevPlugged =3D true; =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 for (i =3D 0; i < tapfdSize; i++) VIR_FORCE_CLOSE(tapfd[i]); @@ -1507,18 +1507,18 @@ qemuDomainAttachNetDevice(virQEMUDriver *driver, qemuDomainObjEnterMonitor(driver, vm); =20 if (qemuDomainAttachExtensionDevice(priv->mon, &net->info) < 0) { - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); virDomainAuditNet(vm, NULL, net, "attach", false); goto try_remove; } =20 if (qemuMonitorAddDeviceProps(priv->mon, &nicprops) < 0) { ignore_value(qemuDomainDetachExtensionDevice(priv->mon, &net->info= )); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); virDomainAuditNet(vm, NULL, net, "attach", false); goto try_remove; } - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 /* set link state */ if (net->linkstate =3D=3D VIR_DOMAIN_NET_INTERFACE_LINK_STATE_DOWN) { @@ -1529,12 +1529,12 @@ qemuDomainAttachNetDevice(virQEMUDriver *driver, qemuDomainObjEnterMonitor(driver, vm); =20 if (qemuMonitorSetLink(priv->mon, net->info.alias, VIR_DOMAIN_= NET_INTERFACE_LINK_STATE_DOWN) < 0) { - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); virDomainAuditNet(vm, NULL, net, "attach", false); goto try_remove; } =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); } /* link set to down */ } @@ -1628,7 +1628,7 @@ qemuDomainAttachNetDevice(virQEMUDriver *driver, qemuMonitorRemoveNetdev(priv->mon, netdev_name) < 0) VIR_WARN("Failed to remove network backend for netdev %s", netdev_name); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); virErrorRestore(&originalError); goto cleanup; } @@ -1739,7 +1739,7 @@ qemuDomainAttachHostPCIDevice(virQEMUDriver *driver, ignore_value(qemuDomainDetachExtensionDevice(priv->mon, hostdev->i= nfo)); =20 exit_monitor: - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 done: virDomainAuditHostdev(vm, hostdev, "attach", ret =3D=3D 0); @@ -1795,7 +1795,7 @@ qemuDomainDelTLSObjects(virQEMUDriver *driver, if (secAlias) ignore_value(qemuMonitorDelObject(priv->mon, secAlias, false)); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 cleanup: virErrorRestore(&orig_err); @@ -1827,12 +1827,12 @@ qemuDomainAddTLSObjects(virQEMUDriver *driver, qemuMonitorAddObject(priv->mon, tlsProps, NULL) < 0) goto error; =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); return 0; =20 error: virErrorPreserveLast(&orig_err); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); virErrorRestore(&orig_err); qemuDomainDelTLSObjects(driver, vm, asyncJob, secAlias, NULL); =20 @@ -1948,7 +1948,7 @@ qemuDomainDelChardevTLSObjects(virQEMUDriver *driver, if (secAlias) ignore_value(qemuMonitorDelObject(priv->mon, secAlias, false)); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 return 0; } @@ -1995,7 +1995,7 @@ int qemuDomainAttachRedirdevDevice(virQEMUDriver *dri= ver, if (qemuMonitorAddDeviceProps(priv->mon, &devprops) < 0) goto exit_monitor; =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 def->redirdevs[def->nredirdevs++] =3D redirdev; ret =3D 0; @@ -2011,7 +2011,7 @@ int qemuDomainAttachRedirdevDevice(virQEMUDriver *dri= ver, /* detach associated chardev on error */ if (chardevAdded) ignore_value(qemuMonitorDetachCharDev(priv->mon, charAlias)); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); virErrorRestore(&orig_err); qemuDomainDelTLSObjects(driver, vm, QEMU_ASYNC_JOB_NONE, secAlias, tlsAlias); @@ -2238,7 +2238,7 @@ qemuDomainAttachChrDevice(virQEMUDriver *driver, qemuFDPassTransferMonitor(charpriv->logfd, priv->mon) < 0) goto exit_monitor; =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); } =20 if (guestfwd) { @@ -2277,7 +2277,7 @@ qemuDomainAttachChrDevice(virQEMUDriver *driver, goto exit_monitor; } =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 qemuDomainChrInsertPreAlloced(vmdef, chr); ret =3D 0; @@ -2305,7 +2305,7 @@ qemuDomainAttachChrDevice(virQEMUDriver *driver, qemuMonitorDetachCharDev(priv->mon, charAlias); qemuFDPassTransferMonitorRollback(charpriv->sourcefd, priv->mon); qemuFDPassTransferMonitorRollback(charpriv->logfd, priv->mon); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); virErrorRestore(&orig_err); =20 qemuDomainDelTLSObjects(driver, vm, QEMU_ASYNC_JOB_NONE, @@ -2385,7 +2385,7 @@ qemuDomainAttachRNGDevice(virQEMUDriver *driver, goto exit_monitor; } =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 VIR_APPEND_ELEMENT_INPLACE(vm->def->rngs, vm->def->nrngs, rng); =20 @@ -2411,7 +2411,7 @@ qemuDomainAttachRNGDevice(virQEMUDriver *driver, ignore_value(qemuMonitorDelObject(priv->mon, objAlias, false)); if (rng->backend =3D=3D VIR_DOMAIN_RNG_BACKEND_EGD && chardevAdded) ignore_value(qemuMonitorDetachCharDev(priv->mon, charAlias)); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); virErrorRestore(&orig_err); =20 qemuDomainDelTLSObjects(driver, vm, QEMU_ASYNC_JOB_NONE, @@ -2504,7 +2504,7 @@ qemuDomainAttachMemory(virQEMUDriver *driver, if (qemuMonitorAddDeviceProps(priv->mon, &devprops) < 0) goto exit_monitor; =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 event =3D virDomainEventDeviceAddedNewFromObj(vm, objalias); virObjectEventStateQueue(driver->domainEventState, event); @@ -2543,7 +2543,7 @@ qemuDomainAttachMemory(virQEMUDriver *driver, virErrorPreserveLast(&orig_err); if (objAdded) ignore_value(qemuMonitorDelObject(priv->mon, objalias, false)); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (objAdded && mem) ignore_value(qemuProcessDestroyMemoryBackingPath(driver, vm, mem)); @@ -2608,7 +2608,7 @@ qemuDomainAttachHostUSBDevice(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorAddDeviceProps(priv->mon, &devprops); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); virDomainAuditHostdev(vm, hostdev, "attach", ret =3D=3D 0); if (ret < 0) goto cleanup; @@ -2701,7 +2701,7 @@ qemuDomainAttachHostSCSIDevice(virQEMUDriver *driver, if (qemuMonitorAddDeviceProps(priv->mon, &devprops) < 0) goto exit_monitor; =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 virDomainAuditHostdev(vm, hostdev, "attach", true); =20 @@ -2727,7 +2727,7 @@ qemuDomainAttachHostSCSIDevice(virQEMUDriver *driver, exit_monitor: virErrorPreserveLast(&orig_err); qemuBlockStorageSourceAttachRollback(priv->mon, data); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); virErrorRestore(&orig_err); =20 virDomainAuditHostdev(vm, hostdev, "attach", false); @@ -2826,7 +2826,7 @@ qemuDomainAttachSCSIVHostDevice(virQEMUDriver *driver, ignore_value(qemuMonitorCloseFileHandle(priv->mon, vhostfdName)); if (removeextension) ignore_value(qemuDomainDetachExtensionDevice(priv->mon, hostdev->i= nfo)); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (ret < 0) goto audit; =20 @@ -2920,7 +2920,7 @@ qemuDomainAttachMediatedDevice(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorAddDeviceProps(priv->mon, &devprops); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 virDomainAuditHostdev(vm, hostdev, "attach", ret =3D=3D 0); if (ret < 0) @@ -3074,7 +3074,7 @@ qemuDomainAttachShmemDevice(virQEMUDriver *driver, goto exit_monitor; } =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 /* Doing a copy here just so the pointer doesn't get nullified * because we need it in the audit function */ @@ -3101,7 +3101,7 @@ qemuDomainAttachShmemDevice(virQEMUDriver *driver, ignore_value(qemuMonitorDelObject(priv->mon, memAlias, false)); } =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 virErrorRestore(&orig_err); =20 @@ -3200,7 +3200,7 @@ qemuDomainAttachWatchdog(virQEMUDriver *driver, if (rv >=3D 0) rv =3D qemuMonitorAddDeviceProps(priv->mon, &props); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rv < 0) goto cleanup; @@ -3287,7 +3287,7 @@ qemuDomainAttachInputDevice(virQEMUDriver *driver, goto exit_monitor; } =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 VIR_APPEND_ELEMENT_COPY_INPLACE(vm->def->inputs, vm->def->ninputs, inp= ut); =20 @@ -3313,7 +3313,7 @@ qemuDomainAttachInputDevice(virQEMUDriver *driver, return ret; =20 exit_monitor: - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); goto audit; } =20 @@ -3370,7 +3370,7 @@ qemuDomainAttachVsockDevice(virQEMUDriver *driver, if ((ret =3D qemuMonitorAddDeviceProps(priv->mon, &devprops)) < 0) goto exit_monitor; =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 vm->def->vsock =3D g_steal_pointer(&vsock); =20 @@ -3391,7 +3391,7 @@ qemuDomainAttachVsockDevice(virQEMUDriver *driver, ignore_value(qemuMonitorCloseFileHandle(priv->mon, fdname)); if (removeextension) ignore_value(qemuDomainDetachExtensionDevice(priv->mon, &vsock->in= fo)); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); goto cleanup; } =20 @@ -3459,7 +3459,7 @@ qemuDomainAttachFSDevice(virQEMUDriver *driver, goto exit_monitor; } =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 VIR_APPEND_ELEMENT_COPY(vm->def->fss, vm->def->nfss, fs); =20 @@ -3483,7 +3483,7 @@ qemuDomainAttachFSDevice(virQEMUDriver *driver, virErrorPreserveLast(&origErr); if (chardevAdded) ignore_value(qemuMonitorDetachCharDev(priv->mon, charAlias)); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); virErrorRestore(&origErr); goto audit; } @@ -3652,7 +3652,7 @@ int qemuDomainChangeNetLinkState(virQEMUDriver *drive= r, dev->linkstate =3D linkstate; =20 cleanup: - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 return ret; } @@ -4246,7 +4246,7 @@ qemuDomainChangeGraphicsPasswords(virQEMUDriver *driv= er, ret =3D qemuMonitorExpirePassword(priv->mon, type, expire); =20 end_job: - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 return ret; } @@ -4521,7 +4521,7 @@ qemuDomainRemoveDiskDevice(virQEMUDriver *driver, if (diskBackend) qemuBlockStorageSourceChainDetach(priv->mon, diskBackend); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 virDomainAuditDisk(vm, disk->src, NULL, "detach", true); =20 @@ -4593,7 +4593,7 @@ qemuDomainRemoveMemoryDevice(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorDelObject(priv->mon, backendAlias, true); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 virDomainAuditMemory(vm, oldmem, newmem, "update", rc =3D=3D 0); if (rc < 0) @@ -4692,7 +4692,7 @@ qemuDomainRemoveHostDevice(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); qemuBlockStorageSourceAttachRollback(priv->mon, detachscsi); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); } =20 if (hostdev->parentnet) { @@ -4809,7 +4809,7 @@ qemuDomainRemoveNetDevice(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); if (qemuMonitorRemoveNetdev(priv->mon, hostnet_name) < 0) { - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); virDomainAuditNet(vm, net, NULL, "detach", false); return -1; } @@ -4826,7 +4826,7 @@ qemuDomainRemoveNetDevice(virQEMUDriver *driver, qemuHotplugRemoveFDSet(priv->mon, net->info.alias, net->data.vdpa.= devicepath); } =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (QEMU_DOMAIN_NETWORK_PRIVATE(net)->slirp) qemuSlirpStop(QEMU_DOMAIN_NETWORK_PRIVATE(net)->slirp, vm, driver,= net); @@ -4901,7 +4901,7 @@ qemuDomainRemoveChrDevice(virQEMUDriver *driver, qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorDetachCharDev(priv->mon, charAlias); qemuHotplugRemoveFDSet(priv->mon, chr->info.alias, NULL); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); } =20 if (rc =3D=3D 0 && @@ -4967,7 +4967,7 @@ qemuDomainRemoveRNGDevice(virQEMUDriver *driver, qemuMonitorDetachCharDev(priv->mon, charAlias) < 0) rc =3D -1; =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rng->backend =3D=3D VIR_DOMAIN_RNG_BACKEND_EGD && rc =3D=3D 0 && @@ -5021,7 +5021,7 @@ qemuDomainRemoveShmemDevice(virQEMUDriver *driver, else rc =3D qemuMonitorDelObject(priv->mon, memAlias, true); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 virDomainAuditShmem(vm, shmem, "detach", rc =3D=3D 0); =20 @@ -5113,7 +5113,7 @@ qemuDomainRemoveRedirdevDevice(virQEMUDriver *driver, */ ignore_value(qemuMonitorDetachCharDev(priv->mon, charAlias)); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (qemuDomainDelChardevTLSObjects(driver, vm, dev->source, charAlias)= < 0) return -1; @@ -5150,7 +5150,7 @@ qemuDomainRemoveFSDevice(virQEMUDriver *driver, if (qemuMonitorDetachCharDev(priv->mon, charAlias) < 0) rc =3D -1; =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); } =20 virDomainAuditFS(vm, fs, NULL, "detach", rc =3D=3D 0); @@ -5891,7 +5891,7 @@ qemuDomainDetachDeviceChr(virQEMUDriver *driver, int rc; qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorRemoveNetdev(priv->mon, tmpChr->info.alias); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rc < 0) goto cleanup; @@ -6431,7 +6431,7 @@ qemuDomainHotplugAddVcpu(virQEMUDriver *driver, =20 rc =3D qemuMonitorAddDeviceProps(qemuDomainGetMonitor(vm), &vcpuprops); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 virDomainAuditVcpu(vm, oldvcpus, oldvcpus + nvcpus, "update", rc =3D= =3D 0); =20 @@ -6913,7 +6913,7 @@ qemuDomainChangeMemoryRequestedSize(virQEMUDriver *dr= iver, =20 qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorChangeMemoryRequestedSize(priv->mon, alias, requeste= dsize); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 return rc; } diff --git a/src/qemu/qemu_migration.c b/src/qemu/qemu_migration.c index 69bcdafdf8..f109598fb4 100644 --- a/src/qemu/qemu_migration.c +++ b/src/qemu/qemu_migration.c @@ -89,16 +89,14 @@ qemuMigrationJobStart(virQEMUDriver *driver, ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2) G_GNUC_WARN_UNUSED_RESULT; =20 static void -qemuMigrationJobSetPhase(virQEMUDriver *driver, - virDomainObj *vm, +qemuMigrationJobSetPhase(virDomainObj *vm, qemuMigrationJobPhase phase) - ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2); + ATTRIBUTE_NONNULL(1); =20 static void -qemuMigrationJobStartPhase(virQEMUDriver *driver, - virDomainObj *vm, +qemuMigrationJobStartPhase(virDomainObj *vm, qemuMigrationJobPhase phase) - ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2); + ATTRIBUTE_NONNULL(1); =20 static void qemuMigrationJobContinue(virDomainObj *obj) @@ -110,9 +108,8 @@ qemuMigrationJobIsActive(virDomainObj *vm, ATTRIBUTE_NONNULL(1); =20 static void -qemuMigrationJobFinish(virQEMUDriver *driver, - virDomainObj *obj) - ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2); +qemuMigrationJobFinish(virDomainObj *obj) + ATTRIBUTE_NONNULL(1); =20 static void qemuMigrationSrcStoreDomainState(virDomainObj *vm) @@ -515,7 +512,7 @@ qemuMigrationDstStartNBDServer(virQEMUDriver *driver, =20 if (qemuBlockExportAddNBD(vm, diskAlias, disk->src, diskAlias, tru= e, NULL) < 0) goto exit_monitor; - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); } =20 if (server.transport =3D=3D VIR_STORAGE_NET_HOST_TRANS_TCP) @@ -529,7 +526,7 @@ qemuMigrationDstStartNBDServer(virQEMUDriver *driver, return ret; =20 exit_monitor: - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); goto cleanup; } =20 @@ -550,7 +547,7 @@ qemuMigrationDstStopNBDServer(virQEMUDriver *driver, =20 if (qemuMonitorNBDServerStop(priv->mon) < 0) VIR_WARN("Unable to stop NBD server"); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 virPortAllocatorRelease(priv->nbdPort); priv->nbdPort =3D 0; @@ -750,7 +747,7 @@ qemuMigrationSrcNBDCopyCancelOne(virQEMUDriver *driver, * consistency on the destination so that we can force cancel the mirr= or */ rv =3D qemuMonitorBlockJobCancel(priv->mon, job->name, abortMigration); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rv < 0) return -1; =20 @@ -871,7 +868,7 @@ qemuMigrationSrcCancelRemoveTempBitmaps(virDomainObj *v= m, if (qemuDomainObjEnterMonitorAsync(driver, vm, asyncJob) < 0) return -1; qemuMonitorBitmapRemove(priv->mon, t->nodename, t->bitmapname); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); } =20 return 0; @@ -969,7 +966,7 @@ qemuMigrationSrcNBDStorageCopyBlockdev(virQEMUDriver *d= river, if (mon_ret !=3D 0) qemuBlockStorageSourceAttachRollback(qemuDomainGetMonitor(vm), dat= a); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (mon_ret < 0) return -1; =20 @@ -1011,7 +1008,7 @@ qemuMigrationSrcNBDStorageCopyDriveMirror(virQEMUDriv= er *driver, diskAlias, nbd_dest, "raw", mirror_speed, 0, 0, mirror_shallow, t= rue); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (mon_ret < 0) return -1; =20 @@ -1690,7 +1687,7 @@ qemuMigrationAnyFetchStats(virQEMUDriver *driver, =20 rv =3D qemuMonitorGetMigrationStats(priv->mon, &stats, error); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rv < 0) return -1; =20 @@ -2055,7 +2052,7 @@ qemuMigrationSrcGraphicsRelocate(virQEMUDriver *drive= r, ret =3D qemuMonitorGraphicsRelocate(priv->mon, type, listenAddress, port, tlsPort, tlsSubject); jobPriv->spiceMigration =3D !ret; - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); } =20 cleanup: @@ -2159,7 +2156,7 @@ qemuMigrationDstRun(virQEMUDriver *driver, rv =3D qemuMonitorMigrateIncoming(priv->mon, uri); =20 exit_monitor: - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rv < 0) return -1; =20 @@ -2206,7 +2203,7 @@ qemuMigrationSrcCleanup(virDomainObj *vm, switch ((qemuMigrationJobPhase) priv->job.phase) { case QEMU_MIGRATION_PHASE_BEGIN3: /* just forget we were about to migrate */ - qemuMigrationJobFinish(driver, vm); + qemuMigrationJobFinish(vm); break; =20 case QEMU_MIGRATION_PHASE_PERFORM3_DONE: @@ -2216,7 +2213,7 @@ qemuMigrationSrcCleanup(virDomainObj *vm, qemuMigrationParamsReset(driver, vm, QEMU_ASYNC_JOB_MIGRATION_OUT, jobPriv->migParams, priv->job.apiFlags); /* clear the job and let higher levels decide what to do */ - qemuMigrationJobFinish(driver, vm); + qemuMigrationJobFinish(vm); break; =20 case QEMU_MIGRATION_PHASE_PERFORM3: @@ -2352,7 +2349,7 @@ qemuMigrationSrcBeginPhase(virQEMUDriver *driver, * change protection. */ if (priv->job.asyncJob =3D=3D QEMU_ASYNC_JOB_MIGRATION_OUT) - qemuMigrationJobSetPhase(driver, vm, QEMU_MIGRATION_PHASE_BEGIN3); + qemuMigrationJobSetPhase(vm, QEMU_MIGRATION_PHASE_BEGIN3); =20 if (!qemuMigrationSrcIsAllowed(driver, vm, true, flags)) return NULL; @@ -2567,9 +2564,9 @@ qemuMigrationSrcBegin(virConnectPtr conn, =20 endjob: if (flags & VIR_MIGRATE_CHANGE_PROTECTION) - qemuMigrationJobFinish(driver, vm); + qemuMigrationJobFinish(vm); else - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); goto cleanup; } =20 @@ -2594,7 +2591,7 @@ qemuMigrationDstPrepareCleanup(virQEMUDriver *driver, =20 if (!qemuMigrationJobIsActive(vm, QEMU_ASYNC_JOB_MIGRATION_IN)) return; - qemuDomainObjDiscardAsyncJob(driver, vm); + qemuDomainObjDiscardAsyncJob(vm); } =20 static qemuProcessIncomingDef * @@ -2931,7 +2928,7 @@ qemuMigrationDstPrepareAny(virQEMUDriver *driver, if (qemuMigrationJobStart(driver, vm, QEMU_ASYNC_JOB_MIGRATION_IN, flags) < 0) goto cleanup; - qemuMigrationJobSetPhase(driver, vm, QEMU_MIGRATION_PHASE_PREPARE); + qemuMigrationJobSetPhase(vm, QEMU_MIGRATION_PHASE_PREPARE); =20 /* Domain starts inactive, even if the domain XML had an id field. */ vm->def->id =3D -1; @@ -3125,7 +3122,7 @@ qemuMigrationDstPrepareAny(virQEMUDriver *driver, QEMU_ASYNC_JOB_MIGRATION_IN, stopFlags); } =20 - qemuMigrationJobFinish(driver, vm); + qemuMigrationJobFinish(vm); goto cleanup; } =20 @@ -3401,7 +3398,7 @@ qemuMigrationSrcConfirmPhase(virQEMUDriver *driver, =20 virCheckFlags(QEMU_MIGRATION_FLAGS, -1); =20 - qemuMigrationJobSetPhase(driver, vm, + qemuMigrationJobSetPhase(vm, retcode =3D=3D 0 ? QEMU_MIGRATION_PHASE_CONFIRM3 : QEMU_MIGRATION_PHASE_CONFIRM3_CANCELLED); @@ -3507,7 +3504,7 @@ qemuMigrationSrcConfirm(virQEMUDriver *driver, else phase =3D QEMU_MIGRATION_PHASE_CONFIRM3; =20 - qemuMigrationJobStartPhase(driver, vm, phase); + qemuMigrationJobStartPhase(vm, phase); virCloseCallbacksUnset(driver->closeCallbacks, vm, qemuMigrationSrcCleanup); =20 @@ -3515,7 +3512,7 @@ qemuMigrationSrcConfirm(virQEMUDriver *driver, cookiein, cookieinlen, flags, cancelled); =20 - qemuMigrationJobFinish(driver, vm); + qemuMigrationJobFinish(vm); if (!virDomainObjIsActive(vm)) { if (!cancelled && ret =3D=3D 0 && flags & VIR_MIGRATE_UNDEFINE_SOU= RCE) { virDomainDeleteConfig(cfg->configDir, cfg->autostartDir, vm); @@ -3829,7 +3826,7 @@ qemuMigrationSrcContinue(virQEMUDriver *driver, =20 ret =3D qemuMonitorMigrateContinue(priv->mon, status); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 return ret; } @@ -3852,7 +3849,7 @@ qemuMigrationSetDBusVMState(virQEMUDriver *driver, =20 rv =3D qemuMonitorSetDBusVMStateIdList(priv->mon, priv->dbusVMStat= eIds); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 return rv; } else { @@ -3952,7 +3949,7 @@ qemuMigrationSrcRunPrepareBlockDirtyBitmapsMerge(virD= omainObj *vm, =20 rc =3D qemuMonitorTransaction(priv->mon, &actions); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) return -1; =20 @@ -4261,7 +4258,7 @@ qemuMigrationSrcRun(virQEMUDriver *driver, break; } =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) goto error; =20 @@ -4383,7 +4380,7 @@ qemuMigrationSrcRun(virQEMUDriver *driver, qemuDomainObjEnterMonitorAsync(driver, vm, QEMU_ASYNC_JOB_MIGRATION_OUT) = =3D=3D 0) { qemuMonitorMigrateCancel(priv->mon); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); } =20 /* cancel any outstanding NBD jobs */ @@ -4404,7 +4401,7 @@ qemuMigrationSrcRun(virQEMUDriver *driver, goto cleanup; =20 exit_monitor: - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); goto error; } =20 @@ -4657,7 +4654,7 @@ qemuMigrationSrcPerformPeer2Peer2(virQEMUDriver *driv= er, * until the migration is complete. */ VIR_DEBUG("Perform %p", sconn); - qemuMigrationJobSetPhase(driver, vm, QEMU_MIGRATION_PHASE_PERFORM2); + qemuMigrationJobSetPhase(vm, QEMU_MIGRATION_PHASE_PERFORM2); if (flags & VIR_MIGRATE_TUNNELLED) ret =3D qemuMigrationSrcPerformTunnel(driver, vm, st, NULL, NULL, 0, NULL, NULL, @@ -4901,7 +4898,7 @@ qemuMigrationSrcPerformPeer2Peer3(virQEMUDriver *driv= er, * confirm migration completion. */ VIR_DEBUG("Perform3 %p uri=3D%s", sconn, NULLSTR(uri)); - qemuMigrationJobSetPhase(driver, vm, QEMU_MIGRATION_PHASE_PERFORM3); + qemuMigrationJobSetPhase(vm, QEMU_MIGRATION_PHASE_PERFORM3); VIR_FREE(cookiein); cookiein =3D g_steal_pointer(&cookieout); cookieinlen =3D cookieoutlen; @@ -4926,8 +4923,7 @@ qemuMigrationSrcPerformPeer2Peer3(virQEMUDriver *driv= er, if (ret < 0) { virErrorPreserveLast(&orig_err); } else { - qemuMigrationJobSetPhase(driver, vm, - QEMU_MIGRATION_PHASE_PERFORM3_DONE); + qemuMigrationJobSetPhase(vm, QEMU_MIGRATION_PHASE_PERFORM3_DONE); } =20 /* If Perform returns < 0, then we need to cancel the VM @@ -5303,7 +5299,7 @@ qemuMigrationSrcPerformJob(virQEMUDriver *driver, migParams, flags, dname, re= source, &v3proto); } else { - qemuMigrationJobSetPhase(driver, vm, QEMU_MIGRATION_PHASE_PERFORM2= ); + qemuMigrationJobSetPhase(vm, QEMU_MIGRATION_PHASE_PERFORM2); ret =3D qemuMigrationSrcPerformNative(driver, vm, persist_xml, uri= , cookiein, cookieinlen, cookieout, cookieoutlen, flags, resource, NULL, NULL, 0= , NULL, @@ -5339,7 +5335,7 @@ qemuMigrationSrcPerformJob(virQEMUDriver *driver, =20 qemuMigrationSrcRestoreDomainState(driver, vm); =20 - qemuMigrationJobFinish(driver, vm); + qemuMigrationJobFinish(vm); if (!virDomainObjIsActive(vm) && ret =3D=3D 0) { if (flags & VIR_MIGRATE_UNDEFINE_SOURCE) { virDomainDeleteConfig(cfg->configDir, cfg->autostartDir, vm); @@ -5389,7 +5385,7 @@ qemuMigrationSrcPerformPhase(virQEMUDriver *driver, return ret; } =20 - qemuMigrationJobStartPhase(driver, vm, QEMU_MIGRATION_PHASE_PERFORM3); + qemuMigrationJobStartPhase(vm, QEMU_MIGRATION_PHASE_PERFORM3); virCloseCallbacksUnset(driver->closeCallbacks, vm, qemuMigrationSrcCleanup); =20 @@ -5403,7 +5399,7 @@ qemuMigrationSrcPerformPhase(virQEMUDriver *driver, goto endjob; } =20 - qemuMigrationJobSetPhase(driver, vm, QEMU_MIGRATION_PHASE_PERFORM3_DON= E); + qemuMigrationJobSetPhase(vm, QEMU_MIGRATION_PHASE_PERFORM3_DONE); =20 if (virCloseCallbacksSet(driver->closeCallbacks, vm, conn, qemuMigrationSrcCleanup) < 0) @@ -5413,7 +5409,7 @@ qemuMigrationSrcPerformPhase(virQEMUDriver *driver, if (ret < 0) { qemuMigrationParamsReset(driver, vm, QEMU_ASYNC_JOB_MIGRATION_OUT, jobPriv->migParams, priv->job.apiFlags); - qemuMigrationJobFinish(driver, vm); + qemuMigrationJobFinish(vm); } else { qemuMigrationJobContinue(vm); } @@ -5648,7 +5644,7 @@ qemuMigrationDstFinish(virQEMUDriver *driver, =20 ignore_value(virTimeMillisNow(&timeReceived)); =20 - qemuMigrationJobStartPhase(driver, vm, + qemuMigrationJobStartPhase(vm, v3proto ? QEMU_MIGRATION_PHASE_FINISH3 : QEMU_MIGRATION_PHASE_FINISH2); =20 @@ -5878,7 +5874,7 @@ qemuMigrationDstFinish(virQEMUDriver *driver, qemuMigrationParamsReset(driver, vm, QEMU_ASYNC_JOB_MIGRATION_IN, jobPriv->migParams, priv->job.apiFlags); =20 - qemuMigrationJobFinish(driver, vm); + qemuMigrationJobFinish(vm); if (!virDomainObjIsActive(vm)) qemuDomainRemoveInactiveJob(driver, vm); =20 @@ -5940,7 +5936,7 @@ qemuMigrationSrcToFile(virQEMUDriver *driver, virDoma= inObj *vm, qemuMonitorSetMigrationSpeed(priv->mon, QEMU_DOMAIN_MIG_BANDWIDTH_MAX); priv->migMaxBandwidth =3D QEMU_DOMAIN_MIG_BANDWIDTH_MAX; - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); } } =20 @@ -5977,11 +5973,11 @@ qemuMigrationSrcToFile(virQEMUDriver *driver, virDo= mainObj *vm, if (virSetCloseExec(pipeFD[1]) < 0) { virReportSystemError(errno, "%s", _("Unable to set cloexec flag")); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); goto cleanup; } if (virCommandRunAsync(compressor, NULL) < 0) { - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); goto cleanup; } rc =3D qemuMonitorMigrateToFd(priv->mon, @@ -5991,7 +5987,7 @@ qemuMigrationSrcToFile(virQEMUDriver *driver, virDoma= inObj *vm, VIR_CLOSE(pipeFD[1]) < 0) VIR_WARN("failed to close intermediate pipe"); } - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) goto cleanup; =20 @@ -6004,7 +6000,7 @@ qemuMigrationSrcToFile(virQEMUDriver *driver, virDoma= inObj *vm, if (virDomainObjIsActive(vm) && qemuDomainObjEnterMonitorAsync(driver, vm, asyncJob) =3D= =3D 0) { qemuMonitorMigrateCancel(priv->mon); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); } } goto cleanup; @@ -6031,7 +6027,7 @@ qemuMigrationSrcToFile(virQEMUDriver *driver, virDoma= inObj *vm, } else { if (qemuDomainObjEnterMonitorAsync(driver, vm, asyncJob) =3D= =3D 0) { qemuMonitorSetMigrationSpeed(priv->mon, saveMigBandwidth); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); } } priv->migMaxBandwidth =3D saveMigBandwidth; @@ -6063,7 +6059,7 @@ qemuMigrationSrcCancel(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); ignore_value(qemuMonitorMigrateCancel(priv->mon)); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 for (i =3D 0; i < vm->def->ndisks; i++) { virDomainDiskDef *disk =3D vm->def->disks[i]; @@ -6125,8 +6121,7 @@ qemuMigrationJobStart(virQEMUDriver *driver, } =20 static void -qemuMigrationJobSetPhase(virQEMUDriver *driver, - virDomainObj *vm, +qemuMigrationJobSetPhase(virDomainObj *vm, qemuMigrationJobPhase phase) { qemuDomainObjPrivate *priv =3D vm->privateData; @@ -6138,15 +6133,14 @@ qemuMigrationJobSetPhase(virQEMUDriver *driver, return; } =20 - qemuDomainObjSetJobPhase(driver, vm, phase); + qemuDomainObjSetJobPhase(vm, phase); } =20 static void -qemuMigrationJobStartPhase(virQEMUDriver *driver, - virDomainObj *vm, +qemuMigrationJobStartPhase(virDomainObj *vm, qemuMigrationJobPhase phase) { - qemuMigrationJobSetPhase(driver, vm, phase); + qemuMigrationJobSetPhase(vm, phase); } =20 static void @@ -6176,9 +6170,9 @@ qemuMigrationJobIsActive(virDomainObj *vm, } =20 static void -qemuMigrationJobFinish(virQEMUDriver *driver, virDomainObj *vm) +qemuMigrationJobFinish(virDomainObj *vm) { - qemuDomainObjEndAsyncJob(driver, vm); + qemuDomainObjEndAsyncJob(vm); } =20 =20 @@ -6263,7 +6257,7 @@ qemuMigrationSrcFetchMirrorStats(virQEMUDriver *drive= r, =20 blockinfo =3D qemuMonitorGetAllBlockJobInfo(priv->mon, false); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (!blockinfo) return -1; =20 diff --git a/src/qemu/qemu_migration_cookie.c b/src/qemu/qemu_migration_coo= kie.c index 079f31eb90..52ed2c518b 100644 --- a/src/qemu/qemu_migration_cookie.c +++ b/src/qemu/qemu_migration_cookie.c @@ -504,7 +504,7 @@ qemuMigrationCookieAddNBD(qemuMigrationCookie *mig, rc =3D qemuMonitorBlockStatsUpdateCapacityBlockdev(priv->mon, stat= s); else rc =3D qemuMonitorBlockStatsUpdateCapacity(priv->mon, stats); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) return -1; =20 diff --git a/src/qemu/qemu_migration_params.c b/src/qemu/qemu_migration_par= ams.c index e30fd04ada..39f84983bc 100644 --- a/src/qemu/qemu_migration_params.c +++ b/src/qemu/qemu_migration_params.c @@ -889,7 +889,7 @@ qemuMigrationParamsApply(virQEMUDriver *driver, ret =3D 0; =20 cleanup: - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (xbzrleCacheSize_old) migParams->params[xbzrle].set =3D true; @@ -1098,7 +1098,7 @@ qemuMigrationParamsFetch(virQEMUDriver *driver, =20 rc =3D qemuMonitorGetMigrationParams(priv->mon, &jsonParams); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) return -1; =20 @@ -1399,7 +1399,7 @@ qemuMigrationCapsCheck(virQEMUDriver *driver, =20 rc =3D qemuMonitorGetMigrationCapabilities(priv->mon, &caps); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) return -1; =20 @@ -1432,7 +1432,7 @@ qemuMigrationCapsCheck(virQEMUDriver *driver, =20 rc =3D qemuMonitorSetMigrationCapabilities(priv->mon, &json); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rc < 0) { virResetLastError(); diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c index b19a6218d0..7931eeb23b 100644 --- a/src/qemu/qemu_process.c +++ b/src/qemu/qemu_process.c @@ -474,7 +474,7 @@ qemuProcessFakeReboot(void *opaque) qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorSystemReset(priv->mon); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rc < 0) goto endjob; @@ -495,7 +495,7 @@ qemuProcessFakeReboot(void *opaque) ret =3D 0; =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: priv->pausedShutdown =3D false; @@ -1906,7 +1906,7 @@ qemuProcessInitMonitor(virQEMUDriver *driver, =20 ret =3D qemuMonitorSetCapabilities(QEMU_DOMAIN_PRIVATE(vm)->mon); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 return ret; } @@ -2200,7 +2200,7 @@ qemuRefreshVirtioChannelState(virQEMUDriver *driver, return -1; =20 rc =3D qemuMonitorGetChardevInfo(priv->mon, &info); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rc < 0) return -1; @@ -2250,7 +2250,7 @@ qemuRefreshPRManagerState(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorGetPRManagerInfo(priv->mon, &info); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rc < 0) return -1; @@ -2277,7 +2277,7 @@ qemuRefreshRTC(virQEMUDriver *driver, qemuDomainObjEnterMonitor(driver, vm); now =3D time(NULL); rv =3D qemuMonitorGetRTCTime(priv->mon, &thenbits); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rv < 0) return; @@ -2316,7 +2316,7 @@ qemuProcessRefreshBalloonState(virQEMUDriver *driver, return -1; =20 rc =3D qemuMonitorGetBalloonInfo(qemuDomainGetMonitor(vm), &balloon); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) return -1; =20 @@ -2367,7 +2367,7 @@ qemuProcessWaitForMonitor(virQEMUDriver *driver, goto cleanup; ret =3D qemuMonitorGetChardevInfo(priv->mon, &info); VIR_DEBUG("qemuMonitorGetChardevInfo returned %i", ret); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (ret =3D=3D 0) { if ((ret =3D qemuProcessFindCharDevicePTYsMonitor(vm, info)) < 0) @@ -2407,7 +2407,7 @@ qemuProcessDetectIOThreadPIDs(virQEMUDriver *driver, if (qemuDomainObjEnterMonitorAsync(driver, vm, asyncJob) < 0) goto cleanup; ret =3D qemuMonitorGetIOThreads(priv->mon, &iothreads, &niothreads); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (ret < 0) goto cleanup; =20 @@ -2582,7 +2582,7 @@ qemuProcessSetLinkStates(virQEMUDriver *driver, ret =3D 0; =20 cleanup: - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); return ret; } =20 @@ -3106,7 +3106,7 @@ qemuProcessUpdateVideoRamSize(virQEMUDriver *driver, =20 } =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 cfg =3D virQEMUDriverGetConfig(driver); ret =3D virDomainObjSave(vm, driver->xmlopt, cfg->stateDir); @@ -3114,7 +3114,7 @@ qemuProcessUpdateVideoRamSize(virQEMUDriver *driver, return ret; =20 error: - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); return -1; } =20 @@ -3237,7 +3237,7 @@ qemuProcessStartCPUs(virQEMUDriver *driver, virDomain= Obj *vm, goto release; =20 ret =3D qemuMonitorStartCPUs(priv->mon); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (ret < 0) goto release; @@ -3274,7 +3274,7 @@ int qemuProcessStopCPUs(virQEMUDriver *driver, goto cleanup; =20 ret =3D qemuMonitorStopCPUs(priv->mon); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (ret < 0) goto cleanup; @@ -3385,7 +3385,7 @@ qemuProcessUpdateState(virQEMUDriver *driver, virDoma= inObj *vm) =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorGetStatus(priv->mon, &running, &reason); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (ret < 0) return -1; @@ -3621,7 +3621,7 @@ qemuProcessRecoverJob(virQEMUDriver *driver, case QEMU_ASYNC_JOB_SNAPSHOT: qemuDomainObjEnterMonitor(driver, vm); ignore_value(qemuMonitorMigrateCancel(priv->mon)); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); /* resume the domain but only if it was paused as a result of * running a migration-to-file operation. Although we are * recovering an async job, this function is run at startup @@ -4282,7 +4282,7 @@ qemuProcessFetchGuestCPU(virQEMUDriver *driver, rc =3D qemuMonitorGetGuestCPUx86(priv->mon, cpuQOMPath, &dataEnabl= ed, &dataDisabled); } =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rc =3D=3D -1) return -1; @@ -4391,7 +4391,7 @@ qemuProcessFetchCPUDefinitions(virQEMUDriver *driver, =20 rc =3D virQEMUCapsFetchCPUModels(priv->mon, vm->def->os.arch, &models); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) return -1; =20 @@ -4634,10 +4634,9 @@ qemuProcessBeginJob(virQEMUDriver *driver, =20 =20 void -qemuProcessEndJob(virQEMUDriver *driver, - virDomainObj *vm) +qemuProcessEndJob(virDomainObj *vm) { - qemuDomainObjEndAsyncJob(driver, vm); + qemuDomainObjEndAsyncJob(vm); } =20 =20 @@ -5105,7 +5104,7 @@ qemuProcessSetupBalloon(virQEMUDriver *driver, ret =3D 0; =20 cleanup: - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); return ret; } =20 @@ -6000,7 +5999,7 @@ qemuProcessSetupHotpluggableVcpus(virQEMUDriver *driv= er, =20 rc =3D qemuMonitorAddDeviceProps(qemuDomainGetMonitor(vm), &vcpupr= ops); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (rc < 0) goto cleanup; @@ -7162,7 +7161,7 @@ qemuProcessSetupDiskThrottlingBlockdev(virQEMUDriver = *driver, ret =3D 0; =20 cleanup: - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); return ret; } =20 @@ -7282,7 +7281,7 @@ qemuProcessSetupDisksTransientHotplug(virDomainObj *v= m, =20 rc =3D qemuMonitorSystemReset(priv->mon); =20 - qemuDomainObjExitMonitor(priv->driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) return -1; } @@ -7334,7 +7333,7 @@ qemuProcessSetupLifecycleActions(virDomainObj *vm, QEMU_MONITOR_ACTION_WATCHDOG_KEEP, QEMU_MONITOR_ACTION_PANIC_KEEP); =20 - qemuDomainObjExitMonitor(priv->driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) return -1; =20 @@ -8370,7 +8369,7 @@ void qemuProcessStop(virQEMUDriver *driver, =20 endjob: if (asyncJob !=3D QEMU_ASYNC_JOB_NONE) - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 cleanup: virErrorRestore(&orig_err); @@ -8395,7 +8394,7 @@ qemuProcessAutoDestroy(virDomainObj *dom, if (priv->job.asyncJob) { VIR_DEBUG("vm=3D%s has long-term job active, cancelling", dom->def->name); - qemuDomainObjDiscardAsyncJob(driver, dom); + qemuDomainObjDiscardAsyncJob(dom); } =20 VIR_DEBUG("Killing domain"); @@ -8413,7 +8412,7 @@ qemuProcessAutoDestroy(virDomainObj *dom, =20 qemuDomainRemoveInactive(driver, dom); =20 - qemuDomainObjEndJob(driver, dom); + qemuDomainObjEndJob(dom); =20 virObjectEventStateQueue(driver->domainEventState, event); } @@ -8457,7 +8456,7 @@ qemuProcessRefreshDisks(virQEMUDriver *driver, =20 if (qemuDomainObjEnterMonitorAsync(driver, vm, asyncJob) =3D=3D 0) { table =3D qemuMonitorGetBlockInfo(priv->mon); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); } =20 if (!table) @@ -8524,7 +8523,7 @@ qemuProcessRefreshCPUMigratability(virQEMUDriver *dri= ver, =20 rc =3D qemuMonitorGetCPUMigratable(priv->mon, cpuQOMPath, &migratable); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) return -1; =20 @@ -8681,7 +8680,7 @@ qemuProcessRefreshLegacyBlockjobs(virQEMUDriver *driv= er, =20 qemuDomainObjEnterMonitor(driver, vm); blockJobs =3D qemuMonitorGetAllBlockJobInfo(qemuDomainGetMonitor(vm), = true); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); =20 if (!blockJobs) return -1; @@ -8998,7 +8997,7 @@ qemuProcessReconnect(void *opaque) if (jobStarted) { if (!virDomainObjIsActive(obj)) qemuDomainRemoveInactive(driver, obj); - qemuDomainObjEndJob(driver, obj); + qemuDomainObjEndJob(obj); } else { if (!virDomainObjIsActive(obj)) qemuDomainRemoveInactiveJob(driver, obj); diff --git a/src/qemu/qemu_process.h b/src/qemu/qemu_process.h index 289cd74eb7..7e6f9f20e5 100644 --- a/src/qemu/qemu_process.h +++ b/src/qemu/qemu_process.h @@ -69,8 +69,7 @@ int qemuProcessBeginJob(virQEMUDriver *driver, virDomainObj *vm, virDomainJobOperation operation, unsigned long apiFlags); -void qemuProcessEndJob(virQEMUDriver *driver, - virDomainObj *vm); +void qemuProcessEndJob(virDomainObj *vm); =20 typedef enum { VIR_QEMU_PROCESS_START_COLD =3D 1 << 0, diff --git a/src/qemu/qemu_snapshot.c b/src/qemu/qemu_snapshot.c index a7901779fc..6eebe69919 100644 --- a/src/qemu/qemu_snapshot.c +++ b/src/qemu/qemu_snapshot.c @@ -322,7 +322,7 @@ qemuSnapshotCreateActiveInternal(virQEMUDriver *driver, } =20 ret =3D qemuMonitorCreateSnapshot(priv->mon, snap->def->name); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (ret < 0) goto cleanup; =20 @@ -875,7 +875,7 @@ qemuSnapshotDiskCleanup(qemuSnapshotDiskData *data, =20 qemuBlockStorageSourceAttachRollback(qemuDomainGetMoni= tor(vm), data[i].crdata->s= rcdata[0]); - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); } } =20 @@ -1024,7 +1024,7 @@ qemuSnapshotDiskPrepareOneBlockdev(virQEMUDriver *dri= ver, rc =3D qemuBlockStorageSourceAttachApply(qemuDomainGetMonitor(vm), dd->crdata->srcdata[0]); =20 - qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(vm); if (rc < 0) return -1; } else { @@ -1281,7 +1281,7 @@ qemuSnapshotDiskCreate(qemuSnapshotDiskContext *snapc= txt) =20 rc =3D qemuMonitorTransaction(priv->mon, &snapctxt->actions); =20 - qemuDomainObjExitMonitor(driver, snapctxt->vm); + qemuDomainObjExitMonitor(snapctxt->vm); =20 for (i =3D 0; i < snapctxt->ndd; i++) { qemuSnapshotDiskData *dd =3D snapctxt->dd + i; @@ -1887,7 +1887,7 @@ qemuSnapshotCreateXML(virDomainPtr domain, snapshot =3D qemuSnapshotCreate(vm, domain, def, driver, cfg, flag= s); } =20 - qemuDomainObjEndAsyncJob(driver, vm); + qemuDomainObjEndAsyncJob(vm); =20 return snapshot; } @@ -2322,7 +2322,7 @@ qemuSnapshotRevert(virDomainObj *vm, } =20 endjob: - qemuProcessEndJob(driver, vm); + qemuProcessEndJob(vm); =20 return ret; } @@ -2454,7 +2454,7 @@ qemuSnapshotDelete(virDomainObj *vm, } =20 endjob: - qemuDomainObjEndJob(driver, vm); + qemuDomainObjEndJob(vm); =20 return ret; } --=20 2.35.1