]> granicus.if.org Git - postgresql/commit
Allow on-detach callbacks for dynamic shared memory segments.
authorRobert Haas <rhaas@postgresql.org>
Wed, 18 Dec 2013 17:57:20 +0000 (12:57 -0500)
committerRobert Haas <rhaas@postgresql.org>
Wed, 18 Dec 2013 18:09:09 +0000 (13:09 -0500)
commit001a573a2011d605f2a6e10aee9996de8581d099
tree5bec5fa0c42508d9d17ae324191f8c6d3ca7e05d
parent613c6d26bd42dd8c2dd0664315be9551475b8864
Allow on-detach callbacks for dynamic shared memory segments.

Just as backends must clean up their shared memory state (releasing
lwlocks, buffer pins, etc.) before exiting, they must also perform
any similar cleanups related to dynamic shared memory segments they
have mapped before unmapping those segments.  So add a mechanism to
ensure that.

Existing on_shmem_exit hooks include both "user level" cleanup such
as transaction abort and removal of leftover temporary relations and
also "low level" cleanup that forcibly released leftover shared
memory resources.  On-detach callbacks should run after the first
group but before the second group, so create a new before_shmem_exit
function for registering the early callbacks and keep on_shmem_exit
for the regular callbacks.  (An earlier draft of this patch added an
additional argument to on_shmem_exit, but that had a much larger
footprint and probably a substantially higher risk of breaking third
party code for no real gain.)

Patch by me, reviewed by KaiGai Kohei and Andres Freund.
src/backend/bootstrap/bootstrap.c
src/backend/catalog/namespace.c
src/backend/commands/async.c
src/backend/storage/ipc/dsm.c
src/backend/storage/ipc/ipc.c
src/backend/utils/init/postinit.c
src/include/storage/dsm.h
src/include/storage/ipc.h