Changeset 26782 for branches/eam_branches/20091201/Ohana/src/opihi/pclient/pclient.c.in

Timestamp:

Feb 5, 2010, 1:20:45 PM (16 years ago)

Author:

eugene

Message:

gracefully exit pclient if pcontrol dies and raised a PIPE signal, but keep running if it is the pclient child process that died

Location:

branches/eam_branches/20091201/Ohana

Files:

• . (modified) (1 prop)
• src/opihi/pclient/pclient.c.in (modified) (2 diffs)

branches/eam_branches/20091201/Ohana/src/opihi/pclient/pclient.c.in

@@ -46 +46 @@
 
   /* ignore the history file.  to change this, see, eg, mana.c */
-  signal (SIGPIPE, gotsignal);
+
+  /* We do not want pclient to exit just because the connection to the 
+     child process is broken.  However, if the connection to the parent 
+     process is broken, we should exit.  To handle this, we define 
+     pipe_signal() which tracks the number of times it has been called.
+     Every time an I/O operation on the child file descriptor raises this signal, 
+     we need to manage the crashed child and then clear the count.  Signals raised by 
+     I/O to the parent do not result in clearing the count.  If the count passes a maximum,
+     we exit pclient */
+     
+  signal (SIGPIPE, pipe_signal);
   signal (SIGTSTP, gotsignal);
   signal (SIGTTIN, gotsignal);
@@ -67 +77 @@
 }
 
+static int Npipe = 0;
+
+void pipe_signal (int signum) {
+  Npipe ++;
+  if (Npipe > 100) {
+    exit (5);
+  }
+  gprint (GP_ERR, "closed pipe : did child process exit unexpectedly?\n");
+  return;
+}
+
+void pipe_signal_clear(void) {
+  Npipe = 0;
+}
+
 void gotsignal (int signum) {
   gprint (GP_ERR, "got signal : %d\n", signum);