diff options
author | Mathieu Chartier <mathieuc@google.com> | 2014-04-28 09:35:55 -0700 |
---|---|---|
committer | Mathieu Chartier <mathieuc@google.com> | 2014-04-28 09:49:07 -0700 |
commit | 4fd2050ff1da3892b5e79276f831e09b2f8e5bc9 (patch) | |
tree | 71e31fe365dcd0fd124073ef626ad4365dce6996 /runtime/gc | |
parent | 5dee5df89aa2cefef6c886d5b9b642cc6f1c595b (diff) | |
download | art-4fd2050ff1da3892b5e79276f831e09b2f8e5bc9.zip art-4fd2050ff1da3892b5e79276f831e09b2f8e5bc9.tar.gz art-4fd2050ff1da3892b5e79276f831e09b2f8e5bc9.tar.bz2 |
Fix racy DCHECKS.
Added a size bracket lock to fix a race condition where we were
inserting into full_runs_ and non_full_runs_ holding a lock but
reading without holding a lock from a different thread.
Bug: 14326370
Change-Id: I5c492bddc4b9927e4a36603f3d787b046961675d
Diffstat (limited to 'runtime/gc')
-rw-r--r-- | runtime/gc/allocator/rosalloc.cc | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/runtime/gc/allocator/rosalloc.cc b/runtime/gc/allocator/rosalloc.cc index 821aa2d..ff59016 100644 --- a/runtime/gc/allocator/rosalloc.cc +++ b/runtime/gc/allocator/rosalloc.cc @@ -600,8 +600,12 @@ void* RosAlloc::AllocFromRun(Thread* self, size_t size, size_t* bytes_allocated) // Use a thread-local run. Run* thread_local_run = reinterpret_cast<Run*>(self->GetRosAllocRun(idx)); // Allow invalid since this will always fail the allocation. - DCHECK(non_full_runs_[idx].find(thread_local_run) == non_full_runs_[idx].end()); - DCHECK(full_runs_[idx].find(thread_local_run) == full_runs_[idx].end()); + if (kIsDebugBuild) { + // Need the lock to prevent race conditions. + MutexLock mu(self, *size_bracket_locks_[idx]); + CHECK(non_full_runs_[idx].find(thread_local_run) == non_full_runs_[idx].end()); + CHECK(full_runs_[idx].find(thread_local_run) == full_runs_[idx].end()); + } DCHECK(thread_local_run != nullptr); DCHECK(thread_local_run->IsThreadLocal() || thread_local_run == dedicated_full_run_); slot_addr = thread_local_run->AllocSlot(); |