]> granicus.if.org Git - postgresql/commit
Reduce size of common allocation header.
authorAndres Freund <andres@anarazel.de>
Tue, 28 Feb 2017 07:32:22 +0000 (23:32 -0800)
committerAndres Freund <andres@anarazel.de>
Wed, 1 Mar 2017 03:42:44 +0000 (19:42 -0800)
commit7e3aa03b418d604d33040ed8fb866857dae82a02
treedd2f01c03693a10f1d1b18cab8c7f2555baf040f
parenteb75f4fced77e108393f18425ec3f7aba2e70a9d
Reduce size of common allocation header.

The new slab allocator needs different per-allocation information than
the classical aset.c.  The definition in 58b25e981 wasn't sufficiently
careful on 32 platforms with 8 byte alignment, leading to buildfarm
failures.  That's not entirely easy to fix by just adjusting the
definition.

As slab.c doesn't actually need the size part(s) of the common header,
all chunks are equally sized after all, it seems better to instead
reduce the header to the part needed by all allocators, namely which
context an allocation belongs to. That has the advantage of reducing
the overhead of slab allocations, and also allows for more flexibility
in future allocators.

To avoid spreading the logic about accessing a chunk's context around,
centralize it in GetMemoryChunkContext(), which allows to delete a
good number of lines.

A followup commit will revise the mmgr/README portion about
StandardChunkHeader, and more.

Author: Andres Freund
Discussion: https://postgr.es/m/20170228074420.aazv4iw6k562mnxg@alap3.anarazel.de
src/backend/utils/mmgr/aset.c
src/backend/utils/mmgr/mcxt.c
src/backend/utils/mmgr/slab.c
src/include/utils/memutils.h
src/tools/pgindent/typedefs.list