diff options
| author | Mathieu Chartier <mathieuc@google.com> | 2016-03-11 17:22:23 -0800 |
|---|---|---|
| committer | Mathieu Chartier <mathieuc@google.com> | 2016-03-14 13:08:32 -0700 |
| commit | 14b0a5ddc0ae305e3ac152c34e4d6fdd0abb854c (patch) | |
| tree | 66f378cf59ddee476981de4828461f62bceafa9f /compiler/optimizing/graph_visualizer.cc | |
| parent | 0b2c1922cc29a7939f747f60d80240a9fb22547c (diff) | |
Fix cases where we miss instrumentation changes
Moved allocation stack push to after we record the allocation since
it can cause thread suspension.
Added handling in entrypoint utils for thread suspension cases.
Keep the AllocRecordObjectMap around since we do not want to delete
it if there are any threads waiting on new_record_condition_. The
condition guards adding stack traces while the GC is running. If we
delete the map and there are still waiters that did not resume, it
caused a CHECK failure. This could happen in cases where one thread
disables allocation tracking while other threads are about to
resume from the condition.
Bug: 27506909
(cherry picked from commit 0f394021d6abfdf9ebea6c7f405ec936a812ea62)
Change-Id: I1dc51e9f000684b4032b57beab59d317ece26f06
Diffstat (limited to 'compiler/optimizing/graph_visualizer.cc')
0 files changed, 0 insertions, 0 deletions