[or-cvs] r14056: Use 8k pages in openbsd malloc code on alpha. Bug and soluti (in tor/trunk: . src/common)

nickm at seul.org nickm at seul.org
Mon Mar 17 02:47:40 UTC 2008


Author: nickm
Date: 2008-03-16 22:47:40 -0400 (Sun, 16 Mar 2008)
New Revision: 14056

Modified:
   tor/trunk/
   tor/trunk/ChangeLog
   tor/trunk/src/common/OpenBSD_malloc_Linux.c
Log:
 r18850 at catbus:  nickm | 2008-03-16 22:35:38 -0400
 Use 8k pages in openbsd malloc code on alpha. Bug and solution found by weasel.  Also, when initializing openbsd malloc code, check that compiled page size matches output of getpagesize().



Property changes on: tor/trunk
___________________________________________________________________
 svk:merge ticket from /tor/trunk [r18850] on 8246c3cf-6607-4228-993b-4d95d33730f1

Modified: tor/trunk/ChangeLog
===================================================================
--- tor/trunk/ChangeLog	2008-03-16 11:42:16 UTC (rev 14055)
+++ tor/trunk/ChangeLog	2008-03-17 02:47:40 UTC (rev 14056)
@@ -24,6 +24,10 @@
     - Use proper log levels in the testsuite call of get_interface_address6().
     - When using a nonstandard malloc, do not use the platform values for
       HAVE_MALLOC_GOOD_SIZE or HAVE_MALLOC_USABLE_SIZE.
+    - Make the openbsd malloc code use 8k pages on alpha CPUs.  Bugfix on
+      0.2.0.x.
+    - Detect mismatched page sizes when using --enable-openbsd-malloc.
+      Bugfix on 0.2.0.x.
 
   o Minor features:
     - Allow separate log levels to be configured for different logging

Modified: tor/trunk/src/common/OpenBSD_malloc_Linux.c
===================================================================
--- tor/trunk/src/common/OpenBSD_malloc_Linux.c	2008-03-16 11:42:16 UTC (rev 14055)
+++ tor/trunk/src/common/OpenBSD_malloc_Linux.c	2008-03-17 02:47:40 UTC (rev 14056)
@@ -96,7 +96,7 @@
 #define _MALLOC_LOCK() {pthread_mutex_lock(&gen_mutex);}
 #define _MALLOC_UNLOCK() {pthread_mutex_unlock(&gen_mutex);}
 
-#if defined(__sparc__)
+#if defined(__sparc__) || defined(__alpha__)
 #define	malloc_pageshift	13U
 #endif /* __sparc__ */
 
@@ -787,6 +787,13 @@
 		    "  Will not be able to dump malloc stats on exit");
 #endif /* MALLOC_STATS */
 
+	if (malloc_pagesize != getpagesize()) {
+		wrterror("malloc() replacement compiled with a different "
+			 "page size from what we're running with.  Failing.");
+		errno = ENOMEM;
+		return;
+	}
+
 	/* Allocate one page for the page directory. */
 	page_dir = (struct pginfo **)MMAP(malloc_pagesize);
 



More information about the tor-commits mailing list