diff options
author | bunnei <bunneidev@gmail.com> | 2021-08-27 11:23:01 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-08-27 11:23:01 +0200 |
commit | c20ea8939032a7141f22607ad16b8aad5e7a3251 (patch) | |
tree | d6f7d065cd8e2f00c85b9749076ef303a705c864 /src/common | |
parent | Merge pull request #6870 from yzct12345/trace-back-stack-back-stack-back (diff) | |
parent | Revert "logging: Display backtrace on crash" (diff) | |
download | yuzu-c20ea8939032a7141f22607ad16b8aad5e7a3251.tar yuzu-c20ea8939032a7141f22607ad16b8aad5e7a3251.tar.gz yuzu-c20ea8939032a7141f22607ad16b8aad5e7a3251.tar.bz2 yuzu-c20ea8939032a7141f22607ad16b8aad5e7a3251.tar.lz yuzu-c20ea8939032a7141f22607ad16b8aad5e7a3251.tar.xz yuzu-c20ea8939032a7141f22607ad16b8aad5e7a3251.tar.zst yuzu-c20ea8939032a7141f22607ad16b8aad5e7a3251.zip |
Diffstat (limited to '')
-rw-r--r-- | src/common/CMakeLists.txt | 3 | ||||
-rw-r--r-- | src/common/logging/backend.cpp | 112 |
2 files changed, 1 insertions, 114 deletions
diff --git a/src/common/CMakeLists.txt b/src/common/CMakeLists.txt index 316c4dedc..57922b51c 100644 --- a/src/common/CMakeLists.txt +++ b/src/common/CMakeLists.txt @@ -176,6 +176,3 @@ if (MSVC) else() target_link_libraries(common PRIVATE zstd) endif() -if (${CMAKE_SYSTEM_NAME} STREQUAL "Linux" AND CMAKE_CXX_COMPILER_ID STREQUAL GNU) - target_link_libraries(common PRIVATE backtrace) -endif() diff --git a/src/common/logging/backend.cpp b/src/common/logging/backend.cpp index 6f1d251e1..949384fd3 100644 --- a/src/common/logging/backend.cpp +++ b/src/common/logging/backend.cpp @@ -13,14 +13,6 @@ #include <windows.h> // For OutputDebugStringW #endif -#if defined(__linux__) && defined(__GNUG__) && !defined(__clang__) -#define BOOST_STACKTRACE_USE_BACKTRACE -#include <boost/stacktrace.hpp> -#undef BOOST_STACKTRACE_USE_BACKTRACE -#include <signal.h> -#define YUZU_LINUX_GCC_BACKTRACE -#endif - #include "common/fs/file.h" #include "common/fs/fs.h" #include "common/fs/fs_paths.h" @@ -163,14 +155,6 @@ public: bool initialization_in_progress_suppress_logging = true; -#ifdef YUZU_LINUX_GCC_BACKTRACE -[[noreturn]] void SleepForever() { - while (true) { - pause(); - } -} -#endif - /** * Static state as a singleton. */ @@ -242,66 +226,9 @@ private: while (max_logs_to_write-- && message_queue.Pop(entry)) { write_logs(); } - })} { -#ifdef YUZU_LINUX_GCC_BACKTRACE - int waker_pipefd[2]; - int done_printing_pipefd[2]; - if (pipe2(waker_pipefd, O_CLOEXEC) || pipe2(done_printing_pipefd, O_CLOEXEC)) { - abort(); - } - backtrace_thread_waker_fd = waker_pipefd[1]; - backtrace_done_printing_fd = done_printing_pipefd[0]; - std::thread([this, wait_fd = waker_pipefd[0], done_fd = done_printing_pipefd[1]] { - Common::SetCurrentThreadName("yuzu:Crash"); - for (u8 ignore = 0; read(wait_fd, &ignore, 1) != 1;) - ; - const int sig = received_signal; - if (sig <= 0) { - abort(); - } - StopBackendThread(); - const auto signal_entry = - CreateEntry(Class::Log, Level::Critical, "?", 0, "?", - fmt::vformat("Received signal {}", fmt::make_format_args(sig))); - ForEachBackend([&signal_entry](Backend& backend) { - backend.EnableForStacktrace(); - backend.Write(signal_entry); - }); - const auto backtrace = - boost::stacktrace::stacktrace::from_dump(backtrace_storage.data(), 4096); - for (const auto& frame : backtrace.as_vector()) { - auto line = boost::stacktrace::detail::to_string(&frame, 1); - if (line.empty()) { - abort(); - } - line.pop_back(); // Remove newline - const auto frame_entry = - CreateEntry(Class::Log, Level::Critical, "?", 0, "?", line); - ForEachBackend([&frame_entry](Backend& backend) { backend.Write(frame_entry); }); - } - using namespace std::literals; - const auto rip_entry = CreateEntry(Class::Log, Level::Critical, "?", 0, "?", "RIP"s); - ForEachBackend([&rip_entry](Backend& backend) { - backend.Write(rip_entry); - backend.Flush(); - }); - for (const u8 anything = 0; write(done_fd, &anything, 1) != 1;) - ; - // Abort on original thread to help debugging - SleepForever(); - }).detach(); - signal(SIGSEGV, &HandleSignal); - signal(SIGABRT, &HandleSignal); -#endif - } + })} {} ~Impl() { -#ifdef YUZU_LINUX_GCC_BACKTRACE - if (int zero_or_ignore = 0; - !received_signal.compare_exchange_strong(zero_or_ignore, SIGKILL)) { - SleepForever(); - } -#endif StopBackendThread(); } @@ -340,36 +267,6 @@ private: delete ptr; } -#ifdef YUZU_LINUX_GCC_BACKTRACE - [[noreturn]] static void HandleSignal(int sig) { - signal(SIGABRT, SIG_DFL); - signal(SIGSEGV, SIG_DFL); - if (sig <= 0) { - abort(); - } - instance->InstanceHandleSignal(sig); - } - - [[noreturn]] void InstanceHandleSignal(int sig) { - if (int zero_or_ignore = 0; !received_signal.compare_exchange_strong(zero_or_ignore, sig)) { - if (received_signal == SIGKILL) { - abort(); - } - SleepForever(); - } - // Don't restart like boost suggests. We want to append to the log file and not lose dynamic - // symbols. This may segfault if it unwinds outside C/C++ code but we'll just have to fall - // back to core dumps. - boost::stacktrace::safe_dump_to(backtrace_storage.data(), 4096); - std::atomic_thread_fence(std::memory_order_seq_cst); - for (const int anything = 0; write(backtrace_thread_waker_fd, &anything, 1) != 1;) - ; - for (u8 ignore = 0; read(backtrace_done_printing_fd, &ignore, 1) != 1;) - ; - abort(); - } -#endif - static inline std::unique_ptr<Impl, decltype(&Deleter)> instance{nullptr, Deleter}; Filter filter; @@ -380,13 +277,6 @@ private: std::thread backend_thread; MPSCQueue<Entry> message_queue{}; std::chrono::steady_clock::time_point time_origin{std::chrono::steady_clock::now()}; - -#ifdef YUZU_LINUX_GCC_BACKTRACE - std::atomic_int received_signal{0}; - std::array<u8, 4096> backtrace_storage{}; - int backtrace_thread_waker_fd; - int backtrace_done_printing_fd; -#endif }; } // namespace |