diff options
author | Gauvain "GovanifY" Roussel-Tarbouriech <gauvain@govanify.com> | 2020-02-23 21:33:49 +0100 |
---|---|---|
committer | FearlessTobi <thm.frey@gmail.com> | 2020-03-17 11:18:13 +0100 |
commit | 38036eb1c8633d151c721992e085e1aca5658f9d (patch) | |
tree | d9866995796f24394134f4568f567343c89af288 /src/core | |
parent | Merge pull request #3521 from ReinUsesLisp/nsight-debug (diff) | |
download | yuzu-38036eb1c8633d151c721992e085e1aca5658f9d.tar yuzu-38036eb1c8633d151c721992e085e1aca5658f9d.tar.gz yuzu-38036eb1c8633d151c721992e085e1aca5658f9d.tar.bz2 yuzu-38036eb1c8633d151c721992e085e1aca5658f9d.tar.lz yuzu-38036eb1c8633d151c721992e085e1aca5658f9d.tar.xz yuzu-38036eb1c8633d151c721992e085e1aca5658f9d.tar.zst yuzu-38036eb1c8633d151c721992e085e1aca5658f9d.zip |
Diffstat (limited to 'src/core')
-rw-r--r-- | src/core/core.cpp | 2 | ||||
-rw-r--r-- | src/core/gdbstub/gdbstub.cpp | 9 | ||||
-rw-r--r-- | src/core/gdbstub/gdbstub.h | 7 |
3 files changed, 16 insertions, 2 deletions
diff --git a/src/core/core.cpp b/src/core/core.cpp index 218508126..d1bc9340d 100644 --- a/src/core/core.cpp +++ b/src/core/core.cpp @@ -166,7 +166,7 @@ struct System::Impl { service_manager = std::make_shared<Service::SM::ServiceManager>(); Service::Init(service_manager, system); - GDBStub::Init(); + GDBStub::DeferStart(); renderer = VideoCore::CreateRenderer(emu_window, system); if (!renderer->Init()) { diff --git a/src/core/gdbstub/gdbstub.cpp b/src/core/gdbstub/gdbstub.cpp index e8d8871a7..6ee4df6b5 100644 --- a/src/core/gdbstub/gdbstub.cpp +++ b/src/core/gdbstub/gdbstub.cpp @@ -141,6 +141,7 @@ constexpr char target_xml[] = )"; int gdbserver_socket = -1; +bool defer_start = false; u8 command_buffer[GDB_BUFFER_SIZE]; u32 command_length; @@ -1165,7 +1166,8 @@ static void RemoveBreakpoint() { } void HandlePacket() { - if (!IsConnected()) { + if (!IsConnected() && defer_start) { + ToggleServer(true); return; } @@ -1256,6 +1258,10 @@ void ToggleServer(bool status) { } } +void DeferStart() { + defer_start = true; +} + static void Init(u16 port) { if (!server_enabled) { // Set the halt loop to false in case the user enabled the gdbstub mid-execution. @@ -1341,6 +1347,7 @@ void Shutdown() { if (!server_enabled) { return; } + defer_start = false; LOG_INFO(Debug_GDBStub, "Stopping GDB ..."); if (gdbserver_socket != -1) { diff --git a/src/core/gdbstub/gdbstub.h b/src/core/gdbstub/gdbstub.h index 5a36524b2..8fe3c320b 100644 --- a/src/core/gdbstub/gdbstub.h +++ b/src/core/gdbstub/gdbstub.h @@ -43,6 +43,13 @@ void ToggleServer(bool status); /// Start the gdbstub server. void Init(); +/** + * Defer initialization of the gdbstub to the first packet processing functions. + * This avoids a case where the gdbstub thread is frozen after initialization + * and fails to respond in time to packets. + */ +void DeferStart(); + /// Stop gdbstub server. void Shutdown(); |