diff options
| author | Andreas Gampe <agampe@google.com> | 2015-02-27 13:35:39 -0800 |
|---|---|---|
| committer | Andreas Gampe <agampe@google.com> | 2015-02-27 20:12:47 -0800 |
| commit | 00a1f5bfa57bb4007a08435ba83b029dcebde5c0 (patch) | |
| tree | a3d2562b439ba15faafafa60544eb9a342ff1bd8 | |
| parent | 242026e246a8b9efe098a0cce008fd525e011e5b (diff) | |
| download | art-00a1f5bfa57bb4007a08435ba83b029dcebde5c0.zip art-00a1f5bfa57bb4007a08435ba83b029dcebde5c0.tar.gz art-00a1f5bfa57bb4007a08435ba83b029dcebde5c0.tar.bz2 | |
Revert "Revert "Re-enable one thread dumping the native stack of another.""
This reverts commit edfdaf37d5520a3c3a858b6b3fba4f759c60dadc.
Tests seem to succeed. Might be helpful to understand some deadlock/timeout
situations.
Bug: 15446488
Bug: 18713034
Change-Id: I5be225363f698556221576b5827d9501e26799f3
| -rw-r--r-- | runtime/utils.cc | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/runtime/utils.cc b/runtime/utils.cc index 6afc373..fd2f110 100644 --- a/runtime/utils.cc +++ b/runtime/utils.cc @@ -1264,14 +1264,6 @@ void DumpNativeStack(std::ostream& os, pid_t tid, const char* prefix, return; } -#if !defined(HAVE_ANDROID_OS) - if (GetTid() != tid) { - // TODO: dumping of other threads is disabled to avoid crashes during stress testing. - // b/15446488. - return; - } -#endif - std::unique_ptr<Backtrace> backtrace(Backtrace::Create(BACKTRACE_CURRENT_PROCESS, tid)); if (!backtrace->Unwind(0, reinterpret_cast<ucontext*>(ucontext_ptr))) { os << prefix << "(backtrace::Unwind failed for thread " << tid << ")\n"; |