Permit super-MaxAllocSize allocations with MemoryContextAllocHuge().
authorNoah Misch <noah@leadboat.com>
Thu, 27 Jun 2013 18:53:57 +0000 (14:53 -0400)
committerNoah Misch <noah@leadboat.com>
Thu, 27 Jun 2013 18:53:57 +0000 (14:53 -0400)
The MaxAllocSize guard is convenient for most callers, because it
reduces the need for careful attention to overflow, data type selection,
and the SET_VARSIZE() limit.  A handful of callers are happy to navigate
those hazards in exchange for the ability to allocate a larger chunk.
Introduce MemoryContextAllocHuge() and repalloc_huge().  Use this in
tuplesort.c and tuplestore.c, enabling internal sorts of up to INT_MAX
tuples, a factor-of-48 increase.  In particular, B-tree index builds can
now benefit from much-larger maintenance_work_mem settings.

Reviewed by Stephen Frost, Simon Riggs and Jeff Janes.

src/backend/utils/mmgr/aset.c
src/backend/utils/mmgr/mcxt.c
src/backend/utils/sort/tuplesort.c
src/backend/utils/sort/tuplestore.c
src/include/utils/memutils.h
src/include/utils/palloc.h
src/include/utils/tuplesort.h

index ab93620ae269ad7a9edfa23d3021c11b9c5ac08c..ff04a38cda2abcb13a4c600100be805745a141f5 100644 (file)
@@ -458,6 +458,7 @@ AllocSetContextCreate(MemoryContext parent,
    maxBlockSize = MAXALIGN(maxBlockSize);
    if (maxBlockSize < initBlockSize)
        maxBlockSize = initBlockSize;
+   Assert(AllocHugeSizeIsValid(maxBlockSize)); /* must be safe to double */
    context->initBlockSize = initBlockSize;
    context->maxBlockSize = maxBlockSize;
    context->nextBlockSize = initBlockSize;
@@ -643,6 +644,10 @@ AllocSetDelete(MemoryContext context)
  * AllocSetAlloc
  *     Returns pointer to allocated memory of given size; memory is added
  *     to the set.
+ *
+ * No request may exceed:
+ *     MAXALIGN_DOWN(SIZE_MAX) - ALLOC_BLOCKHDRSZ - ALLOC_CHUNKHDRSZ
+ * All callers use a much-lower limit.
  */
 static void *
 AllocSetAlloc(MemoryContext context, Size size)
index 46961e9ee990ee4041acd133d73ec98a74d94a18..9574fd3c7a3886fc6900d599fa64ca2eaffce71f 100644 (file)
@@ -455,14 +455,7 @@ MemoryContextContains(MemoryContext context, void *pointer)
    header = (StandardChunkHeader *)
        ((char *) pointer - STANDARDCHUNKHEADERSIZE);
 
-   /*
-    * If the context link doesn't match then we certainly have a non-member
-    * chunk.  Also check for a reasonable-looking size as extra guard against
-    * being fooled by bogus pointers.
-    */
-   if (header->context == context && AllocSizeIsValid(header->size))
-       return true;
-   return false;
+   return header->context == context;
 }
 
 /*--------------------
@@ -757,6 +750,71 @@ repalloc(void *pointer, Size size)
    return ret;
 }
 
+/*
+ * MemoryContextAllocHuge
+ *     Allocate (possibly-expansive) space within the specified context.
+ *
+ * See considerations in comment at MaxAllocHugeSize.
+ */
+void *
+MemoryContextAllocHuge(MemoryContext context, Size size)
+{
+   void       *ret;
+
+   AssertArg(MemoryContextIsValid(context));
+
+   if (!AllocHugeSizeIsValid(size))
+       elog(ERROR, "invalid memory alloc request size %lu",
+            (unsigned long) size);
+
+   context->isReset = false;
+
+   ret = (*context->methods->alloc) (context, size);
+   VALGRIND_MEMPOOL_ALLOC(context, ret, size);
+
+   return ret;
+}
+
+/*
+ * repalloc_huge
+ *     Adjust the size of a previously allocated chunk, permitting a large
+ *     value.  The previous allocation need not have been "huge".
+ */
+void *
+repalloc_huge(void *pointer, Size size)
+{
+   MemoryContext context;
+   void       *ret;
+
+   if (!AllocHugeSizeIsValid(size))
+       elog(ERROR, "invalid memory alloc request size %lu",
+            (unsigned long) size);
+
+   /*
+    * Try to detect bogus pointers handed to us, poorly though we can.
+    * Presumably, a pointer that isn't MAXALIGNED isn't pointing at an
+    * allocated chunk.
+    */
+   Assert(pointer != NULL);
+   Assert(pointer == (void *) MAXALIGN(pointer));
+
+   /*
+    * OK, it's probably safe to look at the chunk header.
+    */
+   context = ((StandardChunkHeader *)
+              ((char *) pointer - STANDARDCHUNKHEADERSIZE))->context;
+
+   AssertArg(MemoryContextIsValid(context));
+
+   /* isReset must be false already */
+   Assert(!context->isReset);
+
+   ret = (*context->methods->realloc) (context, pointer, size);
+   VALGRIND_MEMPOOL_CHANGE(context, pointer, ret, size);
+
+   return ret;
+}
+
 /*
  * MemoryContextStrdup
  *     Like strdup(), but allocate from the specified context
index d876369c6bb6145b6cbaf7b859f07cad3c3cc488..efc076064006b2f6d37128961ab7282c8f8e8c0c 100644 (file)
@@ -211,8 +211,8 @@ struct Tuplesortstate
                                 * tuples to return? */
    bool        boundUsed;      /* true if we made use of a bounded heap */
    int         bound;          /* if bounded, the maximum number of tuples */
-   long        availMem;       /* remaining memory available, in bytes */
-   long        allowedMem;     /* total memory allowed, in bytes */
+   Size        availMem;       /* remaining memory available, in bytes */
+   Size        allowedMem;     /* total memory allowed, in bytes */
    int         maxTapes;       /* number of tapes (Knuth's T) */
    int         tapeRange;      /* maxTapes-1 (Knuth's P) */
    MemoryContext sortcontext;  /* memory context holding all sort data */
@@ -308,7 +308,7 @@ struct Tuplesortstate
    int        *mergenext;      /* first preread tuple for each source */
    int        *mergelast;      /* last preread tuple for each source */
    int        *mergeavailslots;    /* slots left for prereading each tape */
-   long       *mergeavailmem;  /* availMem for prereading each tape */
+   Size       *mergeavailmem;  /* availMem for prereading each tape */
    int         mergefreelist;  /* head of freelist of recycled slots */
    int         mergefirstfree; /* first slot never used in this merge */
 
@@ -961,25 +961,26 @@ tuplesort_end(Tuplesortstate *state)
 }
 
 /*
- * Grow the memtuples[] array, if possible within our memory constraint.
- * Return TRUE if we were able to enlarge the array, FALSE if not.
+ * Grow the memtuples[] array, if possible within our memory constraint.  We
+ * must not exceed INT_MAX tuples in memory or the caller-provided memory
+ * limit.  Return TRUE if we were able to enlarge the array, FALSE if not.
  *
- * Normally, at each increment we double the size of the array.  When we no
- * longer have enough memory to do that, we attempt one last, smaller increase
- * (and then clear the growmemtuples flag so we don't try any more).  That
- * allows us to use allowedMem as fully as possible; sticking to the pure
- * doubling rule could result in almost half of allowedMem going unused.
- * Because availMem moves around with tuple addition/removal, we need some
- * rule to prevent making repeated small increases in memtupsize, which would
- * just be useless thrashing.  The growmemtuples flag accomplishes that and
- * also prevents useless recalculations in this function.
+ * Normally, at each increment we double the size of the array.  When doing
+ * that would exceed a limit, we attempt one last, smaller increase (and then
+ * clear the growmemtuples flag so we don't try any more).  That allows us to
+ * use memory as fully as permitted; sticking to the pure doubling rule could
+ * result in almost half going unused.  Because availMem moves around with
+ * tuple addition/removal, we need some rule to prevent making repeated small
+ * increases in memtupsize, which would just be useless thrashing.  The
+ * growmemtuples flag accomplishes that and also prevents useless
+ * recalculations in this function.
  */
 static bool
 grow_memtuples(Tuplesortstate *state)
 {
    int         newmemtupsize;
    int         memtupsize = state->memtupsize;
-   long        memNowUsed = state->allowedMem - state->availMem;
+   Size        memNowUsed = state->allowedMem - state->availMem;
 
    /* Forget it if we've already maxed out memtuples, per comment above */
    if (!state->growmemtuples)
@@ -989,14 +990,16 @@ grow_memtuples(Tuplesortstate *state)
    if (memNowUsed <= state->availMem)
    {
        /*
-        * It is surely safe to double memtupsize if we've used no more than
-        * half of allowedMem.
-        *
-        * Note: it might seem that we need to worry about memtupsize * 2
-        * overflowing an int, but the MaxAllocSize clamp applied below
-        * ensures the existing memtupsize can't be large enough for that.
+        * We've used no more than half of allowedMem; double our usage,
+        * clamping at INT_MAX.
         */
-       newmemtupsize = memtupsize * 2;
+       if (memtupsize < INT_MAX / 2)
+           newmemtupsize = memtupsize * 2;
+       else
+       {
+           newmemtupsize = INT_MAX;
+           state->growmemtuples = false;
+       }
    }
    else
    {
@@ -1012,7 +1015,8 @@ grow_memtuples(Tuplesortstate *state)
         * we've already seen, and thus we can extrapolate from the space
         * consumption so far to estimate an appropriate new size for the
         * memtuples array.  The optimal value might be higher or lower than
-        * this estimate, but it's hard to know that in advance.
+        * this estimate, but it's hard to know that in advance.  We again
+        * clamp at INT_MAX tuples.
         *
         * This calculation is safe against enlarging the array so much that
         * LACKMEM becomes true, because the memory currently used includes
@@ -1020,16 +1024,18 @@ grow_memtuples(Tuplesortstate *state)
         * new array elements even if no other memory were currently used.
         *
         * We do the arithmetic in float8, because otherwise the product of
-        * memtupsize and allowedMem could overflow.  (A little algebra shows
-        * that grow_ratio must be less than 2 here, so we are not risking
-        * integer overflow this way.)  Any inaccuracy in the result should be
-        * insignificant; but even if we computed a completely insane result,
-        * the checks below will prevent anything really bad from happening.
+        * memtupsize and allowedMem could overflow.  Any inaccuracy in the
+        * result should be insignificant; but even if we computed a
+        * completely insane result, the checks below will prevent anything
+        * really bad from happening.
         */
        double      grow_ratio;
 
        grow_ratio = (double) state->allowedMem / (double) memNowUsed;
-       newmemtupsize = (int) (memtupsize * grow_ratio);
+       if (memtupsize * grow_ratio < INT_MAX)
+           newmemtupsize = (int) (memtupsize * grow_ratio);
+       else
+           newmemtupsize = INT_MAX;
 
        /* We won't make any further enlargement attempts */
        state->growmemtuples = false;
@@ -1040,12 +1046,13 @@ grow_memtuples(Tuplesortstate *state)
        goto noalloc;
 
    /*
-    * On a 64-bit machine, allowedMem could be more than MaxAllocSize.  Clamp
-    * to ensure our request won't be rejected by palloc.
+    * On a 32-bit machine, allowedMem could exceed MaxAllocHugeSize.  Clamp
+    * to ensure our request won't be rejected.  Note that we can easily
+    * exhaust address space before facing this outcome.
     */
-   if ((Size) newmemtupsize >= MaxAllocSize / sizeof(SortTuple))
+   if ((Size) newmemtupsize >= MaxAllocHugeSize / sizeof(SortTuple))
    {
-       newmemtupsize = (int) (MaxAllocSize / sizeof(SortTuple));
+       newmemtupsize = (int) (MaxAllocHugeSize / sizeof(SortTuple));
        state->growmemtuples = false;   /* can't grow any more */
    }
 
@@ -1060,15 +1067,15 @@ grow_memtuples(Tuplesortstate *state)
     * palloc would be treating both old and new arrays as separate chunks.
     * But we'll check LACKMEM explicitly below just in case.)
     */
-   if (state->availMem < (long) ((newmemtupsize - memtupsize) * sizeof(SortTuple)))
+   if (state->availMem < (Size) ((newmemtupsize - memtupsize) * sizeof(SortTuple)))
        goto noalloc;
 
    /* OK, do it */
    FREEMEM(state, GetMemoryChunkSpace(state->memtuples));
    state->memtupsize = newmemtupsize;
    state->memtuples = (SortTuple *)
-       repalloc(state->memtuples,
-                state->memtupsize * sizeof(SortTuple));
+       repalloc_huge(state->memtuples,
+                     state->memtupsize * sizeof(SortTuple));
    USEMEM(state, GetMemoryChunkSpace(state->memtuples));
    if (LACKMEM(state))
        elog(ERROR, "unexpected out-of-memory situation during sort");
@@ -1715,7 +1722,7 @@ tuplesort_getdatum(Tuplesortstate *state, bool forward,
  * This is exported for use by the planner.  allowedMem is in bytes.
  */
 int
-tuplesort_merge_order(long allowedMem)
+tuplesort_merge_order(Size allowedMem)
 {
    int         mOrder;
 
@@ -1749,7 +1756,7 @@ inittapes(Tuplesortstate *state)
    int         maxTapes,
                ntuples,
                j;
-   long        tapeSpace;
+   Size        tapeSpace;
 
    /* Compute number of tapes to use: merge order plus 1 */
    maxTapes = tuplesort_merge_order(state->allowedMem) + 1;
@@ -1798,7 +1805,7 @@ inittapes(Tuplesortstate *state)
    state->mergenext = (int *) palloc0(maxTapes * sizeof(int));
    state->mergelast = (int *) palloc0(maxTapes * sizeof(int));
    state->mergeavailslots = (int *) palloc0(maxTapes * sizeof(int));
-   state->mergeavailmem = (long *) palloc0(maxTapes * sizeof(long));
+   state->mergeavailmem = (Size *) palloc0(maxTapes * sizeof(Size));
    state->tp_fib = (int *) palloc0(maxTapes * sizeof(int));
    state->tp_runs = (int *) palloc0(maxTapes * sizeof(int));
    state->tp_dummy = (int *) palloc0(maxTapes * sizeof(int));
@@ -2026,7 +2033,7 @@ mergeonerun(Tuplesortstate *state)
    int         srcTape;
    int         tupIndex;
    SortTuple  *tup;
-   long        priorAvail,
+   Size        priorAvail,
                spaceFreed;
 
    /*
@@ -2100,7 +2107,7 @@ beginmerge(Tuplesortstate *state)
    int         tapenum;
    int         srcTape;
    int         slotsPerTape;
-   long        spacePerTape;
+   Size        spacePerTape;
 
    /* Heap should be empty here */
    Assert(state->memtupcount == 0);
@@ -2221,7 +2228,7 @@ mergeprereadone(Tuplesortstate *state, int srcTape)
    unsigned int tuplen;
    SortTuple   stup;
    int         tupIndex;
-   long        priorAvail,
+   Size        priorAvail,
                spaceUsed;
 
    if (!state->mergeactive[srcTape])
index ea9bc04823dd070fb383cfb27544d4a1c6afb267..ce1d47611c23c4806105ef02e94206f09e4d661b 100644 (file)
@@ -104,8 +104,8 @@ struct Tuplestorestate
    bool        backward;       /* store extra length words in file? */
    bool        interXact;      /* keep open through transactions? */
    bool        truncated;      /* tuplestore_trim has removed tuples? */
-   long        availMem;       /* remaining memory available, in bytes */
-   long        allowedMem;     /* total memory allowed, in bytes */
+   Size        availMem;       /* remaining memory available, in bytes */
+   Size        allowedMem;     /* total memory allowed, in bytes */
    BufFile    *myfile;         /* underlying file, or NULL if none */
    MemoryContext context;      /* memory context for holding tuples */
    ResourceOwner resowner;     /* resowner for holding temp files */
@@ -531,25 +531,26 @@ tuplestore_ateof(Tuplestorestate *state)
 }
 
 /*
- * Grow the memtuples[] array, if possible within our memory constraint.
- * Return TRUE if we were able to enlarge the array, FALSE if not.
+ * Grow the memtuples[] array, if possible within our memory constraint.  We
+ * must not exceed INT_MAX tuples in memory or the caller-provided memory
+ * limit.  Return TRUE if we were able to enlarge the array, FALSE if not.
  *
- * Normally, at each increment we double the size of the array.  When we no
- * longer have enough memory to do that, we attempt one last, smaller increase
- * (and then clear the growmemtuples flag so we don't try any more).  That
- * allows us to use allowedMem as fully as possible; sticking to the pure
- * doubling rule could result in almost half of allowedMem going unused.
- * Because availMem moves around with tuple addition/removal, we need some
- * rule to prevent making repeated small increases in memtupsize, which would
- * just be useless thrashing.  The growmemtuples flag accomplishes that and
- * also prevents useless recalculations in this function.
+ * Normally, at each increment we double the size of the array.  When doing
+ * that would exceed a limit, we attempt one last, smaller increase (and then
+ * clear the growmemtuples flag so we don't try any more).  That allows us to
+ * use memory as fully as permitted; sticking to the pure doubling rule could
+ * result in almost half going unused.  Because availMem moves around with
+ * tuple addition/removal, we need some rule to prevent making repeated small
+ * increases in memtupsize, which would just be useless thrashing.  The
+ * growmemtuples flag accomplishes that and also prevents useless
+ * recalculations in this function.
  */
 static bool
 grow_memtuples(Tuplestorestate *state)
 {
    int         newmemtupsize;
    int         memtupsize = state->memtupsize;
-   long        memNowUsed = state->allowedMem - state->availMem;
+   Size        memNowUsed = state->allowedMem - state->availMem;
 
    /* Forget it if we've already maxed out memtuples, per comment above */
    if (!state->growmemtuples)
@@ -559,14 +560,16 @@ grow_memtuples(Tuplestorestate *state)
    if (memNowUsed <= state->availMem)
    {
        /*
-        * It is surely safe to double memtupsize if we've used no more than
-        * half of allowedMem.
-        *
-        * Note: it might seem that we need to worry about memtupsize * 2
-        * overflowing an int, but the MaxAllocSize clamp applied below
-        * ensures the existing memtupsize can't be large enough for that.
+        * We've used no more than half of allowedMem; double our usage,
+        * clamping at INT_MAX.
         */
-       newmemtupsize = memtupsize * 2;
+       if (memtupsize < INT_MAX / 2)
+           newmemtupsize = memtupsize * 2;
+       else
+       {
+           newmemtupsize = INT_MAX;
+           state->growmemtuples = false;
+       }
    }
    else
    {
@@ -582,7 +585,8 @@ grow_memtuples(Tuplestorestate *state)
         * we've already seen, and thus we can extrapolate from the space
         * consumption so far to estimate an appropriate new size for the
         * memtuples array.  The optimal value might be higher or lower than
-        * this estimate, but it's hard to know that in advance.
+        * this estimate, but it's hard to know that in advance.  We again
+        * clamp at INT_MAX tuples.
         *
         * This calculation is safe against enlarging the array so much that
         * LACKMEM becomes true, because the memory currently used includes
@@ -590,16 +594,18 @@ grow_memtuples(Tuplestorestate *state)
         * new array elements even if no other memory were currently used.
         *
         * We do the arithmetic in float8, because otherwise the product of
-        * memtupsize and allowedMem could overflow.  (A little algebra shows
-        * that grow_ratio must be less than 2 here, so we are not risking
-        * integer overflow this way.)  Any inaccuracy in the result should be
-        * insignificant; but even if we computed a completely insane result,
-        * the checks below will prevent anything really bad from happening.
+        * memtupsize and allowedMem could overflow.  Any inaccuracy in the
+        * result should be insignificant; but even if we computed a
+        * completely insane result, the checks below will prevent anything
+        * really bad from happening.
         */
        double      grow_ratio;
 
        grow_ratio = (double) state->allowedMem / (double) memNowUsed;
-       newmemtupsize = (int) (memtupsize * grow_ratio);
+       if (memtupsize * grow_ratio < INT_MAX)
+           newmemtupsize = (int) (memtupsize * grow_ratio);
+       else
+           newmemtupsize = INT_MAX;
 
        /* We won't make any further enlargement attempts */
        state->growmemtuples = false;
@@ -610,12 +616,13 @@ grow_memtuples(Tuplestorestate *state)
        goto noalloc;
 
    /*
-    * On a 64-bit machine, allowedMem could be more than MaxAllocSize.  Clamp
-    * to ensure our request won't be rejected by palloc.
+    * On a 32-bit machine, allowedMem could exceed MaxAllocHugeSize.  Clamp
+    * to ensure our request won't be rejected.  Note that we can easily
+    * exhaust address space before facing this outcome.
     */
-   if ((Size) newmemtupsize >= MaxAllocSize / sizeof(void *))
+   if ((Size) newmemtupsize >= MaxAllocHugeSize / sizeof(void *))
    {
-       newmemtupsize = (int) (MaxAllocSize / sizeof(void *));
+       newmemtupsize = (int) (MaxAllocHugeSize / sizeof(void *));
        state->growmemtuples = false;   /* can't grow any more */
    }
 
@@ -630,15 +637,15 @@ grow_memtuples(Tuplestorestate *state)
     * palloc would be treating both old and new arrays as separate chunks.
     * But we'll check LACKMEM explicitly below just in case.)
     */
-   if (state->availMem < (long) ((newmemtupsize - memtupsize) * sizeof(void *)))
+   if (state->availMem < (Size) ((newmemtupsize - memtupsize) * sizeof(void *)))
        goto noalloc;
 
    /* OK, do it */
    FREEMEM(state, GetMemoryChunkSpace(state->memtuples));
    state->memtupsize = newmemtupsize;
    state->memtuples = (void **)
-       repalloc(state->memtuples,
-                state->memtupsize * sizeof(void *));
+       repalloc_huge(state->memtuples,
+                     state->memtupsize * sizeof(void *));
    USEMEM(state, GetMemoryChunkSpace(state->memtuples));
    if (LACKMEM(state))
        elog(ERROR, "unexpected out-of-memory situation during sort");
index 08851bb2c4b3c1b13d3b4f6bc487598b2bc1820d..9acd825e49943e58b54fc49448c07e48e140a463 100644 (file)
 
 
 /*
- * MaxAllocSize
- *     Quasi-arbitrary limit on size of allocations.
+ * MaxAllocSize, MaxAllocHugeSize
+ *     Quasi-arbitrary limits on size of allocations.
  *
  * Note:
- *     There is no guarantee that allocations smaller than MaxAllocSize
- *     will succeed.  Allocation requests larger than MaxAllocSize will
- *     be summarily denied.
+ *     There is no guarantee that smaller allocations will succeed, but
+ *     larger requests will be summarily denied.
  *
- * XXX This is deliberately chosen to correspond to the limiting size
- * of varlena objects under TOAST. See VARSIZE_4B() and related macros
- * in postgres.h.  Many datatypes assume that any allocatable size can
- * be represented in a varlena header.
- *
- * XXX Also, various places in aset.c assume they can compute twice an
- * allocation's size without overflow, so beware of raising this.
+ * palloc() enforces MaxAllocSize, chosen to correspond to the limiting size
+ * of varlena objects under TOAST.  See VARSIZE_4B() and related macros in
+ * postgres.h.  Many datatypes assume that any allocatable size can be
+ * represented in a varlena header.  This limit also permits a caller to use
+ * an "int" variable for an index into or length of an allocation.  Callers
+ * careful to avoid these hazards can access the higher limit with
+ * MemoryContextAllocHuge().  Both limits permit code to assume that it may
+ * compute twice an allocation's size without overflow.
  */
 #define MaxAllocSize   ((Size) 0x3fffffff)     /* 1 gigabyte - 1 */
 
 #define AllocSizeIsValid(size) ((Size) (size) <= MaxAllocSize)
 
+#define MaxAllocHugeSize   ((Size) -1 >> 1)    /* SIZE_MAX / 2 */
+
+#define AllocHugeSizeIsValid(size) ((Size) (size) <= MaxAllocHugeSize)
+
 /*
  * All chunks allocated by any memory context manager are required to be
  * preceded by a StandardChunkHeader at a spacing of STANDARDCHUNKHEADERSIZE.
index 420bde7ee4a1b75afddc9cdd42b9e6ebdffd2e20..01e7db5d4f9cdeccc19af5e5c223409f5024c7cf 100644 (file)
@@ -51,6 +51,10 @@ extern void *MemoryContextAlloc(MemoryContext context, Size size);
 extern void *MemoryContextAllocZero(MemoryContext context, Size size);
 extern void *MemoryContextAllocZeroAligned(MemoryContext context, Size size);
 
+/* Higher-limit allocators. */
+extern void *MemoryContextAllocHuge(MemoryContext context, Size size);
+extern void *repalloc_huge(void *pointer, Size size);
+
 /*
  * The result of palloc() is always word-aligned, so we can skip testing
  * alignment of the pointer when deciding which MemSet variant to use.
index e69aafcd903679cc99d5aada435be5fd43a06788..d5b391367ef3b7299cd559be2c9bffc721482e9b 100644 (file)
@@ -106,7 +106,7 @@ extern void tuplesort_get_stats(Tuplesortstate *state,
                    const char **spaceType,
                    long *spaceUsed);
 
-extern int tuplesort_merge_order(long allowedMem);
+extern int tuplesort_merge_order(Size allowedMem);
 
 /*
  * These routines may only be called if randomAccess was specified 'true'.