postmaster: Don't open-code TerminateChildren() in HandleChildCrash()

After removing the duplication no user of sigquit_child() remains, therefore
remove it.

Reviewed-by: Bertrand Drouvot <[email protected]>
Discussion: https://postgr.es/m/kgng5nrvnlv335evmsuvpnh354rw7qyazl73kdysev2cr2v5zu@m3cfzxicm5kp

diff --git a/src/backend/postmaster/postmaster.c b/src/backend/postmaster/postmaster.c
index 5488289a489bbba4fa9866a3ec605a95cb99c905..ea92fb56c80a2739b1173f7380de6b31ea86bbd1 100644 (file)
--- a/src/backend/postmaster/postmaster.c
+++ b/src/backend/postmaster/postmaster.c
@@ -243,6 +243,13 @@ bool       enable_bonjour = false;
 char      *bonjour_name;
 bool       restart_after_crash = true;
 bool       remove_temp_files_after_crash = true;
+
+/*
+ * When terminating child processes after fatal errors, like a crash of a
+ * child process, we normally send SIGQUIT -- and most other comments in this
+ * file are written on the assumption that we do -- but developers might
+ * prefer to use SIGABRT to collect per-child core dumps.
+ */
 bool       send_abort_for_crash = false;
 bool       send_abort_for_kill = false;
 
@@ -424,7 +431,6 @@ static int  BackendStartup(ClientSocket *client_sock);
 static void report_fork_failure_to_client(ClientSocket *client_sock, int errnum);
 static CAC_state canAcceptConnections(BackendType backend_type);
 static void signal_child(PMChild *pmchild, int signal);
-static void sigquit_child(PMChild *pmchild);
 static bool SignalChildren(int signal, BackendTypeMask targetMask);
 static void TerminateChildren(int signal);
 static int CountChildren(BackendTypeMask targetMask);
@@ -2701,32 +2707,12 @@ HandleChildCrash(int pid, int exitstatus, const char *procname)
    /*
     * Signal all other child processes to exit.  The crashed process has
     * already been removed from ActiveChildList.
+    *
+    * We could exclude dead-end children here, but at least when sending
+    * SIGABRT it seems better to include them.
     */
    if (take_action)
-   {
-       dlist_iter  iter;
-
-       dlist_foreach(iter, &ActiveChildList)
-       {
-           PMChild    *bp = dlist_container(PMChild, elem, iter.cur);
-
-           /* We do NOT restart the syslogger */
-           if (bp == SysLoggerPMChild)
-               continue;
-
-           if (bp == StartupPMChild)
-               StartupStatus = STARTUP_SIGNALED;
-
-           /*
-            * This backend is still alive.  Unless we did so already, tell it
-            * to commit hara-kiri.
-            *
-            * We could exclude dead-end children here, but at least when
-            * sending SIGABRT it seems better to include them.
-            */
-           sigquit_child(bp);
-       }
-   }
+       TerminateChildren(send_abort_for_crash ? SIGABRT : SIGQUIT);
 
    if (Shutdown != ImmediateShutdown)
        FatalError = true;
@@ -3349,19 +3335,6 @@ signal_child(PMChild *pmchild, int signal)
 #endif
 }
 
-/*
- * Convenience function for killing a child process after a crash of some
- * other child process.  We apply send_abort_for_crash to decide which signal
- * to send.  Normally it's SIGQUIT -- and most other comments in this file are
- * written on the assumption that it is -- but developers might prefer to use
- * SIGABRT to collect per-child core dumps.
- */
-static void
-sigquit_child(PMChild *pmchild)
-{
-   signal_child(pmchild, (send_abort_for_crash ? SIGABRT : SIGQUIT));
-}
-
 /*
  * Send a signal to the targeted children.
  */