|
 |
76daa3 |
From f6a5bfe48ac9cddf5baa2de1e7ae9ca119251079 Mon Sep 17 00:00:00 2001
|
|
|
76daa3 |
From: Eric Blake <eblake@redhat.com>
|
|
|
76daa3 |
Date: Thu, 1 Jun 2017 16:58:17 +0200
|
|
|
76daa3 |
Subject: [PATCH 01/17] shutdown: Simplify shutdown_signal
|
|
|
76daa3 |
|
|
|
76daa3 |
RH-Author: Eric Blake <eblake@redhat.com>
|
|
|
76daa3 |
Message-id: <20170601165821.26810-2-eblake@redhat.com>
|
|
|
76daa3 |
Patchwork-id: 75464
|
|
|
76daa3 |
O-Subject: [RHEL-7.4 qemu-kvm-rhev PATCH v2 1/5] shutdown: Simplify shutdown_signal
|
|
|
76daa3 |
Bugzilla: 1418927
|
|
|
76daa3 |
RH-Acked-by: John Snow <jsnow@redhat.com>
|
|
|
76daa3 |
RH-Acked-by: Markus Armbruster <armbru@redhat.com>
|
|
|
76daa3 |
RH-Acked-by: Thomas Huth <thuth@redhat.com>
|
|
|
76daa3 |
|
|
|
76daa3 |
There is no signal 0 (kill(pid, 0) has special semantics to probe whether
|
|
|
76daa3 |
a process is alive), rather than actually sending a signal 0). So we
|
|
|
76daa3 |
can use the simpler 0, instead of -1, for our sentinel of whether a
|
|
|
76daa3 |
shutdown request due to a signal has happened.
|
|
|
76daa3 |
|
|
|
76daa3 |
Suggested-by: Markus Armbruster <armbru@redhat.com>
|
|
|
76daa3 |
Signed-off-by: Eric Blake <eblake@redhat.com>
|
|
|
76daa3 |
Reviewed-by: Markus Armbruster <armbru@redhat.com>
|
|
|
76daa3 |
Reviewed-by: Alistair Francis <alistair.francis@xilinx.com>
|
|
|
76daa3 |
Message-Id: <20170515214114.15442-2-eblake@redhat.com>
|
|
|
76daa3 |
Signed-off-by: Markus Armbruster <armbru@redhat.com>
|
|
|
76daa3 |
(cherry picked from commit 7af88279e4972dd6bf735b620876d54b7a355c4d)
|
|
|
76daa3 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
76daa3 |
---
|
|
|
76daa3 |
vl.c | 6 +++---
|
|
|
76daa3 |
1 file changed, 3 insertions(+), 3 deletions(-)
|
|
|
76daa3 |
|
|
|
76daa3 |
diff --git a/vl.c b/vl.c
|
|
|
76daa3 |
index 183f070..f5dea2c 100644
|
|
|
76daa3 |
--- a/vl.c
|
|
|
76daa3 |
+++ b/vl.c
|
|
|
76daa3 |
@@ -1597,7 +1597,7 @@ void vm_state_notify(int running, RunState state)
|
|
|
76daa3 |
}
|
|
|
76daa3 |
|
|
|
76daa3 |
static int reset_requested;
|
|
|
76daa3 |
-static int shutdown_requested, shutdown_signal = -1;
|
|
|
76daa3 |
+static int shutdown_requested, shutdown_signal;
|
|
|
76daa3 |
static pid_t shutdown_pid;
|
|
|
76daa3 |
static int powerdown_requested;
|
|
|
76daa3 |
static int debug_requested;
|
|
|
76daa3 |
@@ -1628,7 +1628,7 @@ static int qemu_shutdown_requested(void)
|
|
|
76daa3 |
|
|
|
76daa3 |
static void qemu_kill_report(void)
|
|
|
76daa3 |
{
|
|
|
76daa3 |
- if (!qtest_driver() && shutdown_signal != -1) {
|
|
|
76daa3 |
+ if (!qtest_driver() && shutdown_signal) {
|
|
|
76daa3 |
if (shutdown_pid == 0) {
|
|
|
76daa3 |
/* This happens for eg ^C at the terminal, so it's worth
|
|
|
76daa3 |
* avoiding printing an odd message in that case.
|
|
|
76daa3 |
@@ -1642,7 +1642,7 @@ static void qemu_kill_report(void)
|
|
|
76daa3 |
shutdown_cmd ? shutdown_cmd : "<unknown process>");
|
|
|
76daa3 |
g_free(shutdown_cmd);
|
|
|
76daa3 |
}
|
|
|
76daa3 |
- shutdown_signal = -1;
|
|
|
76daa3 |
+ shutdown_signal = 0;
|
|
|
76daa3 |
}
|
|
|
76daa3 |
}
|
|
|
76daa3 |
|
|
|
76daa3 |
--
|
|
|
76daa3 |
1.8.3.1
|
|
|
76daa3 |
|