pgsql: Fix two bugs in setting the vm bit of empty pages. - Mailing list pgsql-committers

From Heikki Linnakangas
Subject pgsql: Fix two bugs in setting the vm bit of empty pages.
Date
Msg-id E1VYwgS-0005bq-QU@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix two bugs in setting the vm bit of empty pages.

Use a critical section when setting the all-visible flag on an empty page,
and WAL-logging it. log_newpage_buffer() contains an assertion that it
must be called inside a critical section, and it's the right thing to do
when modifying a buffer anyway.

Also, the page should be marked dirty before calling log_newpage_buffer(),
per the comment in log_newpage_buffer() and src/backend/access/transam/README.

Patch by Andres Freund, in response to my report. Backpatch to 9.2, like
the patch that introduced these bugs (a6370fd9).

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/83eb54001cb69d6ee8f0813c4e280be876823068

Modified Files
--------------
src/backend/commands/vacuumlazy.c |    7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Suppress a couple of compiler warnings seen with older gcc versi
Next
From: Heikki Linnakangas
Date:
Subject: pgsql: Fix two bugs in setting the vm bit of empty pages.