[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v1 7/8] gdbstub: tidy away reverse debugging check into function
From: |
Alex Bennée |
Subject: |
[PATCH v1 7/8] gdbstub: tidy away reverse debugging check into function |
Date: |
Thu, 20 May 2021 18:43:02 +0100 |
In theory we don't need an actual record/replay to enact reverse
debugging on a purely deterministic system (i.e one with no external
inputs running under icount). Tidy away the logic into a little
function.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
gdbstub.c | 13 +++++++++++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/gdbstub.c b/gdbstub.c
index 84ce770a04..52bde5bdc9 100644
--- a/gdbstub.c
+++ b/gdbstub.c
@@ -465,6 +465,15 @@ int use_gdb_syscalls(void)
return gdb_syscall_mode == GDB_SYS_ENABLED;
}
+static bool stub_can_reverse(void)
+{
+#ifdef CONFIG_USER_ONLY
+ return false;
+#else
+ return replay_mode == REPLAY_MODE_PLAY;
+#endif
+}
+
/* Resume execution. */
static inline void gdb_continue(void)
{
@@ -1880,7 +1889,7 @@ static void handle_step(GArray *params, void *user_ctx)
static void handle_backward(GArray *params, void *user_ctx)
{
- if (replay_mode != REPLAY_MODE_PLAY) {
+ if (!stub_can_reverse()) {
put_packet("E22");
}
if (params->len == 1) {
@@ -2153,7 +2162,7 @@ static void handle_query_supported(GArray *params, void
*user_ctx)
g_string_append(gdbserver_state.str_buf, ";qXfer:features:read+");
}
- if (replay_mode == REPLAY_MODE_PLAY) {
+ if (stub_can_reverse()) {
g_string_append(gdbserver_state.str_buf,
";ReverseStep+;ReverseContinue+");
}
--
2.20.1
- [PATCH v1 1/8] tests/tcg: add a multiarch signals test to stress test signal delivery, (continued)
- [PATCH v1 1/8] tests/tcg: add a multiarch signals test to stress test signal delivery, Alex Bennée, 2021/05/20
- [PATCH v1 3/8] gitlab: add special rule for the hexagon container, Alex Bennée, 2021/05/20
- [PATCH v1 4/8] gdbstub: Constify GdbCmdParseEntry, Alex Bennée, 2021/05/20
- [PATCH v1 6/8] hmp-commands: expand type of icount to "l" in replay commands, Alex Bennée, 2021/05/20
- [PATCH v1 5/8] gdbstub: Replace GdbCmdContext with plain g_array(), Alex Bennée, 2021/05/20
- [PATCH v1 8/8] plugins/syscall: Added a table-like summary output, Alex Bennée, 2021/05/20
- [PATCH v1 7/8] gdbstub: tidy away reverse debugging check into function,
Alex Bennée <=
- Re: [PATCH v1 0/8] various misc fixes (gitlab, gdbstub, plugins), Alex Bennée, 2021/05/24