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 d052493..1640c5d 100644 (file)
--- 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 69959fd..8e06bfa 100644 (file)
--- 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 41d2393..a3860ab 100644 (file)
--- 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: