Wake up autovacuum launcher from postmaster when a worker exits

When an autovacuum worker exits, the launcher needs to be notified
with SIGUSR2, so that it can rebalance and possibly launch a new
worker. The launcher must be notified only after the worker has
finished ProcKill(), so that the worker slot is available for a new
worker. Before this commit, the autovacuum worker was responsible for
that, which required a slightly complicated dance to pass the
launcher's PID from FreeWorkerInfo() to ProcKill() in a global
variable.

Simplify that by moving the responsibility of the signaling to the
postmaster. The postmaster was already doing it when it failed to fork
a worker process, so it seems logical to make it responsible for
notifying the launcher on worker exit too. That's also how the
notification on background worker exit is done.

Reviewed-by: Nathan Bossart <[email protected]>
Reviewed-by: Masahiko Sawada <[email protected]>
Reviewed-by: li carol <[email protected]>
Discussion: 
https://www.postgresql.org/message-id/[email protected]

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/d9c3c943653740d1df44385c3cd658ed67bb4fb2

Modified Files
--------------
src/backend/postmaster/autovacuum.c | 22 ----------------------
src/backend/postmaster/postmaster.c |  8 ++++++++
src/backend/storage/lmgr/proc.c     |  4 ----
src/include/postmaster/autovacuum.h |  3 ---
4 files changed, 8 insertions(+), 29 deletions(-)

Reply via email to