From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Noah Misch <noah(at)leadboat(dot)com> |
Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: Unexpected "shared memory block is still in use" |
Date: | 2019-09-05 00:25:22 |
Message-ID: | 31925.1567643122@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
I wrote:
> This still isn't committable as-is, since the test will just curl up
> and die on machines lacking IPC::SharedMem.
After a bit of research, here's a version that takes a stab at fixing
that. There may be cleaner ways to do it, but this successfully skips
the test if it can't import the needed IPC modules.
This also fixes a problem that the previous script had with leaking
a shmem segment. That's due to something that could be considered
a pre-existing bug, which is that if we use shmem key X+1, and the
postmaster crashes, and the next start is able to get shmem key X,
we don't clean up the shmem segment at X+1. In principle, we could
note from the contents of postmaster.pid that X+1 was used before and
try to remove it. In practice, I doubt this is worth worrying about
given how small the shmem segments are now, and the very low probability
of key collisions in the new regime. Anyway it would be material for
a different patch.
I think this could be considered committable, but if anyone wants to
improve the test script, step right up.
regards, tom lane
Attachment | Content-Type | Size |
---|---|---|
use-data-dir-inode-for-ipc-keys-3.patch | text/x-diff | 20.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Bossart, Nathan | 2019-09-05 00:37:40 | ERROR: multixact X from before cutoff Y found to be still running |
Previous Message | Michael Paquier | 2019-09-04 23:59:41 | Re: Plug-in common/logging.h with vacuumlo and oid2name |