]> granicus.if.org Git - postgresql/commit
Ensure sufficient alignment for ParallelTableScanDescData in BTShared.
authorAndres Freund <andres@anarazel.de>
Mon, 11 Mar 2019 21:26:43 +0000 (14:26 -0700)
committerAndres Freund <andres@anarazel.de>
Mon, 11 Mar 2019 21:26:43 +0000 (14:26 -0700)
commit8cacea7a725103f1a037a5ee06112ebe31051c66
tree6b5e8872494979b1fb6daf202c3a661972a55a62
parentc2fe139c201c48f1133e9fbea2dd99b8efe2fadd
Ensure sufficient alignment for ParallelTableScanDescData in BTShared.

Previously ParallelTableScanDescData was just a member in BTShared,
but after c2fe139c2 that doesn't guarantee sufficient alignment as
specific AMs might (are likely to) need atomic variables in the
struct.

One might think that MAXALIGNing would be sufficient, but as a
comment in shm_toc_allocate() explains, that's not enough. For now,
copy the hack described there.

For parallel sequential scans no such change is needed, as its
allocations go through shm_toc_allocate().

An alternative approach would have been to allocate the parallel scan
descriptor in a separate TOC entry, but there seems little benefit in
doing so.

Per buildfarm member dromedary.

Author: Andres Freund
Discussion: https://postgr.es/m/20190311203126.ty5gbfz42gjbm6i6@alap3.anarazel.de
src/backend/access/nbtree/nbtsort.c