Mailing List Archive

commit 5d09f8cbdc22aca97f0ea5de0067ebd2820953a9
Author: Leigh Brown <leigh@solinno.co.uk>
AuthorDate: Fri Mar 29 11:10:52 2024 +0000
Commit: Andrew Cooper <andrew.cooper3@citrix.com>
CommitDate: Thu Apr 11 14:16:53 2024 +0100

tools/misc: rework xenwatchdogd signal handling

Rework xenwatchdogd signal handling to do the minimum in the signal
handler. This is a very minor enhancement.

Signed-off-by: Leigh Brown <leigh@solinno.co.uk>
Reviewed-by: Anthony PERARD <anthony.perard@citrix.com>
---
tools/misc/xenwatchdogd.c | 20 ++++++++++++--------
1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/tools/misc/xenwatchdogd.c b/tools/misc/xenwatchdogd.c
index 2f7c822d61..ecc3f01a0e 100644
--- a/tools/misc/xenwatchdogd.c
+++ b/tools/misc/xenwatchdogd.c
@@ -9,9 +9,11 @@
#include <unistd.h>
#include <signal.h>
#include <stdio.h>
+#include <stdbool.h>

xc_interface *h;
-int id = 0;
+static volatile bool safeexit = false;
+static volatile bool done = false;

void daemonize(void)
{
@@ -38,20 +40,18 @@ void daemonize(void)

void catch_exit(int sig)
{
- if (id)
- xc_watchdog(h, id, 300);
- exit(EXIT_SUCCESS);
+ done = true;
}

void catch_usr1(int sig)
{
- if (id)
- xc_watchdog(h, id, 0);
- exit(EXIT_SUCCESS);
+ safeexit = true;
+ done = true;
}

int main(int argc, char **argv)
{
+ int id;
int t, s;
int ret;

@@ -90,10 +90,14 @@ int main(int argc, char **argv)
if (id <= 0)
err(EXIT_FAILURE, "xc_watchdog setup");

- for (;;) {
+ while (!done) {
sleep(s);
ret = xc_watchdog(h, id, t);
if (ret != 0)
err(EXIT_FAILURE, "xc_watchdog");
}
+
+ // Zero seconds timeout will disarm the watchdog timer
+ xc_watchdog(h, id, safeexit ? 0 : 300);
+ return 0;
}
--
generated by git-patchbot for /home/xen/git/xen.git#staging