[Home]

Summary:ASTERISK-15901: [patch] minmemfree does not work
Reporter:loic (loic)Labels:
Date Opened:2010-04-01 08:59:01Date Closed:2011-07-19 04:37:07
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/PBX
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) pbx_c.diff
Description:When you set minmemfree option in asterisk.conf and you place a call, you have already this error :
pbx.c:4501 increase_call_count: Available system memory (~1129MB) is below the configured low watermark (*your minmemfree parameter*MB)
chan_sip.c:20305 handle_request_invite: Failed to start PBX (call limit reached)

There is an error in the calculation of available memory in main.pbx.c.
To avoid that, I replace "curfreemem = sys_info.freeram / sys_info.mem_unit;" by "curfreemem = sys_info.freeram * sys_info.mem_unit;" in function increase_call_count.

The good code :
<inline code removed by lmadsen>
Comments:By: Leif Madsen (lmadsen) 2010-04-01 09:22:48

We can not accept code that is submitted inline. Please submit by attaching a unified diff to this issue. If you have not already signed the electronic license, please do so prior to uploading the file. It will be a link at the top of the page.
By: loic (loic) 2010-04-01 10:36:56

I sent a diff file ;)
By: Leif Madsen (lmadsen) 2010-04-01 11:06:46

Thanks!
By: Digium Subversion (svnbot) 2011-05-06 16:06:34

Repository: asterisk
Revision: 317917

U   branches/1.8/main/pbx.c

------------------------------------------------------------------------
r317917 | russell | 2011-05-06 16:06:34 -0500 (Fri, 06 May 2011) | 7 lines

Fix calculation of free RAM to make minmemfree option work.

(closes issue ASTERISK-15901)
Reported by: loic
Patches:
     pbx_c.diff uploaded by loic (license 1020)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=317917
By: Digium Subversion (svnbot) 2011-05-06 16:07:50

Repository: asterisk
Revision: 317919

_U  trunk/
U   trunk/main/pbx.c

------------------------------------------------------------------------
r317919 | russell | 2011-05-06 16:07:50 -0500 (Fri, 06 May 2011) | 14 lines

Merged revisions 317917 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
 r317917 | russell | 2011-05-06 16:06:33 -0500 (Fri, 06 May 2011) | 7 lines
 
 Fix calculation of free RAM to make minmemfree option work.
 
 (closes issue ASTERISK-15901)
 Reported by: loic
 Patches:
       pbx_c.diff uploaded by loic (license 1020)
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=317919