Re: [PATCH] Add log message to SIGINT handler

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/09/13 15:50, Steven Dake wrote:
On 09/03/2013 02:43 AM, Christine Caulfield wrote:
I've seen a few instances where corosync has shut down for apparently
'no reason'. In fact most of the time the shutdown has been caused by
an external source (often an init script) but it's not been obvious
what has happened and people implicate the deamon

This patch simply adds a log message to the SIGINT handler when it is
called so that the cause of the shutdown is obvious.

Signed-off-by: Christine Caulfield <ccaulfie@xxxxxxxxxx>



While the idea seems interesting, I would recommend generally not doing
this in a signal handler, and figuring out another way to log the
message.   The logging code is not really designed to be called from a
signal handler, and I expect even libqb has same problems.

Angus could you add your insight on libqb signal handle logging?


I must admit I was concerned, but there are far 'worse' things in some of the other signal handlers!

Chrissie
_______________________________________________
discuss mailing list
discuss@xxxxxxxxxxxx
http://lists.corosync.org/mailman/listinfo/discuss




[Index of Archives]     [Linux Clusters]     [Corosync Project]     [Linux USB Devel]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Linux Kernel]     [Linux SCSI]     [X.Org]

  Powered by Linux