postgresql/src/port/pqsignal.c

Ignoring revisions in .git-blame-ignore-revs. Click here to bypass and see the normal blame view.

81 lines
2.8 KiB
C
Raw Normal View History

/*-------------------------------------------------------------------------
*
* pqsignal.c
* reliable BSD-style signal(2) routine stolen from RWW who stole it
* from Stevens...
*
* Portions Copyright (c) 1996-2023, PostgreSQL Global Development Group
* Portions Copyright (c) 1994, Regents of the University of California
*
*
* IDENTIFICATION
* src/port/pqsignal.c
*
* This is the signal() implementation from "Advanced Programming in the UNIX
* Environment", with minor changes. It was originally a replacement needed
* for old SVR4 systems whose signal() behaved as if sa_flags = SA_RESETHAND |
* SA_NODEFER, also known as "unreliable" signals due to races when the
* handler was reset.
*
* By now, all known modern Unix systems have a "reliable" signal() call.
* We still don't want to use it though, because it remains
* implementation-defined by both C99 and POSIX whether the handler is reset
* or signals are blocked when the handler runs, and default restart behavior
* is also unspecified. Therefore we take POSIX's advice and call sigaction()
* so we can provide explicit sa_flags, but wrap it in this more convenient
* traditional interface style. It also provides a place to set any extra
* flags we want everywhere, such as SA_NOCLDSTOP.
*
* Windows, of course, is resolutely in a class by itself. In the backend,
* this relies on pqsigaction() in src/backend/port/win32/signal.c, which
* provides limited emulation of reliable signals.
*
* Frontend programs can use this version of pqsignal() to forward to the
* native Windows signal() call if they wish, but beware that Windows signals
* behave quite differently. Only the 6 signals required by C are supported.
* SIGINT handlers run in another thread instead of interrupting an existing
* thread, and the others don't interrupt system calls either, so SA_RESTART
* is moot. All except SIGFPE have SA_RESETHAND semantics, meaning the
* handler is reset to SIG_DFL each time it runs. The set of things you are
* allowed to do in a handler is also much more restricted than on Unix,
* according to the documentation.
*
* ------------------------------------------------------------------------
*/
#include "c.h"
#include <signal.h>
#ifndef FRONTEND
#include "libpq/pqsignal.h"
#endif
/*
Run the postmaster's signal handlers without SA_RESTART. The postmaster keeps signals blocked everywhere except while waiting for something to happen in ServerLoop(). The code expects that the select(2) will be cancelled with EINTR if an interrupt occurs; without that, followup actions that should be performed by ServerLoop() itself will be delayed. However, some platforms interpret the SA_RESTART signal flag as meaning that they should restart rather than cancel the select(2). Worse yet, some of them restart it with the original timeout delay, meaning that a steady stream of signal interrupts can prevent ServerLoop() from iterating at all if there are no incoming connection requests. Observable symptoms of this, on an affected platform such as HPUX 10, include extremely slow parallel query startup (possibly as much as 30 seconds) and failure to update timestamps on the postmaster's sockets and lockfiles when no new connections arrive for a long time. We can fix this by running the postmaster's signal handlers without SA_RESTART. That would be quite a scary change if the range of code where signals are accepted weren't so tiny, but as it is, it seems safe enough. (Note that postmaster children do, and must, reset all the handlers before unblocking signals; so this change should not affect any child process.) There is talk of rewriting the postmaster to use a WaitEventSet and not do signal response work in signal handlers, at which point it might be appropriate to revert this patch. But that's not happening before v11 at the earliest. Back-patch to 9.6. The problem exists much further back, but the worst symptom arises only in connection with parallel query, so it does not seem worth taking any portability risks in older branches. Discussion: https://postgr.es/m/9205.1492833041@sss.pgh.pa.us
2017-04-24 19:00:23 +02:00
* Set up a signal handler, with SA_RESTART, for signal "signo"
*
* Returns the previous handler.
*/
pqsigfunc
pqsignal(int signo, pqsigfunc func)
{
#if !(defined(WIN32) && defined(FRONTEND))
struct sigaction act,
oact;
act.sa_handler = func;
sigemptyset(&act.sa_mask);
act.sa_flags = SA_RESTART;
#ifdef SA_NOCLDSTOP
if (signo == SIGCHLD)
act.sa_flags |= SA_NOCLDSTOP;
#endif
if (sigaction(signo, &act, &oact) < 0)
return SIG_ERR;
return oact.sa_handler;
#else
/* Forward to Windows native signal system. */
return signal(signo, func);
#endif
}