[PATCH] calloc should not duplicate malloc logic.
Carlos O'Donell
carlos@redhat.com
Mon Mar 3 17:32:00 GMT 2014
On 03/03/2014 01:02 AM, Will Newton wrote:
>> A faster way would be additionally apply following. There are three
>> parts covered, first one is locking and libc_malloc has almost identical
>> code as this. Second is mmap that I for some reason forgotten. Third
>> part is optimizing memset which should go away as it contains
>> assumptions that will be broken, like that chunk is old multiple of 8.
>>
>> * malloc/malloc.c ( __libc_calloc): Add back mmaped memory
>> handling.
>
> My personal feeling is that the original patch did not meet the
> requirements of consensus and should be reverted on that basis, rather
> than continuing to patch up the results.
Given the objections I've reverted the calloc simplification.
I tested the reverted code works correctly on x86-64.
commit d6285c9f0b3972369356554727b1ede5a6eb0731
Author: Carlos O'Donell <carlos@redhat.com>
Date: Mon Mar 3 12:28:25 2014 -0500
Revert 4248f0da6ff9e7dd63464cdecec2dec332dfc2f0.
Objections were raised surrounding the calloc simplification
and it is better to revert the patch, continue discussions
and then submit a new patch for inclusion with all issues
fully addressed.
Cheers,
Carlos.
More information about the Libc-alpha
mailing list