diff --git a/src/backend/bootstrap/bootstrap.c b/src/backend/bootstrap/bootstrap.c index a9c5d1fd53b631ec9fecaf48900918cf249a0f7c..080d80e296b7b8276f193c648e254e7c6d0dbb1d 100644 --- a/src/backend/bootstrap/bootstrap.c +++ b/src/backend/bootstrap/bootstrap.c @@ -8,7 +8,7 @@ * Portions Copyright (c) 1994, Regents of the University of California * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/bootstrap/bootstrap.c,v 1.260 2010/02/26 02:00:35 momjian Exp $ + * $PostgreSQL: pgsql/src/backend/bootstrap/bootstrap.c,v 1.261 2010/04/20 01:38:52 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -397,14 +397,13 @@ AuxiliaryProcessMain(int argc, char *argv[]) switch (auxType) { case CheckerProcess: - bootstrap_signals(); + /* don't set signals, they're useless here */ CheckerModeMain(); proc_exit(1); /* should never return */ case BootstrapProcess: bootstrap_signals(); BootStrapXLOG(); - StartupXLOG(); BootstrapModeMain(); proc_exit(1); /* should never return */ @@ -438,23 +437,12 @@ AuxiliaryProcessMain(int argc, char *argv[]) /* * In shared memory checker mode, all we really want to do is create shared * memory and semaphores (just to prove we can do it with the current GUC - * settings). + * settings). Since, in fact, that was already done by BaseInit(), + * we have nothing more to do here. */ static void CheckerModeMain(void) { - /* - * We must be getting invoked for bootstrap mode - */ - Assert(!IsUnderPostmaster); - - SetProcessingMode(BootstrapProcessing); - - /* - * Do backend-like initialization for bootstrap mode - */ - InitProcess(); - InitPostgres(NULL, InvalidOid, NULL, NULL); proc_exit(0); } @@ -478,6 +466,7 @@ BootstrapModeMain(void) * Do backend-like initialization for bootstrap mode */ InitProcess(); + InitPostgres(NULL, InvalidOid, NULL, NULL); /* Initialize stuff for bootstrap-file processing */ @@ -498,10 +487,6 @@ BootstrapModeMain(void) */ RelationMapFinishBootstrap(); - /* Perform a checkpoint to ensure everything's down to disk */ - SetProcessingMode(NormalProcessing); - CreateCheckPoint(CHECKPOINT_IS_SHUTDOWN | CHECKPOINT_IMMEDIATE); - /* Clean up and exit */ cleanup(); proc_exit(0); diff --git a/src/backend/tcop/postgres.c b/src/backend/tcop/postgres.c index 287445b535e4f9411b108478b11673c398bc3c69..b1d700a97e40e9fed1e6c010848e98a6848f553b 100644 --- a/src/backend/tcop/postgres.c +++ b/src/backend/tcop/postgres.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/tcop/postgres.c,v 1.592 2010/03/21 00:17:58 petere Exp $ + * $PostgreSQL: pgsql/src/backend/tcop/postgres.c,v 1.593 2010/04/20 01:38:52 tgl Exp $ * * NOTES * this is the "main" module of the postgres backend and @@ -3531,11 +3531,7 @@ PostgresMain(int argc, char *argv[], const char *username) PG_SETMASK(&BlockSig); /* block everything except SIGQUIT */ - if (IsUnderPostmaster) - { - BaseInit(); - } - else + if (!IsUnderPostmaster) { /* * Validate we have been given a reasonable-looking DataDir (if under @@ -3551,17 +3547,11 @@ PostgresMain(int argc, char *argv[], const char *username) * Create lockfile for data directory. */ CreateDataDirLockFile(false); - - BaseInit(); - - /* - * Start up xlog for standalone backend, and register to have it - * closed down at exit. - */ - StartupXLOG(); - on_shmem_exit(ShutdownXLOG, 0); } + /* Early initialization */ + BaseInit(); + /* * Create a per-backend PGPROC struct in shared memory, except in the * EXEC_BACKEND case where this was done in SubPostmasterMain. We must do diff --git a/src/backend/utils/init/postinit.c b/src/backend/utils/init/postinit.c index afd4bc65b273b30264266fc38d040b785f1c7a45..5be80b8f6892202025b7d61371a30e84fe35cd99 100644 --- a/src/backend/utils/init/postinit.c +++ b/src/backend/utils/init/postinit.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/utils/init/postinit.c,v 1.208 2010/03/25 20:40:17 sriggs Exp $ + * $PostgreSQL: pgsql/src/backend/utils/init/postinit.c,v 1.209 2010/04/20 01:38:52 tgl Exp $ * * *------------------------------------------------------------------------- @@ -427,11 +427,9 @@ pg_split_opts(char **argv, int *argcp, char *optstr) * Early initialization of a backend (either standalone or under postmaster). * This happens even before InitPostgres. * - * If you're wondering why this is separate from InitPostgres at all: - * the critical distinction is that this stuff has to happen before we can - * run XLOG-related initialization, which is done before InitPostgres --- in - * fact, for cases such as the background writer process, InitPostgres may - * never be done at all. + * This is separate from InitPostgres because it is also called by auxiliary + * processes, such as the background writer process, which may not call + * InitPostgres at all. */ void BaseInit(void) @@ -512,11 +510,28 @@ InitPostgres(const char *in_dbname, Oid dboid, const char *username, InitBufferPoolBackend(); /* - * Initialize local process's access to XLOG, if appropriate. In - * bootstrap case we skip this since StartupXLOG() was run instead. + * Initialize local process's access to XLOG. */ - if (!bootstrap) + if (IsUnderPostmaster) + { + /* + * The postmaster already started the XLOG machinery, but we need + * to call InitXLOGAccess(), if the system isn't in hot-standby mode. + * This is handled by calling RecoveryInProgress and ignoring the + * result. + */ (void) RecoveryInProgress(); + } + else + { + /* + * We are either a bootstrap process or a standalone backend. + * Either way, start up the XLOG machinery, and register to have it + * closed down at exit. + */ + StartupXLOG(); + on_shmem_exit(ShutdownXLOG, 0); + } /* * Initialize the relation cache and the system catalog caches. Note that