Hi, On 2018-04-05 22:06:36 +0200, Magnus Hagander wrote: > I have now pushed this latest version with some minor text adjustments and > a catversion bump.
Is there any sort of locking that guarantees that worker processes see an up2date value of DataChecksumsNeedWrite()/ControlFile->data_checksum_version? Afaict there's not. So you can afaict end up with checksums being computed by the worker, but concurrent writes missing them. The window is going to be at most one missed checksum per process (as the unlocking of the page is a barrier) and is probably not easy to hit, but that's dangerous enough. Just plonking a barrier into DataChecksumsNeedWrite() etc is a possibility, but it's also not free... Greetings, Andres Freund