Re: [PATCH] Remove all uses of mallinfo(3)

From: Kinkie <gkinkie_at_gmail.com>
Date: Wed, 19 Mar 2014 18:54:24 +0100

Hi,
  the whole train has now been merge in trunk r13318.

On Tue, Mar 18, 2014 at 11:11 AM, Kinkie <gkinkie_at_gmail.com> wrote:
>> I'm favour of this and have been for some time. However the patch does
>> not go far eonough to do what this submission claims.
>>
>> 1) do_mallinfo is removed in tools.cc but the globals.h and main.cc
>> parts of it are not removed.
>
> Removed.
>
>> 2) The high_memory_warning directive now depends uniquely on HAVE_MSTATS
>> && HAVE_GNUMALLOC_H.
>> * Please add an cf.data.pre IFDEF check on those macros being defined
>
> Done
>
>> * Please update the directive documentation to omit mallinfo
>> and define the documentation.
>
> Done.
>
>> * perhapse a TODO entry about linking that directive to a mempools
>> capacity limit would be good idea.
>
> Done
>
>> 3) was --enable-snmp used at the time?
>> IIRC this should be interacting with some of the memory OID stats but I
>> don't seen any OID calculatiosn or docs being adjusted.
>
> It uses memPoolsTotalAllocated(), so nothing to do there.
>
> --
> Francesco

-- 
    Francesco
Received on Wed Mar 19 2014 - 17:54:38 MDT

This archive was generated by hypermail 2.2.0 : Thu Mar 20 2014 - 12:00:13 MDT