diff options
| author | Andres Freund <andres@anarazel.de> | 2025-10-09 16:27:08 -0400 |
|---|---|---|
| committer | Andres Freund <andres@anarazel.de> | 2025-10-09 19:17:13 -0400 |
| commit | c819d1017ddb349d92ab323d2631bc1f10bb4e86 (patch) | |
| tree | d5f169df2f959aeac21c4f2663ffad6dbea5bf8e /src/backend/access/transam/xlogstats.c | |
| parent | 9d46b86529e8337e34fb5a65b1a7dca9dc53938f (diff) | |
bufmgr: Fix valgrind checking for buffers pinned in StrategyGetBuffer()
In 5e899859287 I made StrategyGetBuffer() pin buffers with a single CAS,
instead of using PinBuffer_Locked(). Unfortunately I missed that
PinBuffer_Locked() marked the page as defined for valgrind.
Fix this oversight by centralizing the valgrind initialization into
TrackNewBufferPin(), which also allows us to reduce the number of places doing
VALGRIND_MAKE_MEM_DEFINED.
Per buildfarm animal skink and Amit Langote.
Discussion: https://postgr.es/m/fvfmkr5kk4nyex56ejgxj3uzi63isfxovp2biecb4bspbjrze7@az2pljabhnff
Discussion: https://postgr.es/m/CA+HiwqGKJ6nEXEPQW7EpykVsEtzxp5-up_xhtcUAkWFtATVQvQ@mail.gmail.com
Diffstat (limited to 'src/backend/access/transam/xlogstats.c')
0 files changed, 0 insertions, 0 deletions