]> granicus.if.org Git - postgresql/commit
Allow repalloc() to give back space when a large chunk is downsized.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 3 Oct 2019 17:56:26 +0000 (13:56 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 3 Oct 2019 17:56:26 +0000 (13:56 -0400)
commit9ad1b572d8b7539d9c94abf66dbe0f70d002c9e3
tree58e8f6937a918e89bde97859fcb065ad142befb2
parentede0ab6ccce498018e0b2112bcc2748a9799df40
Allow repalloc() to give back space when a large chunk is downsized.

Up to now, if you resized a large (>8K) palloc chunk down to a smaller
size, aset.c made no attempt to return any space to the malloc pool.
That's unpleasant if a really large allocation is resized to a
significantly smaller size.  I think no such cases existed when this
code was designed, and I'm not sure whether they're common even yet,
but an upcoming fix to encoding conversion will certainly create such
cases.  Therefore, fix AllocSetRealloc so that it gives realloc()
a chance to do something with the block.  This doesn't noticeably
increase complexity, we mostly just have to change the order in which
the cases are considered.

Back-patch to all supported branches.

Discussion: https://postgr.es/m/20190816181418.GA898@alvherre.pgsql
Discussion: https://postgr.es/m/3614.1569359690@sss.pgh.pa.us
src/backend/utils/mmgr/aset.c