main-loop.c: Handle SIGINT, SIGHUP and SIGTERM synchronously
authorPeter Maydell <peter.maydell@linaro.org>
Thu, 25 Sep 2014 15:51:09 +0000 (16:51 +0100)
committerPeter Maydell <peter.maydell@linaro.org>
Fri, 26 Sep 2014 10:47:30 +0000 (11:47 +0100)
commit15124e142034d21341ec9f1a304a1dc5a6c25681
tree82bda052bc66201b095fbb4d6764067a2b964494
parent6a0fcbdf2de908e6ef910aaee778abe987e49d52
main-loop.c: Handle SIGINT, SIGHUP and SIGTERM synchronously

Add the termination signals SIGINT, SIGHUP and SIGTERM to the
list of signals which we handle synchronously via a signalfd.
This avoids a race condition where if we took the SIGTERM
in the middle of qemu_shutdown_requested:
    int r = shutdown_requested;
[SIGTERM here...]
    shutdown_requested = 0;

then the setting of the shutdown_requested flag by
termsig_handler() would be lost and QEMU would fail to
shut down. This was causing 'make check' to hang occasionally.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Tested-by: Alex Bennée <alex.bennee@linaro.org>
Message-id: 1411660269-11081-1-git-send-email-peter.maydell@linaro.org
Cc: qemu-stable@nongnu.org
main-loop.c