diff --git a/TODO b/TODO index 4ad6076759..d1fbed5c58 100644 --- a/TODO +++ b/TODO @@ -34,8 +34,6 @@ Bugfixes: Features: -* switch-root killing spree: don't waitpid, and think about signalfd() serialization - * flush jobs when switching root * autorestart of journald after switch-root is broken diff --git a/src/core/killall.c b/src/core/killall.c index 5f589eaf47..6fcaf847cf 100644 --- a/src/core/killall.c +++ b/src/core/killall.c @@ -150,7 +150,7 @@ static int killall(int sig) { return n_processes; } -void broadcast_signal(int sig) { +void broadcast_signal(int sig, bool wait) { sigset_t mask, oldmask; int n_processes; @@ -169,7 +169,8 @@ void broadcast_signal(int sig) { if (n_processes <= 0) goto finish; - wait_for_children(n_processes, &mask); + if (wait) + wait_for_children(n_processes, &mask); finish: sigprocmask(SIG_SETMASK, &oldmask, NULL); diff --git a/src/core/killall.h b/src/core/killall.h index 8b5bd52de5..d08ac142f0 100644 --- a/src/core/killall.h +++ b/src/core/killall.h @@ -22,6 +22,6 @@ along with systemd; If not, see . ***/ -void broadcast_signal(int sig); +void broadcast_signal(int sig, bool wait); #endif diff --git a/src/core/main.c b/src/core/main.c index f1f8b21ae0..022d05a31b 100644 --- a/src/core/main.c +++ b/src/core/main.c @@ -1692,9 +1692,11 @@ finish: watchdog_close(true); if (switch_root_dir) { - /* Kill all remaining processes from the initrd */ - broadcast_signal(SIGTERM); - broadcast_signal(SIGKILL); + /* Kill all remaining processes from the + * initrd, but don't wait for them, so that we + * can handle the SIGCHLD for them after + * deserializing. */ + broadcast_signal(SIGTERM, false); /* And switch root */ r = switch_root(switch_root_dir); diff --git a/src/core/shutdown.c b/src/core/shutdown.c index 78dccfb724..105a604542 100644 --- a/src/core/shutdown.c +++ b/src/core/shutdown.c @@ -177,10 +177,10 @@ int main(int argc, char *argv[]) { mlockall(MCL_CURRENT|MCL_FUTURE); log_info("Sending SIGTERM to remaining processes..."); - broadcast_signal(SIGTERM); + broadcast_signal(SIGTERM, true); log_info("Sending SIGKILL to remaining processes..."); - broadcast_signal(SIGKILL); + broadcast_signal(SIGKILL, true); if (in_container) { need_swapoff = false;