Fix latch event policy that hid socket events.
authorThomas Munro <tmunro@postgresql.org>
Mon, 20 Jan 2025 02:17:47 +0000 (15:17 +1300)
committerThomas Munro <tmunro@postgresql.org>
Mon, 20 Jan 2025 03:43:29 +0000 (16:43 +1300)
commit73f6b9a3b0fe161cd3d35b1e3109cd47864a4b33
tree6cf31b3c26e292cdd0f149e6e1cb5f08acba4ed6
parent6cf1647d87e7cd423d71525a8759b75c4e4a47ec
Fix latch event policy that hid socket events.

If a WaitEventSetWait() caller asks for multiple events, an already set
latch would previously prevent other events from being reported at the
same time.  Now, we'll also poll the kernel for other events that would
fit in the caller's output buffer with a zero wait time.  This policy
change doesn't affect callers that ask for only one event.

The main caller affected is the postmaster.  If its latch is set
extremely frequently by backends launching workers and workers exiting,
we don't want it to handle only those jobs and ignore incoming client
connections.

Back-patch to 16 where the postmaster began using the API.  The
fast-return policy changed here is older than that, but doesn't cause
any known problems in earlier releases.

Reported-by: Nathan Bossart <nathandbossart@gmail.com>
Reviewed-by: Nathan Bossart <nathandbossart@gmail.com>
Discussion: https://postgr.es/m/Z1n5UpAiGDmFcMmd%40nathan
src/backend/storage/ipc/latch.c