diff options
author | Kostya Kortchinsky <kostyak@google.com> | 2019-01-25 17:23:29 +0000 |
---|---|---|
committer | Kostya Kortchinsky <kostyak@google.com> | 2019-01-25 17:23:29 +0000 |
commit | fca89fee889eadb316fb7a09c5f87b8bbbe9f4ff (patch) | |
tree | c9f9caa45dcb888ed84f5ab15932a2c94f43c268 /test/scudo | |
parent | dd358a8a7ce65eb3189740d99b6a450605947aab (diff) | |
download | compiler-rt-fca89fee889eadb316fb7a09c5f87b8bbbe9f4ff.tar.gz |
[scudo] Delay allocations in the RSS check test
Summary:
D57116 fails on the armv7 bots, which is I assume due to the timing of
the RSS check on the platform. While I don't have a platform to test
that change on, I assume this would do.
The test could be made more reliable by either delaying more the
allocations, or allocating more large-chunks, but both those options
have a somewhat non negligible impact (more memory used, longer test).
Hence me trying to keep the additional sleeping/allocating to a
minimum.
Reviewers: eugenis, yroux
Reviewed By: yroux
Subscribers: javed.absar, kristof.beyls, delcypher, #sanitizers, llvm-commits
Differential Revision: https://reviews.llvm.org/D57241
git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk@352220 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test/scudo')
-rw-r--r-- | test/scudo/rss.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/test/scudo/rss.c b/test/scudo/rss.c index f98bf9536..0b76857b9 100644 --- a/test/scudo/rss.c +++ b/test/scudo/rss.c @@ -30,7 +30,8 @@ static void *allocs[kNumAllocs]; int main(int argc, char *argv[]) { int returned_null = 0; for (int i = 0; i < kNumAllocs; i++) { - if ((i & 0xf) == 0) + // sleep for 100ms every 8 allocations, to allow the RSS check to catch up. + if (i != 0 && (i & 0x7) == 0) usleep(100000); allocs[i] = malloc(kAllocSize); if (allocs[i]) |