[PATCH] malloc: Correct the documentation of the top_pad default
Siddhesh Poyarekar
siddhesh@gotplt.org
Thu Aug 4 15:15:33 GMT 2022
On 2022-08-04 11:14, Florian Weimer wrote:
> * Siddhesh Poyarekar:
>
>> On 2022-08-04 04:48, Florian Weimer via Libc-alpha wrote:
>>> DEFAULT_TOP_PAD is defined as 131072 in
>>> sysdeps/generic/malloc-machine.h.
>>
>> LGTM, but I wonder if we should tighten this so that it doesn't appear
>> that DEFAULT_TOP_PAD could be 0 in any of our build configurations.
>> e.g. we could replace the definition with a #error "define
>> DEFAULT_TOP_PAD" or something like that to ensure that it's always
>> defined through malloc-machine.h. What do you think?
>
> We could just remove the various #ifdefs, I guess. The other question
> is whether we can use the constant in the tunable definition file. I
> haven't investigate any of this.
>
> Should I push my change in the meantime?
Yes please.
Thanks,
Sid
More information about the Libc-alpha
mailing list