[svn-commits] seanbright: trunk r199630 - in /trunk: ./ include/asterisk/utils.h

SVN commits to the Digium repositories svn-commits at lists.digium.com
Mon Jun 8 14:33:14 CDT 2009


Author: seanbright
Date: Mon Jun  8 14:33:09 2009
New Revision: 199630

URL: http://svn.asterisk.org/svn-view/asterisk?view=rev&rev=199630
Log:
Merged revisions 199626,199628 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r199626 | seanbright | 2009-06-08 15:24:32 -0400 (Mon, 08 Jun 2009) | 21 lines
  
  Increase the size of our thread stack on 64 bit processors.
  
  We were setting the stack size for each thread to 240KB regardless of
  architecture, which meant that in some scenarios we actually had less available
  stack space on 64 bit processors (pointers use 8 bytes instead of 4).  So now we
  calculate the stack size we reserve based on the platform's __WORDSIZE, which
  gives us:
  
       32 bit -> 240KB
       64 bit -> 496KB
      128 bit -> 1008KB (that's right, we're ready for 128 bit processors)
  
  Patch typed by me but written by several members of #asterisk-dev, including
  Kevin, Tilghman, and Qwell.
  
  (closes issue #14932)
  Reported by: jpiszcz
  Patches:
        06052009_issue14932.patch uploaded by seanbright (license 71)
  Tested by: seanbright
........
  r199628 | seanbright | 2009-06-08 15:28:33 -0400 (Mon, 08 Jun 2009) | 2 lines
  
  Fix a typo in the stack size calculation just introduced.
........

Modified:
    trunk/   (props changed)
    trunk/include/asterisk/utils.h

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-1.4-merged' - no diff available.

Modified: trunk/include/asterisk/utils.h
URL: http://svn.asterisk.org/svn-view/asterisk/trunk/include/asterisk/utils.h?view=diff&rev=199630&r1=199629&r2=199630
==============================================================================
--- trunk/include/asterisk/utils.h (original)
+++ trunk/include/asterisk/utils.h Mon Jun  8 14:33:09 2009
@@ -349,13 +349,13 @@
 /*
  * Thread management support (should be moved to lock.h or a different header)
  */
- 
-#define AST_STACKSIZE 240 * 1024
+
+#define AST_STACKSIZE (((__WORDSIZE * 8) - 16) * 1024)
 
 #if defined(LOW_MEMORY)
-#define AST_BACKGROUND_STACKSIZE 48 * 1024
+#define AST_BACKGROUND_STACKSIZE (((__WORDSIZE * 2) - 16) * 1024)
 #else
-#define AST_BACKGROUND_STACKSIZE 240 * 1024
+#define AST_BACKGROUND_STACKSIZE AST_STACKSIZE
 #endif
 
 void ast_register_thread(char *name);




More information about the svn-commits mailing list