pgsql: Use static inline function for BufferGetPage() - Mailing list pgsql-committers

From Kevin Grittner
Subject pgsql: Use static inline function for BufferGetPage()
Date
Msg-id E1apjnI-0000cF-HS@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Use static inline function for BufferGetPage()

I was initially concerned that the some of the hundreds of
references to BufferGetPage() where the literal
BGP_NO_SNAPSHOT_TEST were passed might not optimize as well as a
macro, leading to some hard-to-find performance regressions in
corner cases.  Inspection of disassembled code has shown identical
code at all inspected locations, and the size difference doesn't
amount to even one byte per such call.  So make it readable.

Per gripes from Álvaro Herrera and Tom Lane

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/a6f6b78196a701702ec4ff6df56c346bdcf9abd2

Modified Files
--------------
src/backend/storage/buffer/bufmgr.c |  4 +--
src/include/storage/bufmgr.h        | 50 ++++++++++++++++++-------------------
2 files changed, 25 insertions(+), 29 deletions(-)


pgsql-committers by date:

Previous
From: Kevin Grittner
Date:
Subject: pgsql: Make oldSnapshotControl a pointer to a volatile structure
Next
From: Tom Lane
Date:
Subject: pgsql: Fix freshly-introduced PL/Python portability bug.