From: | Thomas Munro <tmunro(at)postgresql(dot)org> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: Fix RBM_ZERO_AND_LOCK. |
Date: | 2024-06-10 02:08:06 |
Message-ID: | E1sGUS5-000orA-Im@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Fix RBM_ZERO_AND_LOCK.
Commit 210622c6 accidentally zeroed out pages even if they were found in
the buffer pool. It should always lock the page, but it should only
zero pages that were not already valid. Otherwise, concurrent readers
that hold only a pin could see corrupted page contents changing under
their feet.
While here, rename ZeroAndLockBuffer() to match the RBM_ flag name.
Also restore a some useful comments lost by 210622c6's refactoring, and
add some new ones to clarify why we need to use the BM_IO_IN_PROGRESS
infrastructure despite not doing I/O.
Reported-by: Noah Misch <noah(at)leadboat(dot)com>
Reported-by: Alexander Lakhin <exclusion(at)gmail(dot)com>
Reviewed-by: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> (earlier version)
Reviewed-by: Robert Haas <robertmhaas(at)gmail(dot)com> (earlier version)
Discussion: https://postgr.es/m/20240512171658.7e.nmisch@google.com
Discussion: https://postgr.es/m/7ed10231-ce47-03d5-d3f9-4aea0dc7d5a4%40gmail.com
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/e656657f2b93a96b83fd4038ec5120c5ec9845e5
Modified Files
--------------
src/backend/storage/buffer/bufmgr.c | 88 ++++++++++++++++++++++++++++---------
1 file changed, 67 insertions(+), 21 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Richard Guo | 2024-06-10 04:10:27 | pgsql: Fix comment about cross-checking the varnullingrels |
Previous Message | Tom Lane | 2024-06-07 20:46:12 | pgsql: Tighten test_predtest's input checks, and improve error messages |