From 376c216138c75e161d39767650ea30536f23b482 Mon Sep 17 00:00:00 2001 From: Heikki Linnakangas Date: Fri, 26 Jan 2024 01:04:58 +0200 Subject: [PATCH] Update comment, generation mem contexts have a "keeper" block The keeper block was introduced in commit 1b0d9aa4f7, but it forgot to update this comment. --- src/backend/utils/mmgr/generation.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/backend/utils/mmgr/generation.c b/src/backend/utils/mmgr/generation.c index 65fd00d0137..f9016a7ed72 100644 --- a/src/backend/utils/mmgr/generation.c +++ b/src/backend/utils/mmgr/generation.c @@ -271,8 +271,10 @@ GenerationContextCreate(MemoryContext parent, * GenerationReset * Frees all memory which is allocated in the given set. * - * The code simply frees all the blocks in the context - we don't keep any - * keeper blocks or anything like that. + * The initial "keeper" block (which shares a malloc chunk with the context + * header) is not given back to the operating system though. In this way, we + * don't thrash malloc() when a context is repeatedly reset after small + * allocations. */ void GenerationReset(MemoryContext context) -- 2.39.5