On 5/21/07, Folkert van Heusden <[EMAIL PROTECTED]> wrote:
> >What about the following enhancement: I check with sig_fatal if it would > >kill the process and only then emit a message. So when an application > >takes care itself of handling it nothing is printed. > >+ /* emit some logging for unhandled signals > >+ */ > >+ if (sig_fatal(t, sig)) > Not unhandled_signal()?Can we already use that one in send_signal? As the signal needs to be send first I think before we know if it was handled or not? sig_fatal checks if the handler is set to default - which is it is not taken care of. > >+ { > if (sig_fatal(t, sig)) { > >+ printk(KERN_WARNING "Sig %d send to %d owned by %d.%d (%s)\n", > s/send/sent/; > >+ sig, t -> pid, t -> uid, t -> gid, t -> comm); > t->pid, t->uid, t->gid, t->comm); Description: This patch adds code to the signal-sender making it log a message when an unhandled fatal signal will be delivered.
Gargh ... why does this want to be in the *kernel*'s logs? In any case, can you please make this KERN_INFO (or lower) instead of KERN_WARNING. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
