On Wed, Oct 30, 2013 at 1:22 AM, Amit Kapila <amit.kapil...@gmail.com> wrote:
> On Wed, Oct 30, 2013 at 12:42 AM, Tom Lane <t...@sss.pgh.pa.us> wrote:
>> The last two buildfarm runs on frogmouth have failed in initdb,
>> like this:
>> creating directory 
>> d:/mingw-bf/root/HEAD/pgsql.2492/src/test/regress/./tmp_check/data ... ok
>> creating subdirectories ... ok
>> selecting default max_connections ... 100
>> selecting default shared_buffers ... 128MB
>> selecting dynamic shared memory implementation ... windows
>> creating configuration files ... ok
>> creating template1 database in 
>> d:/mingw-bf/root/HEAD/pgsql.2492/src/test/regress/./tmp_check/data/base/1 
>> ... FATAL:  could not open shared memory segment 
>> "Global/PostgreSQL.851401618": Not enough space
>> child process exited with exit code 1
> In windows implementation of dynamic shared memory, Size calculation
> for creating dynamic shared memory is assuming that requested size for
> creation of dynamic shared memory segment is uint64, which is changed
> by commit d2aecae, so we need to change that calculation as well.
> Please find the attached patch to fix this problem.

I find it hard to believe this is the right fix.  I know we have
similar code in win32_shmem.c, but surely if size is a 32-bit unsigned
quantity then size >> 0 is simply 0 anyway.

Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:

Reply via email to