Let threads opt in to having their stack traces printed
The video decoder thread is the pilot user.
For now this is an Android-only feature, since that's the only
platform we can print stack traces on.
Bug: webrtc:9987
Change-Id: Ie638c619673b5f159d91a32683fd787baf46479a
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/126222
Reviewed-by: Magnus Jedvert <magjed@webrtc.org>
Commit-Queue: Karl Wiberg <kwiberg@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#27127}diff --git a/rtc_base/system/BUILD.gn b/rtc_base/system/BUILD.gn
index 8b83d04..6448348 100644
--- a/rtc_base/system/BUILD.gn
+++ b/rtc_base/system/BUILD.gn
@@ -78,3 +78,19 @@
libs = [ "Foundation.framework" ]
}
}
+
+rtc_source_set("thread_registry") {
+ sources = [
+ "thread_registry.h",
+ ]
+ deps = [
+ "..:rtc_base_approved",
+ ]
+ if (is_android && !build_with_chromium) {
+ sources += [ "thread_registry.cc" ]
+ deps += [
+ "../../sdk/android:native_api_stacktrace",
+ "//third_party/abseil-cpp/absl/base:core_headers",
+ ]
+ }
+}
diff --git a/rtc_base/system/DEPS b/rtc_base/system/DEPS
new file mode 100644
index 0000000..09d6c93
--- /dev/null
+++ b/rtc_base/system/DEPS
@@ -0,0 +1,5 @@
+specific_include_rules = {
+ "thread_registry\.cc": [
+ "+sdk/android/native_api/stacktrace/stacktrace.h",
+ ],
+}
diff --git a/rtc_base/system/thread_registry.cc b/rtc_base/system/thread_registry.cc
new file mode 100644
index 0000000..8d7cd58
--- /dev/null
+++ b/rtc_base/system/thread_registry.cc
@@ -0,0 +1,71 @@
+/*
+ * Copyright 2019 The WebRTC Project Authors. All rights reserved.
+ *
+ * Use of this source code is governed by a BSD-style license
+ * that can be found in the LICENSE file in the root of the source
+ * tree. An additional intellectual property rights grant can be found
+ * in the file PATENTS. All contributing project authors may
+ * be found in the AUTHORS file in the root of the source tree.
+ */
+
+#include "rtc_base/system/thread_registry.h"
+
+#include <map>
+#include <utility>
+
+#include "absl/base/attributes.h"
+#include "rtc_base/critical_section.h"
+#include "rtc_base/logging.h"
+#include "rtc_base/platform_thread_types.h"
+#include "sdk/android/native_api/stacktrace/stacktrace.h"
+
+namespace webrtc {
+
+namespace {
+
+struct ThreadData {
+ const rtc::PlatformThreadId thread_id;
+ const rtc::Location location;
+};
+
+// The map of registered threads, and the lock that protects it. We create the
+// map on first use, and never destroy it.
+ABSL_CONST_INIT rtc::GlobalLockPod g_thread_registry_lock = {};
+ABSL_CONST_INIT std::map<const ScopedRegisterThreadForDebugging*, ThreadData>*
+ g_registered_threads = nullptr;
+
+} // namespace
+
+ScopedRegisterThreadForDebugging::ScopedRegisterThreadForDebugging(
+ rtc::Location location) {
+ rtc::GlobalLockScope gls(&g_thread_registry_lock);
+ if (g_registered_threads == nullptr) {
+ g_registered_threads =
+ new std::map<const ScopedRegisterThreadForDebugging*, ThreadData>();
+ }
+ const auto result = g_registered_threads->insert(
+ std::make_pair(this, ThreadData{rtc::CurrentThreadId(), location}));
+ RTC_DCHECK(result.second); // Insertion succeeded without collisions.
+}
+
+ScopedRegisterThreadForDebugging::~ScopedRegisterThreadForDebugging() {
+ rtc::GlobalLockScope gls(&g_thread_registry_lock);
+ RTC_DCHECK(g_registered_threads != nullptr);
+ const int num_erased = g_registered_threads->erase(this);
+ RTC_DCHECK_EQ(num_erased, 1);
+}
+
+void PrintStackTracesOfRegisteredThreads() {
+ rtc::GlobalLockScope gls(&g_thread_registry_lock);
+ if (g_registered_threads == nullptr) {
+ return;
+ }
+ for (const auto& e : *g_registered_threads) {
+ const ThreadData& td = e.second;
+ RTC_LOG(LS_WARNING) << "Thread " << td.thread_id << " registered at "
+ << td.location.ToString() << ":";
+ RTC_LOG(LS_WARNING) << StackTraceToString(GetStackTrace(td.thread_id));
+ }
+}
+
+} // namespace webrtc
diff --git a/rtc_base/system/thread_registry.h b/rtc_base/system/thread_registry.h
new file mode 100644
index 0000000..0e3187b
--- /dev/null
+++ b/rtc_base/system/thread_registry.h
@@ -0,0 +1,46 @@
+/*
+ * Copyright 2019 The WebRTC Project Authors. All rights reserved.
+ *
+ * Use of this source code is governed by a BSD-style license
+ * that can be found in the LICENSE file in the root of the source
+ * tree. An additional intellectual property rights grant can be found
+ * in the file PATENTS. All contributing project authors may
+ * be found in the AUTHORS file in the root of the source tree.
+ */
+
+#ifndef RTC_BASE_SYSTEM_THREAD_REGISTRY_H_
+#define RTC_BASE_SYSTEM_THREAD_REGISTRY_H_
+
+#include "rtc_base/location.h"
+
+namespace webrtc {
+
+class ScopedRegisterThreadForDebugging {
+ public:
+#if defined(WEBRTC_ANDROID) && !defined(WEBRTC_CHROMIUM_BUILD)
+ explicit ScopedRegisterThreadForDebugging(rtc::Location location);
+ ~ScopedRegisterThreadForDebugging();
+#else
+ explicit ScopedRegisterThreadForDebugging(rtc::Location) {}
+#endif
+
+ // Not movable or copyable, because we can't duplicate the resource it owns,
+ // and it needs a constant address.
+ ScopedRegisterThreadForDebugging(const ScopedRegisterThreadForDebugging&) =
+ delete;
+ ScopedRegisterThreadForDebugging(ScopedRegisterThreadForDebugging&&) = delete;
+ ScopedRegisterThreadForDebugging& operator=(
+ const ScopedRegisterThreadForDebugging&) = delete;
+ ScopedRegisterThreadForDebugging& operator=(
+ ScopedRegisterThreadForDebugging&&) = delete;
+};
+
+#if defined(WEBRTC_ANDROID) && !defined(WEBRTC_CHROMIUM_BUILD)
+void PrintStackTracesOfRegisteredThreads();
+#else
+inline void PrintStackTracesOfRegisteredThreads() {}
+#endif
+
+} // namespace webrtc
+
+#endif // RTC_BASE_SYSTEM_THREAD_REGISTRY_H_