From c7acb87bc602c78e6cbee7a7d73f7bdfeeddeefb Mon Sep 17 00:00:00 2001 From: Simon Marchi Date: Fri, 4 Dec 2020 16:43:55 -0500 Subject: [PATCH] gdb: move displaced stepping types to displaced-stepping.{h,c} Move displaced-stepping related stuff unchanged to displaced-stepping.h and displaced-stepping.c. This helps make the following patch a bit smaller and easier to read. gdb/ChangeLog: * Makefile.in (COMMON_SFILES): Add displaced-stepping.c. * aarch64-tdep.h: Include displaced-stepping.h. * displaced-stepping.h (struct displaced_step_copy_insn_closure): Move here. (displaced_step_copy_insn_closure_up): Move here. (struct buf_displaced_step_copy_insn_closure): Move here. (struct displaced_step_inferior_state): Move here. (debug_displaced): Move here. (displaced_debug_printf_1): Move here. (displaced_debug_printf): Move here. * displaced-stepping.c: New file. * gdbarch.sh: Include displaced-stepping.h in gdbarch.h. * gdbarch.h: Re-generate. * inferior.h: Include displaced-stepping.h. * infrun.h (debug_displaced): Move to displaced-stepping.h. (displaced_debug_printf_1): Likewise. (displaced_debug_printf): Likewise. (struct displaced_step_copy_insn_closure): Likewise. (displaced_step_copy_insn_closure_up): Likewise. (struct buf_displaced_step_copy_insn_closure): Likewise. (struct displaced_step_inferior_state): Likewise. * infrun.c (show_debug_displaced): Move to displaced-stepping.c. (displaced_debug_printf_1): Likewise. (displaced_step_copy_insn_closure::~displaced_step_copy_insn_closure): Likewise. (_initialize_infrun): Don't register "set/show debug displaced". Change-Id: I29935f5959b80425370630a45148fc06cd4227ca --- gdb/ChangeLog | 29 ++++++++++++++ gdb/Makefile.in | 1 + gdb/aarch64-tdep.h | 1 + gdb/displaced-stepping.c | 52 +++++++++++++++++++++++++ gdb/displaced-stepping.h | 84 ++++++++++++++++++++++++++++++++++++++++ gdb/gdbarch.h | 1 + gdb/gdbarch.sh | 1 + gdb/inferior.h | 1 + gdb/infrun.c | 22 ----------- gdb/infrun.h | 78 ------------------------------------- 10 files changed, 170 insertions(+), 100 deletions(-) create mode 100644 gdb/displaced-stepping.c diff --git a/gdb/ChangeLog b/gdb/ChangeLog index f1a292258d4..8512cb449e9 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,32 @@ +2020-12-04 Simon Marchi + + * Makefile.in (COMMON_SFILES): Add displaced-stepping.c. + * aarch64-tdep.h: Include displaced-stepping.h. + * displaced-stepping.h (struct displaced_step_copy_insn_closure): + Move here. + (displaced_step_copy_insn_closure_up): Move here. + (struct buf_displaced_step_copy_insn_closure): Move here. + (struct displaced_step_inferior_state): Move here. + (debug_displaced): Move here. + (displaced_debug_printf_1): Move here. + (displaced_debug_printf): Move here. + * displaced-stepping.c: New file. + * gdbarch.sh: Include displaced-stepping.h in gdbarch.h. + * gdbarch.h: Re-generate. + * inferior.h: Include displaced-stepping.h. + * infrun.h (debug_displaced): Move to displaced-stepping.h. + (displaced_debug_printf_1): Likewise. + (displaced_debug_printf): Likewise. + (struct displaced_step_copy_insn_closure): Likewise. + (displaced_step_copy_insn_closure_up): Likewise. + (struct buf_displaced_step_copy_insn_closure): Likewise. + (struct displaced_step_inferior_state): Likewise. + * infrun.c (show_debug_displaced): Move to displaced-stepping.c. + (displaced_debug_printf_1): Likewise. + (displaced_step_copy_insn_closure::~displaced_step_copy_insn_closure): + Likewise. + (_initialize_infrun): Don't register "set/show debug displaced". + 2020-12-04 Simon Marchi * linux-tdep.c (get_linux_inferior_data): Add inferior diff --git a/gdb/Makefile.in b/gdb/Makefile.in index a86e8d648e6..997c99c30f2 100644 --- a/gdb/Makefile.in +++ b/gdb/Makefile.in @@ -1019,6 +1019,7 @@ COMMON_SFILES = \ debuginfod-support.c \ dictionary.c \ disasm.c \ + displaced-stepping.c \ dummy-frame.c \ dwarf2/abbrev.c \ dwarf2/attribute.c \ diff --git a/gdb/aarch64-tdep.h b/gdb/aarch64-tdep.h index 895aa5977f0..76ff812abc6 100644 --- a/gdb/aarch64-tdep.h +++ b/gdb/aarch64-tdep.h @@ -23,6 +23,7 @@ #define AARCH64_TDEP_H #include "arch/aarch64.h" +#include "displaced-stepping.h" #include "infrun.h" /* Forward declarations. */ diff --git a/gdb/displaced-stepping.c b/gdb/displaced-stepping.c new file mode 100644 index 00000000000..5ae280fac39 --- /dev/null +++ b/gdb/displaced-stepping.c @@ -0,0 +1,52 @@ +/* Displaced stepping related things. + + Copyright (C) 2020 Free Software Foundation, Inc. + + This file is part of GDB. + + This program is free software; you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation; either version 3 of the License, or + (at your option) any later version. + + This program is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with this program. If not, see . */ + +#include "defs.h" +#include "displaced-stepping.h" +#include "cli/cli-cmds.h" + +#include "command.h" + +/* Default destructor for displaced_step_copy_insn_closure. */ + +displaced_step_copy_insn_closure::~displaced_step_copy_insn_closure () + = default; + +bool debug_displaced = false; + +static void +show_debug_displaced (struct ui_file *file, int from_tty, + struct cmd_list_element *c, const char *value) +{ + fprintf_filtered (file, _("Displace stepping debugging is %s.\n"), value); +} + +void _initialize_displaced_stepping (); +void +_initialize_displaced_stepping () +{ + add_setshow_boolean_cmd ("displaced", class_maintenance, + &debug_displaced, _("\ +Set displaced stepping debugging."), _("\ +Show displaced stepping debugging."), _("\ +When non-zero, displaced stepping specific debugging is enabled."), + NULL, + show_debug_displaced, + &setdebuglist, &showdebuglist); +} diff --git a/gdb/displaced-stepping.h b/gdb/displaced-stepping.h index 44aca74111d..928bfc0ccc9 100644 --- a/gdb/displaced-stepping.h +++ b/gdb/displaced-stepping.h @@ -20,6 +20,24 @@ #ifndef DISPLACED_STEPPING_H #define DISPLACED_STEPPING_H +#include "gdbsupport/byte-vector.h" + +struct thread_info; + +/* True if we are debugging displaced stepping. */ + +extern bool debug_displaced; + +/* Print a "displaced" debug statement. */ + +#define displaced_debug_printf(fmt, ...) \ + do \ + { \ + if (debug_displaced) \ + debug_prefixed_printf ("displaced", __func__, fmt, ##__VA_ARGS__); \ + } \ + while (0) + enum displaced_step_prepare_status { /* A displaced stepping buffer was successfully allocated and prepared. */ @@ -44,4 +62,70 @@ enum displaced_step_finish_status DISPLACED_STEP_FINISH_STATUS_NOT_EXECUTED, }; +/* Base class for displaced stepping closures (the arch-specific data). */ + +struct displaced_step_copy_insn_closure +{ + virtual ~displaced_step_copy_insn_closure () = 0; +}; + +using displaced_step_copy_insn_closure_up + = std::unique_ptr; + +/* A simple displaced step closure that contains only a byte buffer. */ + +struct buf_displaced_step_copy_insn_closure : displaced_step_copy_insn_closure +{ + buf_displaced_step_copy_insn_closure (int buf_size) + : buf (buf_size) + {} + + gdb::byte_vector buf; +}; + +/* Per-inferior displaced stepping state. */ + +struct displaced_step_inferior_state +{ + displaced_step_inferior_state () + { + reset (); + } + + /* Put this object back in its original state. */ + void reset () + { + failed_before = 0; + step_thread = nullptr; + step_gdbarch = nullptr; + step_closure.reset (); + step_original = 0; + step_copy = 0; + step_saved_copy.clear (); + } + + /* True if preparing a displaced step ever failed. If so, we won't + try displaced stepping for this inferior again. */ + int failed_before; + + /* If this is not nullptr, this is the thread carrying out a + displaced single-step in process PID. This thread's state will + require fixing up once it has completed its step. */ + thread_info *step_thread; + + /* The architecture the thread had when we stepped it. */ + gdbarch *step_gdbarch; + + /* The closure provided gdbarch_displaced_step_copy_insn, to be used + for post-step cleanup. */ + displaced_step_copy_insn_closure_up step_closure; + + /* The address of the original instruction, and the copy we + made. */ + CORE_ADDR step_original, step_copy; + + /* Saved contents of copy area. */ + gdb::byte_vector step_saved_copy; +}; + #endif /* DISPLACED_STEPPING_H */ diff --git a/gdb/gdbarch.h b/gdb/gdbarch.h index 4f52e8cd874..2b6876a1916 100644 --- a/gdb/gdbarch.h +++ b/gdb/gdbarch.h @@ -31,6 +31,7 @@ #include "gdb_obstack.h" #include "infrun.h" #include "osabi.h" +#include "displaced-stepping.h" struct floatformat; struct ui_file; diff --git a/gdb/gdbarch.sh b/gdb/gdbarch.sh index f67ec9dcd2d..76ce2ff5efb 100755 --- a/gdb/gdbarch.sh +++ b/gdb/gdbarch.sh @@ -1270,6 +1270,7 @@ cat <; - -/* A simple displaced step closure that contains only a byte buffer. */ - -struct buf_displaced_step_copy_insn_closure : displaced_step_copy_insn_closure -{ - buf_displaced_step_copy_insn_closure (int buf_size) - : buf (buf_size) - {} - - gdb::byte_vector buf; -}; - -/* Per-inferior displaced stepping state. */ -struct displaced_step_inferior_state -{ - displaced_step_inferior_state () - { - reset (); - } - - /* Put this object back in its original state. */ - void reset () - { - failed_before = 0; - step_thread = nullptr; - step_gdbarch = nullptr; - step_closure.reset (); - step_original = 0; - step_copy = 0; - step_saved_copy.clear (); - } - - /* True if preparing a displaced step ever failed. If so, we won't - try displaced stepping for this inferior again. */ - int failed_before; - - /* If this is not nullptr, this is the thread carrying out a - displaced single-step in process PID. This thread's state will - require fixing up once it has completed its step. */ - thread_info *step_thread; - - /* The architecture the thread had when we stepped it. */ - gdbarch *step_gdbarch; - - /* The closure provided gdbarch_displaced_step_copy_insn, to be used - for post-step cleanup. */ - displaced_step_copy_insn_closure_up step_closure; - - /* The address of the original instruction, and the copy we - made. */ - CORE_ADDR step_original, step_copy; - - /* Saved contents of copy area. */ - gdb::byte_vector step_saved_copy; -}; - #endif /* INFRUN_H */ -- 2.30.2