From 3214710036436191840e86611082f58ced78fd09 Mon Sep 17 00:00:00 2001 From: Michael Lotz Date: Sat, 25 Jun 2011 22:48:38 +0000 Subject: [PATCH] Use sInDebugger instead of sDebuggerOnCPU to determine if the debugger is running. The former has a broader scope and lasts until the debugger exit is actually done whereas the latter is already reset when the inner loop is exited. This fixes the issue Ingo saw where the USB physical memory manager wasn't able to free resources used for the debug transfer. It has reserved debug memory that it uses depending on debug_debugger_running() and was therefore confused when it returned false when called from the kernel debugger module exit hook. git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@42319 a95241bf-73f2-0310-859d-f6bbb57e9c96 --- src/system/kernel/debug/debug.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/system/kernel/debug/debug.cpp b/src/system/kernel/debug/debug.cpp index 46d9a84318..8340ede362 100644 --- a/src/system/kernel/debug/debug.cpp +++ b/src/system/kernel/debug/debug.cpp @@ -1617,7 +1617,7 @@ debug_stop_screen_debug_output(void) bool debug_debugger_running(void) { - return sDebuggerOnCPU != -1; + return sInDebugger > 0; }