This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[PATCH] Call debug_exit in linux_wait_1
- From: Yao Qi <qiyaoltc at gmail dot com>
- To: gdb-patches at sourceware dot org
- Date: Thu, 22 Sep 2016 15:32:15 +0100
- Subject: [PATCH] Call debug_exit in linux_wait_1
- Authentication-results: sourceware.org; auth=none
When I read the GDBserver debug message, I find the "entering" of
linux_wait_1 doesn't match the "existing" of linux_wait_1. Looks
we don't call debug_exit somewhere in linux_wait_1 on return.
gdb/gdbserver:
2016-09-22 Yao Qi <yao.qi@linaro.org>
* linux-low.c (linux_wait_1): Call debug_exit.
---
gdb/gdbserver/linux-low.c | 13 +++++++++++++
1 file changed, 13 insertions(+)
diff --git a/gdb/gdbserver/linux-low.c b/gdb/gdbserver/linux-low.c
index 785f0c2..dd71b59 100644
--- a/gdb/gdbserver/linux-low.c
+++ b/gdb/gdbserver/linux-low.c
@@ -3452,6 +3452,8 @@ linux_wait_1 (ptid_t ptid,
linux_resume_one_lwp (event_child, 0, 0, NULL);
+ if (debug_threads)
+ debug_exit ();
return ignore_event (ourstatus);
}
}
@@ -3547,6 +3549,9 @@ linux_wait_1 (ptid_t ptid,
linux_resume_one_lwp (event_child, event_child->stepping,
0, NULL);
+
+ if (debug_threads)
+ debug_exit ();
return ignore_event (ourstatus);
}
@@ -3602,6 +3607,10 @@ linux_wait_1 (ptid_t ptid,
linux_resume_one_lwp (event_child, event_child->stepping,
WSTOPSIG (w), info_p);
}
+
+ if (debug_threads)
+ debug_exit ();
+
return ignore_event (ourstatus);
}
@@ -3696,6 +3705,10 @@ linux_wait_1 (ptid_t ptid,
if (debug_threads)
debug_printf ("proceeding all threads.\n");
proceed_all_lwps ();
+
+ if (debug_threads)
+ debug_exit ();
+
return ignore_event (ourstatus);
}
--
1.9.1