]> granicus.if.org Git - postgresql/commitdiff
Replace time_t with pg_time_t (same values, but always int64) in on-disk
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 17 Feb 2008 02:09:32 +0000 (02:09 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 17 Feb 2008 02:09:32 +0000 (02:09 +0000)
data structures and backend internal APIs.  This solves problems we've seen
recently with inconsistent layout of pg_control between machines that have
32-bit time_t and those that have already migrated to 64-bit time_t.  Also,
we can get out from under the problem that Windows' Unix-API emulation is not
consistent about the width of time_t.

There are a few remaining places where local time_t variables are used to hold
the current or recent result of time(NULL).  I didn't bother changing these
since they do not affect any cross-module APIs and surely all platforms will
have 64-bit time_t before overflow becomes an actual risk.  time_t should
be avoided for anything visible to extension modules, however.

17 files changed:
contrib/pgcrypto/internal.c
src/backend/access/transam/xlog.c
src/backend/postmaster/bgwriter.c
src/backend/postmaster/syslogger.c
src/backend/utils/adt/date.c
src/backend/utils/adt/datetime.c
src/backend/utils/adt/nabstime.c
src/backend/utils/adt/timestamp.c
src/backend/utils/init/globals.c
src/bin/pg_controldata/pg_controldata.c
src/bin/pg_resetxlog/pg_resetxlog.c
src/include/access/xlog_internal.h
src/include/catalog/pg_control.h
src/include/miscadmin.h
src/include/port/win32.h
src/include/utils/timestamp.h
src/tools/msvc/Project.pm

index 594308673bc513bd412432ae0d2663358542ff3c..56634aa32a2423f7f5885b2e8f5e00174f8a102c 100644 (file)
@@ -26,7 +26,7 @@
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
  *
- * $PostgreSQL: pgsql/contrib/pgcrypto/internal.c,v 1.27 2007/11/15 21:14:31 momjian Exp $
+ * $PostgreSQL: pgsql/contrib/pgcrypto/internal.c,v 1.28 2008/02/17 02:09:26 tgl Exp $
  */
 
 #include "postgres.h"
@@ -649,7 +649,8 @@ system_reseed(void)
                skip = 1;
        else if ((t - seed_time) > SYSTEM_RESEED_MAX)
                skip = 0;
-       else if (!check_time || (t - check_time) > SYSTEM_RESEED_CHECK_TIME)
+       else if (check_time == 0 ||
+                        (t - check_time) > SYSTEM_RESEED_CHECK_TIME)
        {
                check_time = t;
 
index d2075018d96d0547dcb3bb547897aaec67c32d4c..f61af0ab4d5d628aa2a59a680b68c94c263db603 100644 (file)
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1996-2008, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/backend/access/transam/xlog.c,v 1.292 2008/01/21 11:17:46 petere Exp $
+ * $PostgreSQL: pgsql/src/backend/access/transam/xlog.c,v 1.293 2008/02/17 02:09:27 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -271,7 +271,7 @@ typedef struct XLogCtlWrite
 {
        XLogwrtResult LogwrtResult; /* current value of LogwrtResult */
        int                     curridx;                /* cache index of next block to write */
-       time_t          lastSegSwitchTime;              /* time of last xlog segment switch */
+       pg_time_t       lastSegSwitchTime;              /* time of last xlog segment switch */
 } XLogCtlWrite;
 
 /*
@@ -1553,7 +1553,7 @@ XLogWrite(XLogwrtRqst WriteRqst, bool flexible, bool xlog_switch)
                                if (XLogArchivingActive())
                                        XLogArchiveNotifySeg(openLogId, openLogSeg);
 
-                               Write->lastSegSwitchTime = time(NULL);
+                               Write->lastSegSwitchTime = (pg_time_t) time(NULL);
 
                                /*
                                 * Signal bgwriter to start a checkpoint if we've consumed too
@@ -4217,7 +4217,7 @@ BootStrapXLOG(void)
        checkPoint.nextOid = FirstBootstrapObjectId;
        checkPoint.nextMulti = FirstMultiXactId;
        checkPoint.nextMultiOffset = 0;
-       checkPoint.time = time(NULL);
+       checkPoint.time = (pg_time_t) time(NULL);
 
        ShmemVariableCache->nextXid = checkPoint.nextXid;
        ShmemVariableCache->nextOid = checkPoint.nextOid;
@@ -4972,7 +4972,7 @@ StartupXLOG(void)
                ControlFile->checkPointCopy = checkPoint;
                if (minRecoveryLoc.xlogid != 0 || minRecoveryLoc.xrecoff != 0)
                        ControlFile->minRecoveryPoint = minRecoveryLoc;
-               ControlFile->time = time(NULL);
+               ControlFile->time = (pg_time_t) time(NULL);
                UpdateControlFile();
 
                /*
@@ -5277,7 +5277,7 @@ StartupXLOG(void)
        InRecovery = false;
 
        ControlFile->state = DB_IN_PRODUCTION;
-       ControlFile->time = time(NULL);
+       ControlFile->time = (pg_time_t) time(NULL);
        UpdateControlFile();
 
        /* start the archive_timeout timer running */
@@ -5496,10 +5496,10 @@ GetInsertRecPtr(void)
 /*
  * Get the time of the last xlog segment switch
  */
-time_t
+pg_time_t
 GetLastSegSwitchTime(void)
 {
-       time_t          result;
+       pg_time_t       result;
 
        /* Need WALWriteLock, but shared lock is sufficient */
        LWLockAcquire(WALWriteLock, LW_SHARED);
@@ -5676,7 +5676,7 @@ CreateCheckPoint(int flags)
        if (shutdown)
        {
                ControlFile->state = DB_SHUTDOWNING;
-               ControlFile->time = time(NULL);
+               ControlFile->time = (pg_time_t) time(NULL);
                UpdateControlFile();
        }
 
@@ -5690,7 +5690,7 @@ CreateCheckPoint(int flags)
        /* Begin filling in the checkpoint WAL record */
        MemSet(&checkPoint, 0, sizeof(checkPoint));
        checkPoint.ThisTimeLineID = ThisTimeLineID;
-       checkPoint.time = time(NULL);
+       checkPoint.time = (pg_time_t) time(NULL);
 
        /*
         * We must hold WALInsertLock while examining insert state to determine
@@ -5891,7 +5891,7 @@ CreateCheckPoint(int flags)
        ControlFile->prevCheckPoint = ControlFile->checkPoint;
        ControlFile->checkPoint = ProcLastRecPtr;
        ControlFile->checkPointCopy = checkPoint;
-       ControlFile->time = time(NULL);
+       ControlFile->time = (pg_time_t) time(NULL);
        UpdateControlFile();
        LWLockRelease(ControlFileLock);
 
@@ -5992,7 +5992,7 @@ RecoveryRestartPoint(const CheckPoint *checkPoint)
         * Checking true elapsed time keeps us from doing restartpoints too often
         * while rapidly scanning large amounts of WAL.
         */
-       elapsed_secs = time(NULL) - ControlFile->time;
+       elapsed_secs = (pg_time_t) time(NULL) - ControlFile->time;
        if (elapsed_secs < CheckPointTimeout / 2)
                return;
 
@@ -6028,7 +6028,7 @@ RecoveryRestartPoint(const CheckPoint *checkPoint)
        ControlFile->prevCheckPoint = ControlFile->checkPoint;
        ControlFile->checkPoint = ReadRecPtr;
        ControlFile->checkPointCopy = *checkPoint;
-       ControlFile->time = time(NULL);
+       ControlFile->time = (pg_time_t) time(NULL);
        UpdateControlFile();
 
        ereport((recoveryLogRestartpoints ? LOG : DEBUG2),
index 10586b8fad270ef7b2d3c06d0ddb19a6428f8f1b..5dce3fa6989e511b8e44863c6ffcd7d713841a8a 100644 (file)
@@ -37,7 +37,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/postmaster/bgwriter.c,v 1.48 2008/01/01 19:45:51 momjian Exp $
+ *       $PostgreSQL: pgsql/src/backend/postmaster/bgwriter.c,v 1.49 2008/02/17 02:09:27 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -163,12 +163,12 @@ static bool am_bg_writer = false;
 static bool ckpt_active = false;
 
 /* these values are valid when ckpt_active is true: */
-static time_t ckpt_start_time;
+static pg_time_t ckpt_start_time;
 static XLogRecPtr ckpt_start_recptr;
 static double ckpt_cached_elapsed;
 
-static time_t last_checkpoint_time;
-static time_t last_xlog_switch_time;
+static pg_time_t last_checkpoint_time;
+static pg_time_t last_xlog_switch_time;
 
 /* Prototypes for private functions */
 
@@ -250,7 +250,7 @@ BackgroundWriterMain(void)
        /*
         * Initialize so that first time-driven event happens at the correct time.
         */
-       last_checkpoint_time = last_xlog_switch_time = time(NULL);
+       last_checkpoint_time = last_xlog_switch_time = (pg_time_t) time(NULL);
 
        /*
         * Create a resource owner to keep track of our resources (currently only
@@ -361,7 +361,7 @@ BackgroundWriterMain(void)
        {
                bool            do_checkpoint = false;
                int                     flags = 0;
-               time_t          now;
+               pg_time_t       now;
                int                     elapsed_secs;
 
                /*
@@ -407,7 +407,7 @@ BackgroundWriterMain(void)
                 * occurs without an external request, but we set the CAUSE_TIME flag
                 * bit even if there is also an external request.
                 */
-               now = time(NULL);
+               now = (pg_time_t) time(NULL);
                elapsed_secs = now - last_checkpoint_time;
                if (elapsed_secs >= CheckPointTimeout)
                {
@@ -504,13 +504,13 @@ BackgroundWriterMain(void)
 static void
 CheckArchiveTimeout(void)
 {
-       time_t          now;
-       time_t          last_time;
+       pg_time_t       now;
+       pg_time_t       last_time;
 
        if (XLogArchiveTimeout <= 0)
                return;
 
-       now = time(NULL);
+       now = (pg_time_t) time(NULL);
 
        /* First we do a quick check using possibly-stale local state. */
        if ((int) (now - last_xlog_switch_time) < XLogArchiveTimeout)
@@ -730,7 +730,7 @@ IsCheckpointOnSchedule(double progress)
         * Check progress against time elapsed and checkpoint_timeout.
         */
        gettimeofday(&now, NULL);
-       elapsed_time = ((double) (now.tv_sec - ckpt_start_time) +
+       elapsed_time = ((double) ((pg_time_t) now.tv_sec - ckpt_start_time) +
                                        now.tv_usec / 1000000.0) / CheckPointTimeout;
 
        if (progress < elapsed_time)
index aa5d34ef5b26faee36499121f6ca039beb615efb..91bf56225a18c0035df9104564261e1db3bd3f28 100644 (file)
@@ -18,7 +18,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/postmaster/syslogger.c,v 1.44 2008/01/25 20:42:10 tgl Exp $
+ *       $PostgreSQL: pgsql/src/backend/postmaster/syslogger.c,v 1.45 2008/02/17 02:09:27 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -331,7 +331,7 @@ SysLoggerMain(int argc, char *argv[])
                if (!rotation_requested && Log_RotationAge > 0)
                {
                        /* Do a logfile rotation if it's time */
-                       pg_time_t       now = time(NULL);
+                       pg_time_t       now = (pg_time_t) time(NULL);
 
                        if (now >= next_rotation_time)
                                rotation_requested = time_based_rotation = true;
index bec5e373964444a9446635d597f7705ca78ee07c..70f60c94622448f29365213418dbe7db55be600d 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/utils/adt/date.c,v 1.138 2008/01/01 19:45:52 momjian Exp $
+ *       $PostgreSQL: pgsql/src/backend/utils/adt/date.c,v 1.139 2008/02/17 02:09:28 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -945,8 +945,10 @@ tm2time(struct pg_tm * tm, fsec_t fsec, TimeADT *result)
 
 /* time2tm()
  * Convert time data type to POSIX time structure.
- * For dates within the system-supported time_t range, convert to the
- *     local time zone. If out of this range, leave as GMT. - tgl 97/05/27
+ *
+ * For dates within the range of pg_time_t, convert to the local time zone.
+ * If out of this range, leave as UTC (in practice that could only happen
+ * if pg_time_t is just 32 bits) - thomas 97/05/27
  */
 static int
 time2tm(TimeADT time, struct pg_tm * tm, fsec_t *fsec)
@@ -2466,10 +2468,9 @@ timetz_zone(PG_FUNCTION_ARGS)
        if (tzp)
        {
                /* Get the offset-from-GMT that is valid today for the selected zone */
-               pg_time_t       now;
+               pg_time_t       now = (pg_time_t) time(NULL);
                struct pg_tm *tm;
 
-               now = time(NULL);
                tm = pg_localtime(&now, tzp);
                tz = -tm->tm_gmtoff;
        }
index ed411cf21af8d230f41b1eb839c76e951142e74e..0bfdd0ec2d6fdf945b1328236147a3c158359a51 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/utils/adt/datetime.c,v 1.184 2008/01/01 19:45:52 momjian Exp $
+ *       $PostgreSQL: pgsql/src/backend/utils/adt/datetime.c,v 1.185 2008/02/17 02:09:28 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -650,9 +650,11 @@ ParseDateTime(const char *timestr, char *workbuf, size_t buflen,
  *                             "20011225T040506.789-07"
  *
  * Use the system-provided functions to get the current time zone
- *     if not specified in the input string.
- * If the date is outside the time_t system-supported time range,
- *     then assume UTC time zone. - thomas 1997-05-27
+ * if not specified in the input string.
+ *
+ * If the date is outside the range of pg_time_t (in practice that could only
+ * happen if pg_time_t is just 32 bits), then assume UTC time zone - thomas
+ * 1997-05-27
  */
 int
 DecodeDateTime(char **field, int *ftype, int nf,
index a2e9fc264334f3314bde800915ee4c607f1da732..2265b7d903eca17e5462932f789e8a03942d5aa5 100644 (file)
@@ -10,7 +10,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/utils/adt/nabstime.c,v 1.152 2008/01/01 19:45:52 momjian Exp $
+ *       $PostgreSQL: pgsql/src/backend/utils/adt/nabstime.c,v 1.153 2008/02/17 02:09:28 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -86,6 +86,8 @@ static void parsetinterval(char *i_string,
  * GetCurrentAbsoluteTime()
  *
  * Get the current system time (relative to Unix epoch).
+ *
+ * NB: this will overflow in 2038; it should be gone long before that.
  */
 AbsoluteTime
 GetCurrentAbsoluteTime(void)
@@ -1029,12 +1031,7 @@ tintervalrel(PG_FUNCTION_ARGS)
 Datum
 timenow(PG_FUNCTION_ARGS)
 {
-       time_t          sec;
-
-       if (time(&sec) < 0)
-               PG_RETURN_ABSOLUTETIME(INVALID_ABSTIME);
-
-       PG_RETURN_ABSOLUTETIME((AbsoluteTime) sec);
+       PG_RETURN_ABSOLUTETIME(GetCurrentAbsoluteTime());
 }
 
 /*
index 1277562b7941419bc713da36c05e3fb0cb6333b1..f450922f26e5c00b6acc3ae346acebd95e70b624 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/utils/adt/timestamp.c,v 1.184 2008/01/01 19:45:52 momjian Exp $
+ *       $PostgreSQL: pgsql/src/backend/utils/adt/timestamp.c,v 1.185 2008/02/17 02:09:28 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1264,9 +1264,14 @@ TimestampDifferenceExceeds(TimestampTz start_time,
  *
  * We do not use time_t internally in Postgres, but this is provided for use
  * by functions that need to interpret, say, a stat(2) result.
+ *
+ * To avoid having the function's ABI vary depending on the width of time_t,
+ * we declare the argument as pg_time_t, which is cast-compatible with
+ * time_t but always 64 bits wide (unless the platform has no 64-bit type).
+ * This detail should be invisible to callers, at least at source code level.
  */
 TimestampTz
-time_t_to_timestamptz(time_t tm)
+time_t_to_timestamptz(pg_time_t tm)
 {
        TimestampTz result;
 
@@ -1284,17 +1289,22 @@ time_t_to_timestamptz(time_t tm)
  * Convert a TimestampTz to time_t.
  *
  * This too is just marginally useful, but some places need it.
+ *
+ * To avoid having the function's ABI vary depending on the width of time_t,
+ * we declare the result as pg_time_t, which is cast-compatible with
+ * time_t but always 64 bits wide (unless the platform has no 64-bit type).
+ * This detail should be invisible to callers, at least at source code level.
  */
-time_t
+pg_time_t
 timestamptz_to_time_t(TimestampTz t)
 {
-       time_t          result;
+       pg_time_t       result;
 
 #ifdef HAVE_INT64_TIMESTAMP
-       result = (time_t) (t / USECS_PER_SEC +
+       result = (pg_time_t) (t / USECS_PER_SEC +
                                 ((POSTGRES_EPOCH_JDATE - UNIX_EPOCH_JDATE) * SECS_PER_DAY));
 #else
-       result = (time_t) (t +
+       result = (pg_time_t) (t +
                                 ((POSTGRES_EPOCH_JDATE - UNIX_EPOCH_JDATE) * SECS_PER_DAY));
 #endif
 
index 3cc443b629754cadca7e05e1c194f3f138c7b1e2..d0ce929d7d086e40f996d8acdd145a3c740e6d94 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/utils/init/globals.c,v 1.104 2008/01/01 19:45:53 momjian Exp $
+ *       $PostgreSQL: pgsql/src/backend/utils/init/globals.c,v 1.105 2008/02/17 02:09:29 tgl Exp $
  *
  * NOTES
  *       Globals used all over the place should be declared here and not
@@ -33,7 +33,7 @@ volatile uint32 InterruptHoldoffCount = 0;
 volatile uint32 CritSectionCount = 0;
 
 int                    MyProcPid;
-time_t         MyStartTime;
+pg_time_t      MyStartTime;
 struct Port *MyProcPort;
 long           MyCancelKey;
 
index 682a5cf380779646a197f9c6e3396e5624145630..4cb6ada31293ec8e02788f18b79e4575f7ef3f94 100644 (file)
@@ -6,7 +6,7 @@
  * copyright (c) Oliver Elphick <olly@lfix.co.uk>, 2001;
  * licence: BSD
  *
- * $PostgreSQL: pgsql/src/bin/pg_controldata/pg_controldata.c,v 1.36 2008/01/21 11:17:46 petere Exp $
+ * $PostgreSQL: pgsql/src/bin/pg_controldata/pg_controldata.c,v 1.37 2008/02/17 02:09:29 tgl Exp $
  */
 #include "postgres.h"
 
@@ -69,6 +69,7 @@ main(int argc, char *argv[])
        char            ControlFilePath[MAXPGPATH];
        char       *DataDir;
        pg_crc32        crc;
+       time_t          time_tmp;
        char            pgctime_str[128];
        char            ckpttime_str[128];
        char            sysident_str[32];
@@ -134,13 +135,20 @@ main(int argc, char *argv[])
                                 "is expecting.  The results below are untrustworthy.\n\n"));
 
        /*
+        * This slightly-chintzy coding will work as long as the control file
+        * timestamps are within the range of time_t; that should be the case
+        * in all foreseeable circumstances, so we don't bother importing the
+        * backend's timezone library into pg_controldata.
+        *
         * Use variable for format to suppress overly-anal-retentive gcc warning
         * about %c
         */
+       time_tmp = (time_t) ControlFile.time;
        strftime(pgctime_str, sizeof(pgctime_str), strftime_fmt,
-                        localtime(&(ControlFile.time)));
+                        localtime(&time_tmp));
+       time_tmp = (time_t) ControlFile.checkPointCopy.time;
        strftime(ckpttime_str, sizeof(ckpttime_str), strftime_fmt,
-                        localtime(&(ControlFile.checkPointCopy.time)));
+                        localtime(&time_tmp));
 
        /*
         * Format system_identifier separately to keep platform-dependent format
index 83b6b4b1d7a9f8a2c98fd9f310c2d2c41374d10c..86c4463a2ce7ea5f00f11d89779253100051caf5 100644 (file)
@@ -23,7 +23,7 @@
  * Portions Copyright (c) 1996-2008, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/bin/pg_resetxlog/pg_resetxlog.c,v 1.63 2008/01/01 19:45:55 momjian Exp $
+ * $PostgreSQL: pgsql/src/bin/pg_resetxlog/pg_resetxlog.c,v 1.64 2008/02/17 02:09:30 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -471,10 +471,10 @@ GuessControlValues(void)
        ControlFile.checkPointCopy.nextOid = FirstBootstrapObjectId;
        ControlFile.checkPointCopy.nextMulti = FirstMultiXactId;
        ControlFile.checkPointCopy.nextMultiOffset = 0;
-       ControlFile.checkPointCopy.time = time(NULL);
+       ControlFile.checkPointCopy.time = (pg_time_t) time(NULL);
 
        ControlFile.state = DB_SHUTDOWNED;
-       ControlFile.time = time(NULL);
+       ControlFile.time = (pg_time_t) time(NULL);
        ControlFile.checkPoint = ControlFile.checkPointCopy.redo;
 
        ControlFile.maxAlign = MAXIMUM_ALIGNOF;
@@ -603,10 +603,10 @@ RewriteControlFile(void)
        ControlFile.checkPointCopy.redo.xlogid = newXlogId;
        ControlFile.checkPointCopy.redo.xrecoff =
                newXlogSeg * XLogSegSize + SizeOfXLogLongPHD;
-       ControlFile.checkPointCopy.time = time(NULL);
+       ControlFile.checkPointCopy.time = (pg_time_t) time(NULL);
 
        ControlFile.state = DB_SHUTDOWNED;
-       ControlFile.time = time(NULL);
+       ControlFile.time = (pg_time_t) time(NULL);
        ControlFile.checkPoint = ControlFile.checkPointCopy.redo;
        ControlFile.prevCheckPoint.xlogid = 0;
        ControlFile.prevCheckPoint.xrecoff = 0;
index a4a4e72badf82a5f7c94b4dd335f4c8e2fb0c280..88fe0b6a95ff781c4b80502fd0c60c19b3dc4f1e 100644 (file)
  * Portions Copyright (c) 1996-2008, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/include/access/xlog_internal.h,v 1.22 2008/01/01 19:45:56 momjian Exp $
+ * $PostgreSQL: pgsql/src/include/access/xlog_internal.h,v 1.23 2008/02/17 02:09:30 tgl Exp $
  */
 #ifndef XLOG_INTERNAL_H
 #define XLOG_INTERNAL_H
 
-#include <time.h>
-
 #include "access/xlog.h"
 #include "fmgr.h"
+#include "pgtime.h"
 #include "storage/block.h"
 #include "storage/relfilenode.h"
 
@@ -71,7 +70,7 @@ typedef struct XLogContRecord
 /*
  * Each page of XLOG file has a header like this:
  */
-#define XLOG_PAGE_MAGIC 0xD062 /* can be used as WAL version indicator */
+#define XLOG_PAGE_MAGIC 0xD063 /* can be used as WAL version indicator */
 
 typedef struct XLogPageHeaderData
 {
@@ -242,7 +241,7 @@ extern const RmgrData RmgrTable[];
 /*
  * Exported to support xlog switching from bgwriter
  */
-extern time_t GetLastSegSwitchTime(void);
+extern pg_time_t GetLastSegSwitchTime(void);
 extern XLogRecPtr RequestXLogSwitch(void);
 
 /*
index 84e6d9424e0aafb6d724b3b3836f087b729073ae..e61faed987646c6eb9f7cba12bf9d84c26685fee 100644 (file)
@@ -8,21 +8,20 @@
  * Portions Copyright (c) 1996-2008, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/include/catalog/pg_control.h,v 1.39 2008/01/01 19:45:56 momjian Exp $
+ * $PostgreSQL: pgsql/src/include/catalog/pg_control.h,v 1.40 2008/02/17 02:09:30 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
 #ifndef PG_CONTROL_H
 #define PG_CONTROL_H
 
-#include <time.h>
-
 #include "access/xlogdefs.h"
+#include "pgtime.h"                            /* for pg_time_t */
 #include "utils/pg_crc.h"
 
 
 /* Version identifier for this pg_control format */
-#define PG_CONTROL_VERSION     833
+#define PG_CONTROL_VERSION     841
 
 /*
  * Body of CheckPoint XLOG records.  This is declared here because we keep
@@ -38,7 +37,7 @@ typedef struct CheckPoint
        Oid                     nextOid;                /* next free OID */
        MultiXactId nextMulti;          /* next free MultiXactId */
        MultiXactOffset nextMultiOffset;        /* next free MultiXact offset */
-       time_t          time;                   /* time stamp of checkpoint */
+       pg_time_t       time;                   /* time stamp of checkpoint */
 } CheckPoint;
 
 /* XLOG info values for XLOG rmgr */
@@ -99,7 +98,7 @@ typedef struct ControlFileData
         * System status data
         */
        DBState         state;                  /* see enum above */
-       time_t          time;                   /* time stamp of last pg_control update */
+       pg_time_t       time;                   /* time stamp of last pg_control update */
        XLogRecPtr      checkPoint;             /* last check point record ptr */
        XLogRecPtr      prevCheckPoint; /* previous check point record ptr */
 
index 6ded4c9bc4c539b5a963bc83c74eaa75eb00a4c0..75a7e1f7f7cbb7cb178d83ff01cb29c37eeca143 100644 (file)
@@ -13,7 +13,7 @@
  * Portions Copyright (c) 1996-2008, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/include/miscadmin.h,v 1.199 2008/01/03 21:23:15 tgl Exp $
+ * $PostgreSQL: pgsql/src/include/miscadmin.h,v 1.200 2008/02/17 02:09:31 tgl Exp $
  *
  * NOTES
  *       some of the information in this file should be moved to other files.
@@ -23,7 +23,7 @@
 #ifndef MISCADMIN_H
 #define MISCADMIN_H
 
-#include <time.h>                              /* for time_t */
+#include "pgtime.h"                            /* for pg_time_t */
 
 
 #define PG_VERSIONSTR "postgres (PostgreSQL) " PG_VERSION "\n"
@@ -134,7 +134,7 @@ extern int  MaxBackends;
 extern int     MaxConnections;
 
 extern PGDLLIMPORT int MyProcPid;
-extern PGDLLIMPORT time_t MyStartTime;
+extern PGDLLIMPORT pg_time_t MyStartTime;
 extern PGDLLIMPORT struct Port *MyProcPort;
 extern long MyCancelKey;
 
index 54aafa25cc4071b71f2904783a2cb6eec2257b63..d670576787eb54f122c19f9aed110fb08058c251 100644 (file)
@@ -1,4 +1,4 @@
-/* $PostgreSQL: pgsql/src/include/port/win32.h,v 1.83 2008/01/09 09:16:43 mha Exp $ */
+/* $PostgreSQL: pgsql/src/include/port/win32.h,v 1.84 2008/02/17 02:09:31 tgl Exp $ */
 
 #if defined(_MSC_VER) || defined(__BORLANDC__)
 #define WIN32_ONLY_COMPILER
 
 #define USES_WINSOCK
 
-/*
- * Ensure that anyone building an extension is using a 32 bit time_t.
- * On Mingw/Msys, that should always be the case, but MSVC++ defaults
- * to 64 bits. We set that for our own build in the project files
- */
-#if defined(WIN32_ONLY_COMPILER) && !defined(FRONTEND)
-#ifndef _USE_32BIT_TIME_T
-#error "Postgres uses 32 bit time_t - add #define _USE_32BIT_TIME_T on Windows"
-#endif
-#endif
-
 /* defines for dynamic linking on Win32 platform */
 #if defined(WIN32) || defined(__CYGWIN__)
 
@@ -198,6 +187,7 @@ struct itimerval
        struct timeval it_interval;
        struct timeval it_value;
 };
+
 int                    setitimer(int which, const struct itimerval * value, struct itimerval * ovalue);
 
 
index 1f73a2a081aa73b5425cd36a79491d2810bf2ae6..cc224aa147a4835d0dfd489ae14647bd9ffe5687 100644 (file)
@@ -6,7 +6,7 @@
  * Portions Copyright (c) 1996-2008, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/include/utils/timestamp.h,v 1.74 2008/01/23 21:26:13 tgl Exp $
+ * $PostgreSQL: pgsql/src/include/utils/timestamp.h,v 1.75 2008/02/17 02:09:31 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -310,8 +310,8 @@ extern bool TimestampDifferenceExceeds(TimestampTz start_time,
                                                   TimestampTz stop_time,
                                                   int msec);
 
-extern TimestampTz time_t_to_timestamptz(time_t tm);
-extern time_t timestamptz_to_time_t(TimestampTz t);
+extern TimestampTz time_t_to_timestamptz(pg_time_t tm);
+extern pg_time_t timestamptz_to_time_t(TimestampTz t);
 
 extern const char *timestamptz_to_str(TimestampTz t);
 
index 03985cd18acfe53d6d501c16afd42868ee71cb3e..d0d8084b285fb9a1cb3f8275a5fd3dc9d2af8a29 100644 (file)
@@ -3,7 +3,7 @@ package Project;
 #
 # Package that encapsulates a Visual C++ project file generation
 #
-# $PostgreSQL: pgsql/src/tools/msvc/Project.pm,v 1.16 2008/02/05 14:17:23 mha Exp $
+# $PostgreSQL: pgsql/src/tools/msvc/Project.pm,v 1.17 2008/02/17 02:09:32 tgl Exp $
 #
 use Carp;
 use strict;
@@ -494,7 +494,7 @@ sub WriteConfiguration
        ConfigurationType="$cfgtype" UseOfMFC="0" ATLMinimizesCRunTimeLibraryUsage="FALSE" CharacterSet="2" WholeProgramOptimization="$p->{wholeopt}">
        <Tool Name="VCCLCompilerTool" Optimization="$p->{opt}"
                AdditionalIncludeDirectories="$self->{prefixincludes}src/include;src/include/port/win32;src/include/port/win32_msvc;$self->{includes}"
-               PreprocessorDefinitions="WIN32;_WINDOWS;__WINDOWS__;__WIN32__;EXEC_BACKEND;WIN32_STACK_RLIMIT=4194304;_CRT_SECURE_NO_DEPRECATE;_CRT_NONSTDC_NO_DEPRECATE;_USE_32BIT_TIME_T$self->{defines}$p->{defs}"
+               PreprocessorDefinitions="WIN32;_WINDOWS;__WINDOWS__;__WIN32__;EXEC_BACKEND;WIN32_STACK_RLIMIT=4194304;_CRT_SECURE_NO_DEPRECATE;_CRT_NONSTDC_NO_DEPRECATE$self->{defines}$p->{defs}"
                StringPooling="$p->{strpool}"
                RuntimeLibrary="$p->{runtime}" DisableSpecificWarnings="$self->{disablewarnings}"
 EOF