This is an archive of the discontinued LLVM Phabricator instance.

scudo: Don't memset previously released cached pages in the secondary allocator.
ClosedPublic

Authored by pcc on Nov 4 2020, 7:18 PM.

Details

Summary

There is no need to memset released pages because they are already
zero. On db845c, before:

BM_stdlib_malloc_free_default/131072 34562 ns 34547 ns 20258 bytes_per_second=3.53345G/s

after:

BM_stdlib_malloc_free_default/131072 29618 ns 29589 ns 23485 bytes_per_second=4.12548G/s

Diff Detail

Event Timeline

pcc created this revision.Nov 4 2020, 7:18 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 4 2020, 7:18 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
pcc requested review of this revision.Nov 4 2020, 7:18 PM
cryptoad accepted this revision.Nov 5 2020, 7:35 AM
This revision is now accepted and ready to land.Nov 5 2020, 7:35 AM
This revision was landed with ongoing or failed builds.Nov 5 2020, 9:25 AM
This revision was automatically updated to reflect the committed changes.