From: Sean Bright Date: Mon, 8 Jun 2009 19:24:32 +0000 (+0000) Subject: Increase the size of our thread stack on 64 bit processors. X-Git-Tag: 1.4.26-rc2~5 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0d849d509de5778dab42fd5e3be49fae75b585f4;p=thirdparty%2Fasterisk.git 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 git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@199626 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- diff --git a/include/asterisk/utils.h b/include/asterisk/utils.h index f4790decac..253cfa91d5 100644 --- a/include/asterisk/utils.h +++ b/include/asterisk/utils.h @@ -262,12 +262,12 @@ static force_inline int inaddrcmp(const struct sockaddr_in *sin1, const struct s || (sin1->sin_port != sin2->sin_port)); } -#define AST_STACKSIZE 240 * 1024 +#define AST_STACKSIZE (((__WORDSIZE * 6) - 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);