[PATCH] Handle USING_MADVISE without USING_MMAP

Andi Kleen andi@firstfloor.org
Mon Oct 17 17:43:00 GMT 2011


I plan to check in the following patch as obvious after it passed
testing. 

cygwin has a MADV_DONTNEED, but does not use mmap. The ifdefs for
madvise assumed this wouldn't happen and it broke the cygwin build.

Just don't set USING_MADVISE when USING_MMAP is not set. Thanks
to Kai Titz for testing.

-Andi

2011-10-17  Andi Kleen  <ak@linux.intel.com>

       * ggc-page.c (USING_MADVISE): Adjust ifdef to check for
       USING_MMAP.

diff --git a/gcc/ggc-page.c b/gcc/ggc-page.c
index 9b35291..2da99db 100644
--- a/gcc/ggc-page.c
+++ b/gcc/ggc-page.c
@@ -50,7 +50,7 @@ along with GCC; see the file COPYING3.  If not see
 #define USING_MALLOC_PAGE_GROUPS
 #endif
 
-#if defined(HAVE_MADVISE) && defined(MADV_DONTNEED)
+#if defined(HAVE_MADVISE) && defined(MADV_DONTNEED) && defined(USING_MMAP)
 # define USING_MADVISE
 #endif
 
-- 
1.7.5.4

-- 
ak@linux.intel.com -- Speaking for myself only.



More information about the Gcc-patches mailing list