diff options
| author | Ryan Prichard <rprichard@google.com> | 2020-10-09 05:08:36 -0700 |
|---|---|---|
| committer | Ryan Prichard <rprichard@google.com> | 2020-10-20 01:10:33 -0700 |
| commit | de523c02bbea1923ee5fbe9d1d60a905f6e965b0 (patch) | |
| tree | ca34781ef3496b4b83303bce9f038231bba91433 /tests/malloc_test.cpp | |
| parent | b88edc6739d2582597e6164edf49cdcc669abcbc (diff) | |
Fix __cxa_atexit run-time regression with many registrations
When an array element is added or removed, make only the relevant page
writable, rather than the entire array. The entire array is still made
writable during recompaction and expansion.
This change fixes most of a large regression in __cxa_atexit runtime
(blueline, taskset 10, performance governor, 100000 registrations,
times are in seconds)
- Q: _Exit=0.292380, exit=0.626801
- R: _Exit=28.435082, exit=95.785110
- new: _Exit=0.352285, exit=0.713893
Test: bionic unit tests
Test: adb shell taskset 10 \
/data/benchmarktest64/bionic-spawn-benchmarks/bionic-spawn-benchmarks \
--benchmark_filter='atexit' \
--benchmark_display_aggregates_only=true \
--benchmark_repetitions=10
Bug: http://b/168043760
Change-Id: I88cc15c29c9890b422b7f621f29f98a03ca1f886
Diffstat (limited to 'tests/malloc_test.cpp')
0 files changed, 0 insertions, 0 deletions