diff --git a/src/backend/storage/ipc/ipci.c b/src/backend/storage/ipc/ipci.c index fb27b92d2240a46a40ff6bccc07e85ca1d0f9a22..460f0ee10bba361677cf2d322ee1e8c86ece97e4 100644 --- a/src/backend/storage/ipc/ipci.c +++ b/src/backend/storage/ipc/ipci.c @@ -7,7 +7,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/storage/ipc/ipci.c,v 1.19 1999/02/19 06:06:04 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/storage/ipc/ipci.c,v 1.20 1999/02/19 07:10:47 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -101,7 +101,6 @@ CreateSharedMemoryAndSemaphores(IPCKey key, int maxBackends) * ---------------- */ InitProcGlobal(key, maxBackends); - on_shmem_exit(ProcFreeAllSemaphores, NULL); CreateSharedInvalidationState(key); } diff --git a/src/backend/storage/lmgr/proc.c b/src/backend/storage/lmgr/proc.c index ae8fb17fcabcc3d393e1d58e545bb8b60673de4b..7452270fdf47e9f297d58173e62bddd11f45e583 100644 --- a/src/backend/storage/lmgr/proc.c +++ b/src/backend/storage/lmgr/proc.c @@ -7,7 +7,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/storage/lmgr/proc.c,v 1.49 1999/02/19 06:06:08 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/storage/lmgr/proc.c,v 1.50 1999/02/19 07:10:48 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -46,7 +46,7 @@ * This is so that we can support more backends. (system-wide semaphore * sets run out pretty fast.) -ay 4/95 * - * $Header: /cvsroot/pgsql/src/backend/storage/lmgr/proc.c,v 1.49 1999/02/19 06:06:08 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/storage/lmgr/proc.c,v 1.50 1999/02/19 07:10:48 tgl Exp $ */ #include <sys/time.h> #include <unistd.h> @@ -79,6 +79,7 @@ static void HandleDeadLock(int sig); static PROC *ProcWakeup(PROC *proc, int errType); +static void ProcFreeAllSemaphores(void); #define DeadlockCheckTimer pg_options[OPT_DEADLOCKTIMEOUT] @@ -135,6 +136,8 @@ InitProcGlobal(IPCKey key, int maxBackends) /* -------------------- * We're the first - initialize. + * XXX if found should ever be true, it is a sign of impending doom ... + * ought to complain if so? * -------------------- */ if (!found) @@ -145,6 +148,12 @@ InitProcGlobal(IPCKey key, int maxBackends) ProcGlobal->currKey = IPCGetProcessSemaphoreInitKey(key); for (i = 0; i < MAX_PROC_SEMS / PROC_NSEMS_PER_SET; i++) ProcGlobal->freeSemMap[i] = 0; + + /* Arrange to delete semas on exit --- set this up now so that + * we will clean up if pre-allocation fails... + */ + on_shmem_exit(ProcFreeAllSemaphores, NULL); + /* Pre-create the semaphores for the first maxBackends processes */ for (i = 0; i < (maxBackends+PROC_NSEMS_PER_SET-1) / PROC_NSEMS_PER_SET; @@ -924,7 +933,7 @@ ProcFreeSem(IpcSemaphoreKey semKey, int semNum) * destroying shared state for a failed set of backends. * Free up all the semaphores allocated to the lmgrs of the backends. */ -void +static void ProcFreeAllSemaphores() { int i; diff --git a/src/include/storage/proc.h b/src/include/storage/proc.h index 47ae31d2d0103318c8ede7697f8ef1cda669645e..952f50553ca47a6662604ad0eff2c20259e9fc35 100644 --- a/src/include/storage/proc.h +++ b/src/include/storage/proc.h @@ -6,7 +6,7 @@ * * Copyright (c) 1994, Regents of the University of California * - * $Id: proc.h,v 1.19 1999/02/19 06:06:37 tgl Exp $ + * $Id: proc.h,v 1.20 1999/02/19 07:10:47 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -122,6 +122,5 @@ extern int ProcLockWakeup(PROC_QUEUE *queue, LOCKMETHOD lockmethod, LOCK *lock); extern void ProcAddLock(SHM_QUEUE *elem); extern void ProcReleaseSpins(PROC *proc); -extern void ProcFreeAllSemaphores(void); #endif /* PROC_H */