| From: | Robert Haas <rhaas(at)postgresql(dot)org> |
|---|---|
| To: | pgsql-committers(at)postgresql(dot)org |
| Subject: | pgsql: Fix assorted defects in 09adc9a8c09c9640de05c7023b27fb83c761e91c |
| Date: | 2016-04-21 17:28:40 |
| Message-ID: | E1atIPA-0005MA-3Y@gemulon.postgresql.org |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | 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(-)
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2016-04-21 17:29:15 | Re: [COMMITTERS] pgsql: Move each SLRU's lwlocks to a separate tranche. |
| Previous Message | Kevin Grittner | 2016-04-21 16:58:20 | Re: [COMMITTERS] pgsql: Inline initial comparisons in TestForOldSnapshot() |