switch to a common calloc implementation
authorRich Felker <dalias@aerifal.cx>
Wed, 10 Jun 2020 23:41:27 +0000 (19:41 -0400)
committerRich Felker <dalias@aerifal.cx>
Thu, 11 Jun 2020 00:34:34 +0000 (20:34 -0400)
commit28f64fa6caeb621780bf116bc8cae7d7a48b477e
treef0ac98c4c7f415fb704ddeea84310be8f3317ec3
parent384c0131ccda2656dec23a0416ad3f14101151a7
switch to a common calloc implementation

abstractly, calloc is completely malloc-implementation-independent;
it's malloc followed by memset, or as we do it, a "conditional memset"
that avoids touching fresh zero pages.

previously, calloc was kept separate for the bump allocator, which can
always skip memset, and the version of calloc provided with the full
malloc conditionally skipped the clearing for large direct-mmapped
allocations. the latter is a moderately attractive optimization, and
can be added back if needed. however, further consideration to make it
correct under malloc replacement would be needed.

commit b4b1e10364c8737a632be61582e05a8d3acf5690 documented the
contract for malloc replacement as allowing omission of calloc, and
indeed that worked for dynamic linking, but for static linking it was
possible to get the non-clearing definition from the bump allocator;
if not for that, it would have been a link error trying to pull in
malloc.o.

the conditional-clearing code for the new common calloc is taken from
mal0_clear in oldmalloc, but drops the need to access actual page size
and just uses a fixed value of 4096. this avoids potentially needing
access to global data for the sake of an optimization that at best
marginally helps archs with offensively-large page sizes.
src/malloc/calloc.c [new file with mode: 0644]
src/malloc/lite_malloc.c
src/malloc/oldmalloc/malloc.c