]> git.ipfire.org Git - thirdparty/libvirt.git/commitdiff
hyperv: remove duplicate function hypervGetMemSDByVSSDInstanceId()
authorMatt Coleman <mcoleman@datto.com>
Thu, 22 Oct 2020 16:38:21 +0000 (12:38 -0400)
committerMichal Privoznik <mprivozn@redhat.com>
Mon, 2 Nov 2020 17:44:21 +0000 (18:44 +0100)
Signed-off-by: Matt Coleman <matt@datto.com>
Reviewed-by: Neal Gompa <ngompa13@gmail.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
src/hyperv/hyperv_driver.c

index 3935320ea9529665666d017099d163cf072072f6..6d5e89953589986cba54586cc53a34767f9cab3c 100644 (file)
@@ -246,31 +246,6 @@ hypervGetProcSDByVSSDInstanceId(hypervPrivate *priv, const char *id,
 }
 
 
-static int
-hypervGetMemSDByVSSDInstanceId(hypervPrivate *priv, const char *id,
-                               Msvm_MemorySettingData **data)
-{
-    g_auto(virBuffer) query = VIR_BUFFER_INITIALIZER;
-    virBufferEscapeSQL(&query,
-                       "ASSOCIATORS OF {Msvm_VirtualSystemSettingData.InstanceID='%s'} "
-                       "WHERE AssocClass = Msvm_VirtualSystemSettingDataComponent "
-                       "ResultClass = Msvm_MemorySettingData",
-                       id);
-
-    if (hypervGetWmiClass(Msvm_MemorySettingData, data) < 0)
-        return -1;
-
-    if (!*data) {
-        virReportError(VIR_ERR_INTERNAL_ERROR,
-                       _("Could not look up memory setting data with virtual system instance ID '%s'"),
-                       id);
-        return -1;
-    }
-
-    return 0;
-}
-
-
 static int
 hypervRequestStateChange(virDomainPtr domain, int state)
 {
@@ -1082,11 +1057,10 @@ hypervDomainGetInfo(virDomainPtr domain, virDomainInfoPtr info)
         goto cleanup;
     }
 
-    if (hypervGetMemSDByVSSDInstanceId(priv,
-                                       virtualSystemSettingData->data.common->InstanceID,
-                                       &memorySettingData) < 0) {
+    if (hypervGetMsvmMemorySettingDataFromVSSD(priv,
+                                               virtualSystemSettingData->data.common->InstanceID,
+                                               &memorySettingData) < 0)
         goto cleanup;
-    }
 
     /* Fill struct */
     info->state = hypervMsvmComputerSystemEnabledStateToDomainState(computerSystem);
@@ -1167,11 +1141,10 @@ hypervDomainGetXMLDesc(virDomainPtr domain, unsigned int flags)
         goto cleanup;
     }
 
-    if (hypervGetMemSDByVSSDInstanceId(priv,
-                                       virtualSystemSettingData->data.common->InstanceID,
-                                       &memorySettingData) < 0) {
+    if (hypervGetMsvmMemorySettingDataFromVSSD(priv,
+                                               virtualSystemSettingData->data.common->InstanceID,
+                                               &memorySettingData) < 0)
         goto cleanup;
-    }
 
     /* Fill struct */
     def->virtType = VIR_DOMAIN_VIRT_HYPERV;