Re: pgsql: Preallocate some DSM space at startup.

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Thomas Munro <tmunro(at)postgresql(dot)org>
Cc: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Preallocate some DSM space at startup.
Date: 2020-07-31 06:26:26
Message-ID: 20200731062626.GD3317@paquier.xyz
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Hi Thomas,

On Fri, Jul 31, 2020 at 05:54:45AM +0000, Thomas Munro wrote:
> Preallocate some DSM space at startup.
>
> Create an optional region in the main shared memory segment that can be
> used to acquire and release "fast" DSM segments, and can benefit from
> huge pages allocated at cluster startup time, if configured. Fall back
> to the existing mechanisms when that space is full. The size is
> controlled by a new GUC min_dynamic_shared_memory, defaulting to 0.
>
> Main region DSM segments initially contain whatever garbage the memory
> held last time they were used, rather than zeroes. That change revealed
> that DSA areas failed to initialize themselves correctly in memory that
> wasn't zeroed first, so fix that problem.

longfin is complaning on this one:
https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=longfin&dt=2020-07-31%2005%3A59%3A08
guc.c:2241:38: error: implicit conversion from 'unsigned long' to
'int' changes value from 17592186044415 to -1
[-Werror,-Wconstant-conversion]
0, 0, Min(INT_MAX, SIZE_MAX / 1024 / 1024),
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~
../../../../src/include/c.h:927:40: note: expanded from macro 'Min'

#define Min(x, y) ((x) < (y) ? (x) : (y))
^
1 error generated.
--
Michael

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Thomas Munro 2020-07-31 07:10:34 pgsql: Fix compiler warning from Clang.
Previous Message Thomas Munro 2020-07-31 05:54:45 pgsql: Preallocate some DSM space at startup.