]> xenbits.xensource.com Git - libvirt.git/commitdiff
qemu & hypervisor: move qemuDomainObjResetAgentJob() into hypervisor
authorKristina Hanicova <khanicov@redhat.com>
Wed, 3 Aug 2022 12:43:12 +0000 (14:43 +0200)
committerMichal Privoznik <mprivozn@redhat.com>
Mon, 8 Aug 2022 10:19:24 +0000 (12:19 +0200)
This patch moves qemuDomainObjResetAgentJob() as
virDomainObjResetAgentJob() into hypervisor.

Signed-off-by: Kristina Hanicova <khanicov@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
src/hypervisor/domain_job.c
src/hypervisor/domain_job.h
src/libvirt_private.syms
src/qemu/qemu_domainjob.c

index 5b9762428737dc708612984d7ca6772090736d13..9fbea500c46be44f3eec8e18b1e5c243d19c5e59 100644 (file)
@@ -146,3 +146,12 @@ virDomainObjResetJob(virDomainJobObj *job)
     g_clear_pointer(&job->ownerAPI, g_free);
     job->started = 0;
 }
+
+void
+virDomainObjResetAgentJob(virDomainJobObj *job)
+{
+    job->agentActive = VIR_AGENT_JOB_NONE;
+    job->agentOwner = 0;
+    g_clear_pointer(&job->agentOwnerAPI, g_free);
+    job->agentStarted = 0;
+}
index 4c32296117db12cc7b41abec5497d01200632920..7a9197cbdef44c0f9417c6bc11620129b41034de 100644 (file)
@@ -210,3 +210,5 @@ int virDomainObjInitJob(virDomainJobObj *job,
                         virDomainObjPrivateJobCallbacks *cb);
 
 void virDomainObjResetJob(virDomainJobObj *job);
+
+void virDomainObjResetAgentJob(virDomainJobObj *job);
index 08dbe730da4467d6647bd5d0565bea0f80dc4c6a..a522fb3b1a1bcb0529c70d9c5c46844cde0b32e8 100644 (file)
@@ -1595,6 +1595,7 @@ virDomainJobStatusToType;
 virDomainJobTypeFromString;
 virDomainJobTypeToString;
 virDomainObjInitJob;
+virDomainObjResetAgentJob;
 virDomainObjResetJob;
 
 
index a45103256a39f019d71f4f2298f96dc7f0dd2f4c..0f3b38108c9621afc962cbe63ac6ba05eefb773e 100644 (file)
@@ -150,16 +150,6 @@ qemuDomainEventEmitJobCompleted(virQEMUDriver *driver,
 }
 
 
-static void
-qemuDomainObjResetAgentJob(virDomainJobObj *job)
-{
-    job->agentActive = VIR_AGENT_JOB_NONE;
-    job->agentOwner = 0;
-    g_clear_pointer(&job->agentOwnerAPI, g_free);
-    job->agentStarted = 0;
-}
-
-
 static void
 qemuDomainObjResetAsyncJob(virDomainJobObj *job)
 {
@@ -936,7 +926,7 @@ qemuDomainObjBeginJobInternal(virQEMUDriver *driver,
     }
 
     if (agentJob) {
-        qemuDomainObjResetAgentJob(&priv->job);
+        virDomainObjResetAgentJob(&priv->job);
 
         VIR_DEBUG("Started agent job: %s (vm=%p name=%s job=%s async=%s)",
                   virDomainAgentJobTypeToString(agentJob),
@@ -1191,7 +1181,7 @@ qemuDomainObjEndAgentJob(virDomainObj *obj)
               virDomainAsyncJobTypeToString(priv->job.asyncJob),
               obj, obj->def->name);
 
-    qemuDomainObjResetAgentJob(&priv->job);
+    virDomainObjResetAgentJob(&priv->job);
     /* We indeed need to wake up ALL threads waiting because
      * grabbing a job requires checking more variables. */
     virCondBroadcast(&priv->job.cond);