Re: [RFC PATCH] 22.214.171.124 user-mode linux: before abort, we make it sure all children quit
From: Jeff Dike
Date: Tue Sep 25 2007 - 13:53:48 EST
On Sat, Sep 22, 2007 at 04:01:24PM +0800, lepton wrote:
> In a stock 126.96.36.199 kernel, poweroff a user mode linux guest
> (188.8.131.52 running in skas0 mode) will halt the host linux. I
> think the reason is the kernel thread abort because of a bug.
> Then the sys_reboot in process of user mode linux guest is
> not trapped by the user mode linux kernel and is executed by host.
> I think it is better to make sure all of our children process
> to quit when user mode linux kernel abort.
Below is what I currently have for this patch. As you sent it in, the
kill(0, SIGTERM) would immediately kill the kernel process along with
everything else, before it can dump core. So, I have the kernel
Then, there are still processes which survive. The one case I think I
understand is that a process is handling an infinite sequence of
SIGSEGVs and never sees the SIGTERM. So, I added a loop which waits
for all of the current child processes and kills each one as it
returns some sort of status.
Work email - jdike at linux dot intel dot com
From: Lepton Wu <ytht.net@xxxxxxxxx>
In a stock 184.108.40.206 kernel, poweroff a user mode linux guest
(220.127.116.11 running in skas0 mode) will halt the host linux. I
think the reason is the kernel thread abort because of a bug.
Then the sys_reboot in process of user mode linux guest is
not trapped by the user mode linux kernel and is executed by host.
I think it is better to make sure all of our children process
to quit when user mode linux kernel abort.
[ jdike - the kernel process needs to ignore SIGTERM, plus the
waitpid/kill loop is needed to make sure that all of our children
are dead before the kernel exits ]
Signed-off-by: Lepton Wu <ytht.net@xxxxxxxxx>
Signed-off-by: Jeff Dike <jdike@xxxxxxxxxxxxxxx>
arch/um/os-Linux/skas/process.c | 2 +-
arch/um/os-Linux/util.c | 38 ++++++++++++++++++++++++++++++++++++++
2 files changed, 39 insertions(+), 1 deletion(-)
--- linux-2.6.22.orig/arch/um/os-Linux/util.c 2007-09-25 13:33:48.000000000 -0400
+++ linux-2.6.22/arch/um/os-Linux/util.c 2007-09-25 13:45:33.000000000 -0400
@@ -105,6 +105,44 @@ int setjmp_wrapper(void (*proc)(void *,
+ int pid;
+ * We are about to SIGTERM this entire process group to ensure that
+ * nothing is around to run after the kernel exits. The
+ * kernel wants to abort, not die through SIGTERM, so we
+ * ignore it here.
+ signal(SIGTERM, SIG_IGN);
+ kill(0, SIGTERM);
+ * Most of the other processes associated with this UML are
+ * likely sTopped, so give them a SIGCONT so they see the
+ * SIGTERM.
+ kill(0, SIGCONT);
+ * Now having sent signals to everyone but us, make sure they
+ * die by ptrace. Processes can survive what's been done to
+ * them so far - the mechanism I understand is receiving a
+ * SIGSEGV and segfaulting immediately upon return. There is
+ * always a SIGSEGV pending, and (I'm guessing) signals are
+ * processed in numeric order so the SIGTERM (signal 15 vs
+ * SIGSEGV being signal 11) is never handled.
+ * Run a waitpid loop until we get some kind of error.
+ * Hopefully, it's ECHILD, but there's not a lot we can do if
+ * it's something else. Tell os_kill_ptraced_process not to
+ * wait for the child to report its death because there's
+ * nothing reasonable to do if that fails.
+ while ((pid = waitpid(-1, NULL, WNOHANG)) > 0)
+ os_kill_ptraced_process(pid, 0);
--- linux-2.6.22.orig/arch/um/os-Linux/skas/process.c 2007-09-25 13:34:17.000000000 -0400
+++ linux-2.6.22/arch/um/os-Linux/skas/process.c 2007-09-25 13:45:43.000000000 -0400
@@ -177,7 +177,7 @@ static int userspace_tramp(void *stack)
ptrace(PTRACE_TRACEME, 0, 0, 0);
+ signal(SIGTERM, SIG_DFL);
err = set_interval();
panic("userspace_tramp - setting timer failed, errno = %d\n",
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/