diff options
| author | 2016-03-07 10:25:04 -0800 | |
|---|---|---|
| committer | 2016-03-07 13:14:45 -0800 | |
| commit | bcd9dd7d1223eaa60f188d1a109d26e5e0143e10 (patch) | |
| tree | b9b121de21cbc0a97cf9138a6f66d3df4b1aee65 /runtime/gc/heap.cc | |
| parent | 810ef3768021b5fc7b47e663e037e49c46f8c916 (diff) | |
Don't use ScopedObjectAccess in Heap::DumpSpaces
ScopedObjectAccess does not work well if the mutator lock is
excusively held while in a suspend thread state. This caused
deadlocks and DCHECK failures.
Bug: 27493316
(cherry picked from commit 03d21bc5bed887243ff6ce3531179185ffd3532c)
Change-Id: I5d67f74fc7082761e45dc1d7778b0ea7fceaaf8f
Diffstat (limited to 'runtime/gc/heap.cc')
| -rw-r--r-- | runtime/gc/heap.cc | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/runtime/gc/heap.cc b/runtime/gc/heap.cc index 3480483c34..faa3d3bc14 100644 --- a/runtime/gc/heap.cc +++ b/runtime/gc/heap.cc @@ -1545,7 +1545,6 @@ std::string Heap::DumpSpaces() const { } void Heap::DumpSpaces(std::ostream& stream) const { - ScopedObjectAccess soa(Thread::Current()); for (const auto& space : continuous_spaces_) { accounting::ContinuousSpaceBitmap* live_bitmap = space->GetLiveBitmap(); accounting::ContinuousSpaceBitmap* mark_bitmap = space->GetMarkBitmap(); |