]> xenbits.xensource.com Git - libvirt.git/commitdiff
qemu: Remove getCompressionType
authorJohn Ferlan <jferlan@redhat.com>
Tue, 13 Sep 2016 14:31:37 +0000 (10:31 -0400)
committerJohn Ferlan <jferlan@redhat.com>
Mon, 26 Sep 2016 11:44:42 +0000 (07:44 -0400)
There's only one caller now anyway... Besides it's just a shell for
getting the compress type.  Subsequent patches will return the path
to the compression program.

src/qemu/qemu_driver.c

index 3f03576e2ea2a770ca9aa92b7a32441428812d8d..d5acc30d461723927afa41a828d9182a01258fce 100644 (file)
@@ -3310,23 +3310,6 @@ qemuGetCompressionProgram(const char *imageFormat)
 }
 
 
-static virQEMUSaveFormat
-getCompressionType(virQEMUDriverPtr driver)
-{
-    int ret;
-    virQEMUDriverConfigPtr cfg = virQEMUDriverGetConfig(driver);
-
-    /*
-     * We reuse "save" flag for "dump" here. Then, we can support the same
-     * format in "save" and "dump".
-     */
-    ret = qemuGetCompressionProgram(cfg->dumpImageFormat);
-
-    virObjectUnref(cfg);
-    return ret;
-}
-
-
 static int
 qemuDomainSaveFlags(virDomainPtr dom, const char *path, const char *dxml,
                     unsigned int flags)
@@ -3602,7 +3585,12 @@ doCoreDump(virQEMUDriverPtr driver,
     int directFlag = 0;
     unsigned int flags = VIR_FILE_WRAPPER_NON_BLOCKING;
     const char *memory_dump_format = NULL;
-    virQEMUSaveFormat compress = getCompressionType(driver);
+    virQEMUDriverConfigPtr cfg = virQEMUDriverGetConfig(driver);
+    virQEMUSaveFormat compress;
+
+    /* We reuse "save" flag for "dump" here. Then, we can support the same
+     * format in "save" and "dump". */
+    compress = qemuGetCompressionProgram(cfg->dumpImageFormat);
 
     /* Create an empty file with appropriate ownership.  */
     if (dump_flags & VIR_DUMP_BYPASS_CACHE) {
@@ -3673,6 +3661,7 @@ doCoreDump(virQEMUDriverPtr driver,
     if (ret != 0)
         unlink(path);
     virFileWrapperFdFree(wrapperFd);
+    virObjectUnref(cfg);
     return ret;
 }