database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-06-23 14:01:44 +03:00
Code Activity

postmaster: Improve logging of signals sent by postmaster

Browse Source 
Previously many, in some cases important, signals we never logged. In other
cases the signal name was only included numerically.

As part of this, change the debug log level the signal is logged at to DEBUG3,
previously some where DEBUG2, some DEBUG4.

Also move from direct use of kill() to signal the av launcher to
signal_child(). There doesn't seem to be a reason for directly using kill().

Reviewed-by: Heikki Linnakangas <hlinnaka@iki.fi>
Reviewed-by: Bertrand Drouvot <bertranddrouvot.pg@gmail.com>
Reviewed-by: Nazir Bilal Yavuz <byavuz81@gmail.com>
Discussion: https://postgr.es/m/kgng5nrvnlv335evmsuvpnh354rw7qyazl73kdysev2cr2v5zu@m3cfzxicm5kp
This commit is contained in:
Andres Freund
2025-01-10 11:08:17 -05:00
parent 7148cbbdc6
commit 40d4031abd
1 changed files with 43 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

56
src/backend/postmaster/postmaster.c
View File

@ -1693,7 +1693,7 @@ ServerLoop(void)
{
avlauncher_needs_signal = false;
if (AutoVacLauncherPMChild != NULL)
kill(AutoVacLauncherPMChild->pid, SIGUSR2);
signal_child(AutoVacLauncherPMChild, SIGUSR2);
}
#ifdef HAVE_PTHREAD_IS_THREADED_NP
@ -3256,6 +3256,38 @@ LaunchMissingBackgroundProcesses(void)
maybe_start_bgworkers();
}
/*
* Return string representation of signal.
*
* Because this is only implemented for signals we already rely on in this
* file we don't need to deal with unimplemented or same-numeric-value signals
* (as we'd e.g. have to for EWOULDBLOCK / EAGAIN).
*/
static const char *
pm_signame(int signal)
{
#define PM_TOSTR_CASE(sym) case sym: return #sym
switch (signal)
{
PM_TOSTR_CASE(SIGABRT);
PM_TOSTR_CASE(SIGCHLD);
PM_TOSTR_CASE(SIGHUP);
PM_TOSTR_CASE(SIGINT);
PM_TOSTR_CASE(SIGKILL);
PM_TOSTR_CASE(SIGQUIT);
PM_TOSTR_CASE(SIGTERM);
PM_TOSTR_CASE(SIGUSR1);
PM_TOSTR_CASE(SIGUSR2);
default:
/* all signals sent by postmaster should be listed here */
Assert(false);
return "(unknown)";
}
#undef PM_TOSTR_CASE
return ""; /* silence compiler */
}
/*
* Send a signal to a postmaster child process
*
@ -3277,6 +3309,12 @@ signal_child(PMChild *pmchild, int signal)
{
pid_t pid = pmchild->pid;
ereport(DEBUG3,
(errmsg_internal("sending signal %d/%s to %s process with pid %d",
signal, pm_signame(signal),
GetBackendTypeDesc(pmchild->bkend_type),
(int) pmchild->pid)));
if (kill(pid, signal) < 0)
elog(DEBUG3, "kill(%ld,%d) failed: %m", (long) pid, signal);
#ifdef HAVE_SETSID
@ -3298,19 +3336,14 @@ signal_child(PMChild *pmchild, int signal)
/*
* Convenience function for killing a child process after a crash of some
* other child process. We log the action at a higher level than we would
* otherwise do, and we apply send_abort_for_crash to decide which signal
* to send. Normally it's SIGQUIT -- and most other comments in this file
* are written on the assumption that it is -- but developers might prefer
* to use SIGABRT to collect per-child core dumps.
* other child process. We apply send_abort_for_crash to decide which signal
* to send. Normally it's SIGQUIT -- and most other comments in this file are
* written on the assumption that it is -- but developers might prefer to use
* SIGABRT to collect per-child core dumps.
*/
static void
sigquit_child(PMChild *pmchild)
{
ereport(DEBUG2,
(errmsg_internal("sending %s to process %d",
(send_abort_for_crash ? "SIGABRT" : "SIGQUIT"),
(int) pmchild->pid)));
signal_child(pmchild, (send_abort_for_crash ? SIGABRT : SIGQUIT));
}
@ -3342,9 +3375,6 @@ SignalChildren(int signal, BackendTypeMask targetMask)
if (!btmask_contains(targetMask, bp->bkend_type))
continue;
ereport(DEBUG4,
(errmsg_internal("sending signal %d to %s process %d",
signal, GetBackendTypeDesc(bp->bkend_type), (int) bp->pid)));
signal_child(bp, signal);
signaled = true;
}