From: Vladislav Grishenko <themiron@yandex-team.ru>
Date: Sun, 20 Mar 2022 21:21:42 +0000 (+0500)
Subject: vlib: leave SIGCONT signal with its default handler
X-Git-Tag: v22.10-rc0~231
X-Git-Url: https://gerrit.fd.io/r/gitweb?a=commitdiff_plain;h=8486283cd5efb6b3c78428658eed685c5e8469d7;p=vpp.git

vlib: leave SIGCONT signal with its default handler

Systemd always sends SIGCONT after KillSignal, to ensure that even suspended
tasks can be terminated cleanly. However, the default action of SIGCONT
handler in VPP such as unix_signal_handler() is process termination with
coredump, what is not really desired. So, leave it alike SIGSTOP with its
default handler.

Type: feature
Signed-off-by: Vladislav Grishenko <themiron@yandex-team.ru>
Change-Id: I54c06d21669ec3c709322d746db9e28448c31bb8
---

diff --git a/src/vat/main.c b/src/vat/main.c
index d052493495d..1640c5ded6a 100644
--- a/src/vat/main.c
+++ b/src/vat/main.c
@@ -267,6 +267,7 @@ setup_signal_handlers (void)
 	  /* these signals take the default action */
 	case SIGABRT:
 	case SIGKILL:
+	case SIGCONT:
 	case SIGSTOP:
 	case SIGUSR1:
 	case SIGUSR2:
diff --git a/src/vlib/unix/main.c b/src/vlib/unix/main.c
index 69959fd9137..8e06bfa334b 100644
--- a/src/vlib/unix/main.c
+++ b/src/vlib/unix/main.c
@@ -210,6 +210,7 @@ setup_signal_handlers (unix_main_t * um)
 	{
 	  /* these signals take the default action */
 	case SIGKILL:
+	case SIGCONT:
 	case SIGSTOP:
 	case SIGUSR1:
 	case SIGUSR2:
diff --git a/src/vpp/api/vpp_get_metrics.c b/src/vpp/api/vpp_get_metrics.c
index 41d2393cc96..a3860ab56b0 100644
--- a/src/vpp/api/vpp_get_metrics.c
+++ b/src/vpp/api/vpp_get_metrics.c
@@ -77,6 +77,7 @@ setup_signal_handlers (void)
 	  /* these signals take the default action */
 	case SIGABRT:
 	case SIGKILL:
+	case SIGCONT:
 	case SIGSTOP:
 	case SIGUSR1:
 	case SIGUSR2: