]> granicus.if.org Git - postgresql/commit
Handle impending sinval queue overflow by means of a separate signal
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 23 May 2004 03:50:45 +0000 (03:50 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 23 May 2004 03:50:45 +0000 (03:50 +0000)
commitebfc56d3fb41d914c799555a1f4c9d1a72379e9f
tree7fb674efed5b26eb9a11b0e77733b2c8e7eaf642
parent4d86ae42600c42834a55371630416e98593b7b11
Handle impending sinval queue overflow by means of a separate signal
(SIGUSR1, which we have not been using recently) instead of piggybacking
on SIGUSR2-driven NOTIFY processing.  This has several good results:
the processing needed to drain the sinval queue is a lot less than the
processing needed to answer a NOTIFY; there's less contention since we
don't have a bunch of backends all trying to acquire exclusive lock on
pg_listener; backends that are sitting inside a transaction block can
still drain the queue, whereas NOTIFY processing can't run if there's
an open transaction block.  (This last is a fairly serious issue that
I don't think we ever recognized before --- with clients like JDBC that
tend to sit with open transaction blocks, the sinval queue draining
mechanism never really worked as intended, probably resulting in a lot
of useless cache-reset overhead.)  This is the last of several proposed
changes in response to Philip Warner's recent report of sinval-induced
performance problems.
src/backend/commands/async.c
src/backend/postmaster/postmaster.c
src/backend/storage/ipc/sinval.c
src/backend/storage/ipc/sinvaladt.c
src/backend/tcop/postgres.c
src/include/commands/async.h
src/include/storage/pmsignal.h
src/include/storage/sinval.h