diff options
author | 2019-05-31 13:07:26 -0700 | |
---|---|---|
committer | 2019-06-03 17:20:21 +0000 | |
commit | ba26b51e2e74bf8c4f87e99dc60f1cbca033ecac (patch) | |
tree | a3fa191885db25da6f86cf8efa64ce9de03414e2 /runtime/runtime_test.cc | |
parent | e8e2d00ec05ae561ebc7729cc7029a1dc1e02ee6 (diff) |
ART: Skip all-threads dump on abort if locks are held
The thread-list locks are not recursive. Skip the dump on abort
if the locks are already held. Thread state will still be captured
on-device by a tombstone.
Bug: 134037466
Bug: 134167395
Test: m
Test: m test-art-host-gtest-runtime_test
Change-Id: I15ba38a018a39f1cc12577cdf8fb94876b96bbaa
Diffstat (limited to 'runtime/runtime_test.cc')
-rw-r--r-- | runtime/runtime_test.cc | 43 |
1 files changed, 43 insertions, 0 deletions
diff --git a/runtime/runtime_test.cc b/runtime/runtime_test.cc new file mode 100644 index 0000000000..aa4020e25f --- /dev/null +++ b/runtime/runtime_test.cc @@ -0,0 +1,43 @@ +/* + * Copyright (C) 2019 The Android Open Source Project + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +#include "android-base/logging.h" +#include "base/locks.h" +#include "base/mutex.h" +#include "common_runtime_test.h" +#include "runtime.h" +#include "thread-current-inl.h" + +namespace art { + +class RuntimeTest : public CommonRuntimeTest {}; + +// Ensure that abort works with ThreadList locks held. + +TEST_F(RuntimeTest, AbortWithThreadListLockHeld) { + // This assumes the test is run single-threaded: do not start the runtime to avoid daemon threads. + + constexpr const char* kDeathRegex = "Skipping all-threads dump as locks are held"; + ASSERT_DEATH({ + // The regex only works if we can ensure output goes to stderr. + android::base::SetLogger(android::base::StderrLogger); + + MutexLock mu(Thread::Current(), *Locks::thread_list_lock_); + Runtime::Abort("Attempt to abort"); + }, kDeathRegex); +} + +} // namespace art |