Unix/Process.inc: Revert r72332, "Work around a page size issue on Cygwin."
authorNAKAMURA Takumi <geek4civic@gmail.com>
Wed, 21 Aug 2013 13:47:12 +0000 (13:47 +0000)
committerNAKAMURA Takumi <geek4civic@gmail.com>
Wed, 21 Aug 2013 13:47:12 +0000 (13:47 +0000)
Offset in mmap(3) should be aligned to gepagesize(), 64k, or mmap(3) would fail.

TODO: Invetigate places where 4096 would be required as pagesize, or 4096 would satisfy.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@188903 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Support/Unix/Process.inc

index 8bc8d91eb54406713c0c8a98353450750177813b..cd244f69e746e2d36d65a19c0f63cbf87bdd13fa 100644 (file)
@@ -88,13 +88,10 @@ TimeValue self_process::get_system_time() const {
   return getRUsageTimes().second;
 }
 
+// On Cygwin, getpagesize() returns 64k and offset in mmap(3) should be
+// aligned to its pagesize.
 static unsigned getPageSize() {
-#if defined(__CYGWIN__)
-  // On Cygwin, getpagesize() returns 64k but the page size for the purposes of
-  // memory protection and mmap() is 4k.
-  // See http://www.cygwin.com/ml/cygwin/2009-01/threads.html#00492
-  const int page_size = 0x1000;
-#elif defined(HAVE_GETPAGESIZE)
+#if defined(HAVE_GETPAGESIZE)
   const int page_size = ::getpagesize();
 #elif defined(HAVE_SYSCONF)
   long page_size = ::sysconf(_SC_PAGE_SIZE);