pgsql: Fix assorted defects in 09adc9a8c09c9640de05c7023b27fb83c761e91c - Mailing list pgsql-committers

From Robert Haas
Subject pgsql: Fix assorted defects in 09adc9a8c09c9640de05c7023b27fb83c761e91c
Date
Msg-id E1atIPA-0005MA-3Y@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix assorted defects in 09adc9a8c09c9640de05c7023b27fb83c761e91c.

That commit increased all shared memory allocations to the next higher
multiple of PG_CACHE_LINE_SIZE, but it didn't ensure that allocation
started on a cache line boundary.  It also failed to remove a couple
other pieces of now-useless code.

BUFFERALIGN() is perhaps obsolete at this point, and likely should be
removed at some point, too, but that seems like it can be left to a
future cleanup.

Mistakes all pointed out by Andres Freund.  The patch is mine, with
a few extra assertions which I adopted from his version of this fix.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/9f84280ae94b43b75dcf32aef433545335e7bb16

Modified Files
--------------
src/backend/storage/buffer/buf_init.c | 15 ++++++---------
src/backend/storage/ipc/shmem.c       | 15 +++++++++++----
2 files changed, 17 insertions(+), 13 deletions(-)


pgsql-committers by date:

Previous
From: Kevin Grittner
Date:
Subject: pgsql: Include snapmgr.h in blscan.c
Next
From: Robert Haas
Date:
Subject: pgsql: Comment improvements for ForeignPath.