[jira] Commented: (TS-567) Cleanup of memory allocation in libinktomi++

["Leif Hedstrom (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/TS-567?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12965353#action_12965353 ] 

Leif Hedstrom commented on TS-567:
----------------------------------

Looking at the code, we should use xmalloc / xfree consistently through the code, it's a "debugging" wrapper on top of ink_malloc.

> Cleanup of memory allocation in libinktomi++
> --------------------------------------------
>
>                 Key: TS-567
>                 URL: https://issues.apache.org/jira/browse/TS-567
>             Project: Traffic Server
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Leif Hedstrom
>            Priority: Minor
>             Fix For: 2.1.6
>
>         Attachments: malloc.txt
>
>
> There's a lot of ifdef's and debugging code paths around various allocation schemes in libinktomi++. We should clean this up, and simplify. It'd also be great to make it easy to enable various other allocators, e.g. jemalloc or tcmalloc, as part of the build system, causing ink_malloc / xmalloc and STL containers to use the new allocator. In particular for STL containers, it means we might need to have some allocator setup / initialization?
> Also, when should we use ink_malloc() vs xmalloc() ? It's quite inconsistently used right now I think. And, there are a few places using malloc() (etc.) directly, which we ought to fix.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.