Put #ifdef NOT_USED around posix_fadvise call. We may want to resurrect

this someday, but right now it seems that posix_fadvise is immature to
the point of being broken on many platforms ... and we don't have any
benchmark evidence proving it's worth spending time on.

diff --git a/src/backend/access/transam/xlog.c b/src/backend/access/transam/xlog.c
index e540ba672c9841525f533cfdab3d09178ed1424f..3435b210a45ba361bcea823154db1a839d639a2b 100644 (file)
--- a/src/backend/access/transam/xlog.c
+++ b/src/backend/access/transam/xlog.c
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1996-2006, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/backend/access/transam/xlog.c,v 1.241 2006/06/22 20:42:57 tgl Exp $
+ * $PostgreSQL: pgsql/src/backend/access/transam/xlog.c,v 1.242 2006/06/27 18:59:17 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -2153,7 +2153,16 @@ XLogFileClose(void)
 {
        Assert(openLogFile >= 0);
 
-#if defined(HAVE_DECL_POSIX_FADVISE) && defined(POSIX_FADV_DONTNEED)
+       /*
+        * posix_fadvise is problematic on many platforms: on older x86 Linux
+        * it just dumps core, and there are reports of problems on PPC platforms
+        * as well.  The following is therefore disabled for the time being.
+        * We could consider some kind of configure test to see if it's safe to
+        * use, but since we lack hard evidence that there's any useful performance
+        * gain to be had, spending time on that seems unprofitable for now.
+        */
+#ifdef NOT_USED
+
        /*
         * WAL segment files will not be re-read in normal operation, so we advise
         * OS to release any cached pages.  But do not do so if WAL archiving is
@@ -2163,10 +2172,13 @@ XLogFileClose(void)
         * While O_DIRECT works for O_SYNC, posix_fadvise() works for fsync()
         * and O_SYNC, and some platforms only have posix_fadvise().
         */
+#if defined(HAVE_DECL_POSIX_FADVISE) && defined(POSIX_FADV_DONTNEED)
        if (!XLogArchivingActive())
                posix_fadvise(openLogFile, 0, 0, POSIX_FADV_DONTNEED);
 #endif
 
+#endif /* NOT_USED */
+
        if (close(openLogFile))
                ereport(PANIC,
                        (errcode_for_file_access(),