Summary: | ASTERISK-03622: [patch] utils.c does not compile | ||
Reporter: | jacs (jacs) | Labels: | |
Date Opened: | 2005-03-03 08:21:31.000-0600 | Date Closed: | 2008-01-15 15:26:45.000-0600 |
Priority: | Trivial | Regression? | No |
Status: | Closed/Complete | Components: | Core/General |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) util.c.patch.txt | |
Description: | someone has changed the stack size param to AST_STACKSIZE or should it be AST_STACK_SIZE? Changed utils.h to match usage. I note this change has made the stack half what it was for non linux machines. ****** ADDITIONAL INFORMATION ****** disclaimer on file. | ||
Comments: | By: Mark Spencer (markster) 2005-03-03 10:21:30.000-0600 Wow, yah it's more than that. We have to change the stack size on Linux too, because otherwise with the default 2M stack size, you can only have around 200 or 300 threads. I think in the long run it would be nice to be able to have the caller of ast_pthread_create() recommend a stack size to limit us even less. By: twisted (twisted) 2005-03-03 10:24:31.000-0600 Fixed in CVS HEAD, thanks! By: Russell Bryant (russell) 2005-03-09 12:40:17.000-0600 I think I'm just going to leave this the way it is in 1.0 By: Digium Subversion (svnbot) 2008-01-15 15:26:45.000-0600 Repository: asterisk Revision: 5126 U trunk/include/asterisk/utils.h U trunk/utils.c ------------------------------------------------------------------------ r5126 | markster | 2008-01-15 15:26:45 -0600 (Tue, 15 Jan 2008) | 2 lines Fix stack size (take 2) (bug ASTERISK-3622) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=5126 |