Re: VM corruption on standby
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Andrey Borodin <x4mmm@yandex-team.ru>
Cc: Kirill Reshke <reshkekirill@gmail.com>,
Yura Sokolov <y.sokolov@postgrespro.ru>,
Thomas Munro <thomas.munro@gmail.com>,
PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>,
Melanie Plageman <melanieplageman@gmail.com>,
Michael Paquier <michael@paquier.xyz>
Date: 2025-08-19T19:55:41Z
Lists: pgsql-hackers
Andrey Borodin <x4mmm@yandex-team.ru> writes: > I believe there is a bug with PageIsAllVisible(page) && visibilitymap_clear(). But I cannot prove it with an injection point test. Because injections points rely on CondVar, that per se creates corruption in critical section. So I'm reading this discussion and wonder if CondVar will be fixed in some clever way or I'd better invent new injection point wait mechanism. Yeah, I was coming to similar conclusions in the reply I just sent: we don't really want a policy that we can't put injection-point-based delays inside critical sections. So that infrastructure is leaving something to be desired. Having said that, the test script is also doing something we tell people not to do, namely SIGKILL the postmaster. Could we use SIGQUIT (immediate shutdown) instead? regards, tom lane
Commits
Same data as JSON:
GET /api/v1/messages/:b64id/commits
the thread's linked commits as JSON, with link sources.
API reference →
-
Revert "Get rid of WALBufMappingLock"
- 2ce6abdf5085 18.0 landed
- c13070a27b63 19 (unreleased) landed
-
Get rid of WALBufMappingLock
- bc22dc0e0ddc 18.0 cited
-
Split WaitEventSet functions to separate source file
- 393e0d231405 18.0 cited
-
Add WL_EXIT_ON_PM_DEATH pseudo-event.
- cfdf4dc4fc96 12.0 cited