diff options
| author | Robert Haas <rhaas@postgresql.org> | 2025-06-06 08:18:26 -0400 |
|---|---|---|
| committer | Robert Haas <rhaas@postgresql.org> | 2025-06-06 08:18:26 -0400 |
| commit | e4b8f925a929fd57570097483a0fcf82dfc141da (patch) | |
| tree | 0323332c256fffd899882c1ada8ea3e227dd9ec5 | |
| parent | 767d38e2c0960658251c356fedbf15c52ecb8637 (diff) | |
pg_prewarm: Allow autoprewarm to use more than 1GB to dump blocks.
Reported-by: Daria Shanina <vilensipkdm@gmail.com>
Author: Daria Shanina <vilensipkdm@gmail.com>
Author: Robert Haas <robertmhaas@gmail.com>
Backpatch-through: 13
| -rw-r--r-- | contrib/pg_prewarm/autoprewarm.c | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/contrib/pg_prewarm/autoprewarm.c b/contrib/pg_prewarm/autoprewarm.c index d061731706a..bc540e734b5 100644 --- a/contrib/pg_prewarm/autoprewarm.c +++ b/contrib/pg_prewarm/autoprewarm.c @@ -597,8 +597,15 @@ apw_dump_now(bool is_bgworker, bool dump_unlogged) return 0; } - block_info_array = - (BlockInfoRecord *) palloc(sizeof(BlockInfoRecord) * NBuffers); + /* + * With sufficiently large shared_buffers, allocation will exceed 1GB, so + * allow for a huge allocation to prevent outright failure. + * + * (In the future, it might be a good idea to redesign this to use a more + * memory-efficient data structure.) + */ + block_info_array = (BlockInfoRecord *) + palloc_extended((sizeof(BlockInfoRecord) * NBuffers), MCXT_ALLOC_HUGE); for (num_blocks = 0, i = 0; i < NBuffers; i++) { |