diff options
| author | Thomas Munro <tmunro@postgresql.org> | 2023-01-12 12:34:23 +1300 |
|---|---|---|
| committer | Thomas Munro <tmunro@postgresql.org> | 2023-01-12 16:32:20 +1300 |
| commit | 7389aad63666a2cac18cd6d7496378d7f50ef37b (patch) | |
| tree | dfcfb672a82df147e9e73a536332eb7dd5a2770d /src/backend/tcop/postgres.c | |
| parent | d93d68aeeaeeda0e871825b461fd9ab68c7c0de3 (diff) | |
| download | postgresql-7389aad63666a2cac18cd6d7496378d7f50ef37b.tar.gz postgresql-7389aad63666a2cac18cd6d7496378d7f50ef37b.zip | |
Use WaitEventSet API for postmaster's event loop.
Switch to a design similar to regular backends, instead of the previous
arrangement where signal handlers did non-trivial state management and
called fork(). The main changes are:
* The postmaster now has its own local latch to wait on. (For now, we
don't want other backends setting its latch directly, but that could
probably be made to work with more research on robustness.)
* The existing signal handlers are cut in two: a handle_pm_XXX() part
that just sets pending_pm_XXX flags and the latch, and a
process_pm_XXX() part that runs later when the latch is seen.
* Signal handlers are now installed with the regular pqsignal()
function rather than the special pqsignal_pm() function; historical
portability concerns about the effect of SA_RESTART on select() are no
longer relevant, and we don't need to block signals anymore.
Reviewed-by: Andres Freund <andres@anarazel.de>
Discussion: https://postgr.es/m/CA%2BhUKG%2BZ-HpOj1JsO9eWUP%2Bar7npSVinsC_npxSy%2BjdOMsx%3DGg%40mail.gmail.com
Diffstat (limited to 'src/backend/tcop/postgres.c')
| -rw-r--r-- | src/backend/tcop/postgres.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/backend/tcop/postgres.c b/src/backend/tcop/postgres.c index 224ab290afd..470b734e9e0 100644 --- a/src/backend/tcop/postgres.c +++ b/src/backend/tcop/postgres.c @@ -24,7 +24,6 @@ #include <signal.h> #include <unistd.h> #include <sys/resource.h> -#include <sys/select.h> #include <sys/socket.h> #include <sys/time.h> |