[scudo] Don't track free/use stats for transfer batches.
authorMitch Phillips <31459023+hctim@users.noreply.github.com>
Mon, 3 May 2021 17:42:19 +0000 (10:42 -0700)
committerMitch Phillips <31459023+hctim@users.noreply.github.com>
Mon, 3 May 2021 18:50:00 +0000 (11:50 -0700)
commite8f7241e0b48d557406ece7e0c2f273eea2d7e39
tree380c1968f956a7f13cd7697d600af53672e0fec5
parent39bbfb77264a4a7a216921c2b70a30ba0f27eb56
[scudo] Don't track free/use stats for transfer batches.

The Scudo C unit tests are currently non-hermetic. In particular, adding
or removing a transfer batch is a global state of the allocator that
persists between tests. This can cause flakiness in
ScudoWrappersCTest.MallInfo, because the creation or teardown of a batch
causes mallinfo's uordblks or fordblks to move up or down by the size of
a transfer batch on malloc/free.

It's my opinion that uordblks and fordblks should track the statistics
related to the user's malloc() and free() usage, and not the state of
the internal allocator structures. Thus, excluding the transfer batches
from stat collection does the trick and makes these tests pass.

Repro instructions of the bug:
 1. ninja ./projects/compiler-rt/lib/scudo/standalone/tests/ScudoCUnitTest-x86_64-Test
 2. ./projects/compiler-rt/lib/scudo/standalone/tests/ScudoCUnitTest-x86_64-Test --gtest_filter=ScudoWrappersCTest.MallInfo

Reviewed By: cryptoad

Differential Revision: https://reviews.llvm.org/D101653
compiler-rt/lib/scudo/standalone/local_cache.h