Revert "Add missing padding from MemoryChunk struct"
authorDavid Rowley <drowley@postgresql.org>
Tue, 30 Aug 2022 15:06:31 +0000 (03:06 +1200)
committerDavid Rowley <drowley@postgresql.org>
Tue, 30 Aug 2022 15:06:31 +0000 (03:06 +1200)
This reverts commit df0f4feef.  It turns out the problem which was causing
the 32-bit ARM and PPC animals to fail was due to a MAXALIGN problem in
slab.c.  This was fixed by d5ee4db0e.  The padding that was added in
df0f4feef would only do anything on machines where uint64 was not aligned
to 8 bytes.  The 32-bit machines which were failing are not in that
category, so revert this commit.

Discussion: https://postgr.es/m/3209100.1661787561@sss.pgh.pa.us

src/include/utils/memutils_memorychunk.h

index 1e27d084cd461c06d8aabff38643895ab8f53f4d..685c177b681ad2c3de551ace9bee999a244160d3 100644 (file)
@@ -111,14 +111,6 @@ typedef struct MemoryChunk
 {
 #ifdef MEMORY_CONTEXT_CHECKING
    Size        requested_size;
-#define MEMORYCHUNK_RAWSIZE        (SIZEOF_SIZE_T + 8)
-#else
-#define MEMORYCHUNK_RAWSIZE        8
-#endif                         /* MEMORY_CONTEXT_CHECKING */
-
-   /* ensure proper alignment by adding padding if needed */
-#if (MEMORYCHUNK_RAWSIZE % MAXIMUM_ALIGNOF) != 0
-   char        padding[MAXIMUM_ALIGNOF - MEMORYCHUNK_RAWSIZE % MAXIMUM_ALIGNOF];
 #endif
 
    /* bitfield for storing details about the chunk */