]> xenbits.xensource.com Git - libvirt.git/commitdiff
qemu: Relax -no-shutdown check to [0.14.0, 0.15.50)
authorJiri Denemark <jdenemar@redhat.com>
Tue, 27 Sep 2011 12:46:35 +0000 (14:46 +0200)
committerJiri Denemark <jdenemar@redhat.com>
Tue, 27 Sep 2011 13:46:23 +0000 (15:46 +0200)
SIGTERM handling for -no-shutdown is already fixed in qemu git and
libvirt can safely use it. The downside is that 0.15.50 version of qemu
can be any qemu compiled from git, even that without the fix for
SIGTERM. However, I think this patch is worth it since excluding 0.15.50
from the check makes testing current qemu with libvirt much easier and
someone running qemu from git should be able to rebuild fixed qemu from
git if they hit the problem with a hang on shutdown.

src/qemu/qemu_capabilities.c

index 8e20e3f32c5a8b4e28e6879a3f1c8b3f8da46332..4325f7710103398a984aeef1551bc3859f1ee3e1 100644 (file)
@@ -1017,9 +1017,9 @@ qemuCapsComputeCmdFlags(const char *help,
 
     /* Do not use -no-shutdown if qemu doesn't support it or SIGTERM handling
      * is most likely buggy when used with -no-shutdown (which applies for qemu
-     * 0.14.* and 0.15.*)
+     * 0.14.* and <0.15.50)
      */
-    if (strstr(help, "-no-shutdown") && (version < 14000 || version > 15999))
+    if (strstr(help, "-no-shutdown") && (version < 14000 || version >= 15050))
         qemuCapsSet(flags, QEMU_CAPS_NO_SHUTDOWN);
 
     /*