]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Fix generic read and write barriers for Clang. REL_15_STABLE github/REL_15_STABLE
authorThomas Munro <tmunro@postgresql.org>
Fri, 7 Nov 2025 23:25:45 +0000 (12:25 +1300)
committerThomas Munro <tmunro@postgresql.org>
Fri, 7 Nov 2025 23:30:08 +0000 (12:30 +1300)
commit1c7cba4c52953d2486a1465ea8d91ff65c35610c
treefb14e6daaae1c38cb61ea04c1a6db6db268dc2c4
parenta2e17a2bb20018c0e46a9538e547542cae030103
Fix generic read and write barriers for Clang.

generic-gcc.h maps our read and write barriers to C11 acquire and
release fences using compiler builtins, for platforms where we don't
have our own hand-rolled assembler.  This is apparently enough for GCC,
but the C11 memory model is only defined in terms of atomic accesses,
and our barriers for non-atomic, non-volatile accesses were not always
respected under Clang's stricter interpretation of the standard.

This explains the occasional breakage observed on new RISC-V + Clang
animal greenfly in lock-free PgAioHandle manipulation code containing a
repeating pattern of loads and read barriers.  The problem can also be
observed in code generated for MIPS and LoongAarch, though we aren't
currently testing those with Clang, and on x86, though we use our own
assembler there.  The scariest aspect is that we use the generic version
on very common ARM systems, but it doesn't seem to reorder the relevant
code there (or we'd have debugged this long ago).

Fix by inserting an explicit compiler barrier.  It expands to an empty
assembler block declared to have memory side-effects, so registers are
flushed and reordering is prevented.  In those respects this is like the
architecture-specific assembler versions, but the compiler is still in
charge of generating the appropriate fence instruction.  Done for write
barriers on principle, though concrete problems have only been observed
with read barriers.

Reported-by: Alexander Lakhin <exclusion@gmail.com>
Tested-by: Alexander Lakhin <exclusion@gmail.com>
Discussion: https://postgr.es/m/d79691be-22bd-457d-9d90-18033b78c40a%40gmail.com
Backpatch-through: 13
src/include/port/atomics/generic-gcc.h