]> git.ipfire.org Git - thirdparty/libvirt.git/commitdiff
qemu: capabilities: Remove unused 'virQEMUCapsFilterByMachineType'
authorPeter Krempa <pkrempa@redhat.com>
Mon, 28 Aug 2023 12:45:53 +0000 (14:45 +0200)
committerPeter Krempa <pkrempa@redhat.com>
Mon, 4 Sep 2023 08:31:52 +0000 (10:31 +0200)
The filtering of qemu capabilities by machine type doesn't seem to be
ever used, remove it and adjust callers.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
src/qemu/qemu_capabilities.c
src/qemu/qemu_capabilities.h
src/qemu/qemu_domain.c
src/qemu/qemu_process.c

index fd921304663a76b8ada9a0786c76759d42b70d75..ea4e24834919daa8d8044f09f0f44a4d7066c286 100644 (file)
@@ -5820,42 +5820,6 @@ virQEMUCapsLoadFile(const char *filename,
 }
 
 
-struct virQEMUCapsMachineTypeFilter {
-    const char *machineType;
-    virQEMUCapsFlags *flags;
-    size_t nflags;
-};
-
-static const struct virQEMUCapsMachineTypeFilter virQEMUCapsMachineFilter[] = {
-    /* { "blah", virQEMUCapsMachineBLAHFilter,
-         G_N_ELEMENTS(virQEMUCapsMachineBLAHFilter) }, */
-    { "", NULL, 0 },
-};
-
-
-void
-virQEMUCapsFilterByMachineType(virQEMUCaps *qemuCaps,
-                               virDomainVirtType virtType G_GNUC_UNUSED,
-                               const char *machineType)
-{
-    size_t i;
-
-    if (!machineType)
-        return;
-
-    for (i = 0; i < G_N_ELEMENTS(virQEMUCapsMachineFilter); i++) {
-        const struct virQEMUCapsMachineTypeFilter *filter = &virQEMUCapsMachineFilter[i];
-        size_t j;
-
-        if (STRNEQ(filter->machineType, machineType))
-            continue;
-
-        for (j = 0; j < filter->nflags; j++)
-            virQEMUCapsClear(qemuCaps, filter->flags[j]);
-    }
-}
-
-
 virFileCacheHandlers qemuCapsCacheHandlers = {
     .isValid = virQEMUCapsIsValid,
     .newData = virQEMUCapsNewData,
@@ -5925,9 +5889,7 @@ virQEMUCapsCacheLookup(virFileCache *cache,
 
 virQEMUCaps *
 virQEMUCapsCacheLookupCopy(virFileCache *cache,
-                           virDomainVirtType virtType,
-                           const char *binary,
-                           const char *machineType)
+                           const char *binary)
 {
     virQEMUCaps *qemuCaps = virQEMUCapsCacheLookup(cache, binary);
     virQEMUCaps *ret;
@@ -5938,7 +5900,6 @@ virQEMUCapsCacheLookupCopy(virFileCache *cache,
     ret = virQEMUCapsNewCopy(qemuCaps);
     virObjectUnref(qemuCaps);
 
-    virQEMUCapsFilterByMachineType(ret, virtType, machineType);
     return ret;
 }
 
index d68a819442c1206acf7712b50315f49b88cc683c..ba289ba6b983e15daef088137ac7723d2d983873 100644 (file)
@@ -783,9 +783,6 @@ const char *virQEMUCapsGetMachineDefaultRAMid(virQEMUCaps *qemuCaps,
                                               virDomainVirtType virtType,
                                               const char *name);
 
-void virQEMUCapsFilterByMachineType(virQEMUCaps *qemuCaps,
-                                    virDomainVirtType virtType,
-                                    const char *machineType);
 char * virQEMUCapsGetDefaultEmulator(virArch hostarch,
                                      virArch guestarch);
 
@@ -796,9 +793,7 @@ virFileCache *virQEMUCapsCacheNew(const char *libDir,
 virQEMUCaps *virQEMUCapsCacheLookup(virFileCache *cache,
                                       const char *binary);
 virQEMUCaps *virQEMUCapsCacheLookupCopy(virFileCache *cache,
-                                          virDomainVirtType virtType,
-                                          const char *binary,
-                                          const char *machineType);
+                                          const char *binary);
 virQEMUCaps *virQEMUCapsCacheLookupDefault(virFileCache *cache,
                                              const char *binary,
                                              const char *archStr,
index bf0b349406b1344a7d0ae47d41b8849e66b2e727..9781df1355409eca19e01b76ae6f9c0d85f8d99b 100644 (file)
@@ -6612,9 +6612,7 @@ qemuDomainDefFormatBufInternal(virQEMUDriver *driver,
             qCaps = virObjectRef(qemuCaps);
         } else {
             if (!(qCaps = virQEMUCapsCacheLookupCopy(driver->qemuCapsCache,
-                                                     def->virtType,
-                                                     def->emulator,
-                                                     def->os.machine)))
+                                                     def->emulator)))
                 return -1;
         }
 
index 4a7308034d944291cf532963821ff71505125c7b..7a1cdb030285a8f4f029d370c8ac883973870f98 100644 (file)
@@ -5652,9 +5652,7 @@ qemuProcessPrepareQEMUCaps(virDomainObj *vm,
 
     virObjectUnref(priv->qemuCaps);
     if (!(priv->qemuCaps = virQEMUCapsCacheLookupCopy(qemuCapsCache,
-                                                      vm->def->virtType,
-                                                      vm->def->emulator,
-                                                      vm->def->os.machine)))
+                                                      vm->def->emulator)))
         return -1;
 
     /* Update qemu capabilities according to lists passed in via namespace */