From: | ITAGAKI Takahiro <itagaki(dot)takahiro(at)oss(dot)ntt(dot)co(dot)jp> |
---|---|
To: | pgsql-patches(at)postgresql(dot)org |
Subject: | Reduce the size of PageFreeSpaceInfo on 64bit platform |
Date: | 2007-08-10 01:32:35 |
Message-ID: | 20070810095003.754D.ITAGAKI.TAKAHIRO@oss.ntt.co.jp |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Here is a patch to reduce the size of PageFreeSpaceInfo on 64bit platform.
We will utilize maintenance_work_mem twice with the patch.
The sizeof(PageFreeSpaceInfo) is 16 bytes there because the type of 'avail'
is 'Size', that is typically 8 bytes and needs to be aligned in 8-byte bounds.
I changed the type of the field to uint32. We can store the freespace with
uint16 at smallest, but the alignment issue throws it away.
If we need to store freespace more compactly rather than calculation speed,
it might be good to use FSMPageData instead of PageFreeSpaceInfo, but I did
not change it in this patch.
Index: src/include/storage/freespace.h
===================================================================
--- src/include/storage/freespace.h (head)
+++ src/include/storage/freespace.h (fixed)
@@ -24,7 +24,7 @@
typedef struct PageFreeSpaceInfo
{
BlockNumber blkno; /* which page in relation */
- Size avail; /* space available on this page */
+ uint32 avail; /* space available on this page */
} PageFreeSpaceInfo;
----
Regards,
---
ITAGAKI Takahiro
NTT Open Source Software Center
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2007-08-10 04:45:32 | Re: Reduce the size of PageFreeSpaceInfo on 64bit platform |
Previous Message | Brendan Jurd | 2007-08-09 12:09:19 | Re: Function structure in formatting.c |