From: Oleg Nesterov Date: Wed, 30 Apr 2008 07:52:58 +0000 (-0700) Subject: signals: cleanup the usage of print_fatal_signal() X-Git-Tag: v2.6.26-rc1~282 X-Git-Url: https://openfabrics.org/gitweb/?a=commitdiff_plain;h=2dce81bff28dceb2153c901883a56f278d91db65;p=~shefty%2Frdma-dev.git signals: cleanup the usage of print_fatal_signal() Move the callsite of print_fatal_signal() down, under "if (sig_kernel_coredump(signr))", so we don't need to check signr != SIGKILL. We are only interested in the sig_kernel_coredump() signals anyway, and due to the previous changes we almost never can see other fatal signals here except SIGKILL. Signed-off-by: Oleg Nesterov Cc: Roland McGrath Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds --- diff --git a/kernel/signal.c b/kernel/signal.c index 0a873279393..0db1d93c4d6 100644 --- a/kernel/signal.c +++ b/kernel/signal.c @@ -1787,9 +1787,10 @@ relock: * Anything else is fatal, maybe with a core dump. */ current->flags |= PF_SIGNALED; - if ((signr != SIGKILL) && print_fatal_signals) - print_fatal_signal(regs, signr); + if (sig_kernel_coredump(signr)) { + if (print_fatal_signals) + print_fatal_signal(regs, signr); /* * If it was able to dump core, this kills all * other threads in the group and synchronizes with