Arrange to call localtime() during postmaster startup. On most Unixen,

the first call of localtime() in a process will read /usr/lib/tztab or
local equivalent.  Better to do this once in the postmaster and inherit
the data by fork() than to have to do it during every backend start.

diff --git a/src/backend/postmaster/postmaster.c b/src/backend/postmaster/postmaster.c
index c480bf2f41f60c60451bd2d5921a7ec9bf04fded..e5ce458c0d664d3ea63e3654c0ae4523d7f57fb7 100644 (file)
--- a/src/backend/postmaster/postmaster.c
+++ b/src/backend/postmaster/postmaster.c
@@ -37,7 +37,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/postmaster/postmaster.c,v 1.264 2002/01/10 01:11:45 tgl Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/postmaster/postmaster.c,v 1.265 2002/02/19 19:53:35 tgl Exp $
  *
  * NOTES
  *
@@ -723,6 +723,10 @@ PostmasterMain(int argc, char *argv[])
 
    /*
     * Set up signal handlers for the postmaster process.
+    *
+    * CAUTION: when changing this list, check for side-effects on the
+    * signal handling setup of child processes.  See tcop/postgres.c,
+    * bootstrap/bootstrap.c, and postmaster/pgstat.c.
     */
    pqinitmask();
    PG_SETMASK(&BlockSig);
@@ -749,6 +753,18 @@ PostmasterMain(int argc, char *argv[])
     */
    whereToSendOutput = None;
 
+   /*
+    * On many platforms, the first call of localtime() incurs significant
+    * overhead to load timezone info from the system configuration files.
+    * By doing it once in the postmaster, we avoid having to do it in every
+    * started child process.  The savings are not huge, but they add up...
+    */
+   {
+       time_t      now = time(NULL);
+
+       (void) localtime(&now);
+   }
+
    /*
     * Initialize and startup the statistics collector process
     */
@@ -1793,7 +1809,6 @@ SignalChildren(int signal)
    Dlelem     *curr,
               *next;
    Backend    *bp;
-   int         mypid = getpid();
 
    curr = DLGetHead(BackendList);
    while (curr)
@@ -1801,7 +1816,7 @@ SignalChildren(int signal)
        next = DLGetSucc(curr);
        bp = (Backend *) DLE_VAL(curr);
 
-       if (bp->pid != mypid)
+       if (bp->pid != MyProcPid)
        {
            if (DebugLvl >= 1)
                elog(DEBUG, "SignalChildren: sending signal %d to process %d",
@@ -2412,13 +2427,12 @@ CountChildren(void)
 {
    Dlelem     *curr;
    Backend    *bp;
-   int         mypid = getpid();
    int         cnt = 0;
 
    for (curr = DLGetHead(BackendList); curr; curr = DLGetSucc(curr))
    {
        bp = (Backend *) DLE_VAL(curr);
-       if (bp->pid != mypid)
+       if (bp->pid != MyProcPid)
            cnt++;
    }
    if (CheckPointPID != 0)