[prev in list] [next in list] [prev in thread] [next in thread] 

List:       asterisk-dev
Subject:    Re: [asterisk-dev] [Code Review] MALLOC_DEBUG atexit unreleased malloc memory summary.
From:       "Mark Michelson" <reviewboard () asterisk ! org>
Date:       2012-11-27 23:24:33
Message-ID: 20121127232433.11543.31082 () hotblack ! digium ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2196/#review7451
-----------------------------------------------------------

Ship it!


- Mark


On Nov. 26, 2012, 5:26 p.m., rmudgett wrote:
> =

> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2196/
> -----------------------------------------------------------
> =

> (Updated Nov. 26, 2012, 5:26 p.m.)
> =

> =

> Review request for Asterisk Developers.
> =

> =

> Summary
> -------
> =

> * Adds the following CLI commands to control MALLOC_DEBUG reporting of un=
released malloc memory when Asterisk is shut down.
> memory atexit list on
> memory atexit list off
> memory atexit summary byline
> memory atexit summary byfunc
> memory atexit summary byfile
> memory atexit summary off
> =

> * Made check all remaining allocated region blocks atexit for fence viola=
tions.
> =

> * Increased the allocated region hash table size by about three times.  I=
t still isn't large enough considering the number of malloced blocks Asteri=
sk uses.
> =

> =

> Diffs
> -----
> =

>   /branches/1.8/main/asterisk.c 376620 =

>   /branches/1.8/main/astmm.c 376620 =

> =

> Diff: https://reviewboard.asterisk.org/r/2196/diff
> =

> =

> Testing
> -------
> =

> Started Asterisk.
> CLI "memory atexit list on"
> CLI "memory atexit summary byline"
> CLI "memory atexit summary byfunc"
> CLI "memory atexit summary byfile"
> =

> Output format at Asterisk exit as intended for unreleased memory.
> =

> =

> Thanks,
> =

> rmudgett
> =

>


[Attachment #5 (text/html)]

<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 \
solid;">  <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="https://reviewboard.asterisk.org/r/2196/">https://reviewboard.asterisk.org/r/2196/</a>
  </td>
    </tr>
   </table>
   <br />



 <p>Ship it!</p>









<p>- Mark</p>


<br />
<p>On November 26th, 2012, 5:26 p.m., rmudgett wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" \
style="background-image: \
url('https://reviewboard.asterisk.org/media/rb/images/review_request_box_top_bg.png'); \
background-position: left top; background-repeat: repeat-x; border: 1px black \
solid;">  <tr>
  <td>

<div>Review request for Asterisk Developers.</div>
<div>By rmudgett.</div>


<p style="color: grey;"><i>Updated Nov. 26, 2012, 5:26 p.m.</i></p>




<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: \
1px solid #b8b5a0">  <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">* Adds the following CLI commands to control MALLOC_DEBUG reporting of \
unreleased malloc memory when Asterisk is shut down. memory atexit list on
memory atexit list off
memory atexit summary byline
memory atexit summary byfunc
memory atexit summary byfile
memory atexit summary off

* Made check all remaining allocated region blocks atexit for fence violations.

* Increased the allocated region hash table size by about three times.  It still \
isn&#39;t large enough considering the number of malloced blocks Asterisk uses.</pre> \
</td>  </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: \
1px solid #b8b5a0">  <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">Started Asterisk. CLI &quot;memory atexit list on&quot;
CLI &quot;memory atexit summary byline&quot;
CLI &quot;memory atexit summary byfunc&quot;
CLI &quot;memory atexit summary byfile&quot;

Output format at Asterisk exit as intended for unreleased memory.
</pre>
  </td>
 </tr>
</table>




<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>/branches/1.8/main/asterisk.c <span style="color: grey">(376620)</span></li>

 <li>/branches/1.8/main/astmm.c <span style="color: grey">(376620)</span></li>

</ul>

<p><a href="https://reviewboard.asterisk.org/r/2196/diff/" style="margin-left: \
3em;">View Diff</a></p>




  </td>
 </tr>
</table>








  </div>
 </body>
</html>



--
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --

asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic