aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPedro Alves <palves@redhat.com>2008-07-03 17:38:08 +0000
committerPedro Alves <palves@redhat.com>2008-07-03 17:38:08 +0000
commitdc177b7a5114d1d8385ee3706188aa4e4f675676 (patch)
tree2a6c70e2e00c8e50a75a2dcc03696a8e760a1463
parentcaac88966e74e80449233ec8bc82d9616896a4fa (diff)
downloadgdb-dc177b7a5114d1d8385ee3706188aa4e4f675676.zip
gdb-dc177b7a5114d1d8385ee3706188aa4e4f675676.tar.gz
gdb-dc177b7a5114d1d8385ee3706188aa4e4f675676.tar.bz2
* config/i386/nm-cygwin.h (ATTACH_NO_WAIT): Delete.
* config/i386/nm-i386gnu.h (ATTACH_NO_WAIT): Delete. * target.h (struct target_ops): Add to_attach_no_wait member. (target_attach_no_wait): New. * target.c (update_current_target): Inherit to_attach_no_wait. * infcmd.c: Replace ATTACH_NO_WAIT compile time check by target_attach_no_wait runtime check. * gnu-nat.c (init_gnu_ops): Set to_attach_no_wait in gnu_ops. * win32-nat.c (init_win32_ops): Set to_attach_no_wait in win32_ops.
-rw-r--r--gdb/ChangeLog16
-rw-r--r--gdb/config/i386/nm-cygwin.h1
-rw-r--r--gdb/config/i386/nm-i386gnu.h3
-rw-r--r--gdb/gnu-nat.c1
-rw-r--r--gdb/infcmd.c60
-rw-r--r--gdb/target.c1
-rw-r--r--gdb/target.h8
-rw-r--r--gdb/win32-nat.c1
-rw-r--r--gdb/windows-nat.c1
9 files changed, 59 insertions, 33 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 182a7d6..0cdd935 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,21 @@
2008-07-03 Pedro Alves <pedro@codesourcery.com>
+ * config/i386/nm-cygwin.h (ATTACH_NO_WAIT): Delete.
+ * config/i386/nm-i386gnu.h (ATTACH_NO_WAIT): Delete.
+
+ * target.h (struct target_ops): Add to_attach_no_wait member.
+ (target_attach_no_wait): New.
+ * target.c (update_current_target): Inherit to_attach_no_wait.
+
+ * infcmd.c: Replace ATTACH_NO_WAIT compile time check by
+ target_attach_no_wait runtime check.
+
+ * gnu-nat.c (init_gnu_ops): Set to_attach_no_wait in gnu_ops.
+ * win32-nat.c (init_win32_ops): Set to_attach_no_wait in
+ win32_ops.
+
+2008-07-03 Pedro Alves <pedro@codesourcery.com>
+
* i386-tdep.c (i386_displaced_step_fixup): Condition log printing
on debug_displaced being set.
diff --git a/gdb/config/i386/nm-cygwin.h b/gdb/config/i386/nm-cygwin.h
index 5222b60..b273439 100644
--- a/gdb/config/i386/nm-cygwin.h
+++ b/gdb/config/i386/nm-cygwin.h
@@ -16,7 +16,6 @@
You should have received a copy of the GNU General Public License
along with this program. If not, see <http://www.gnu.org/licenses/>. */
-#define ATTACH_NO_WAIT
#define ADD_SHARED_SYMBOL_FILES dll_symbol_command
void dll_symbol_command (char *, int);
diff --git a/gdb/config/i386/nm-i386gnu.h b/gdb/config/i386/nm-i386gnu.h
index 2be95b4..ee1fe73f 100644
--- a/gdb/config/i386/nm-i386gnu.h
+++ b/gdb/config/i386/nm-i386gnu.h
@@ -26,9 +26,6 @@
extern char *gnu_target_pid_to_str (int pid);
-/* Don't do wait_for_inferior on attach. */
-#define ATTACH_NO_WAIT
-
/* Thread flavors used in re-setting the T bit. */
#define THREAD_STATE_FLAVOR i386_REGS_SEGS_STATE
#define THREAD_STATE_SIZE i386_THREAD_STATE_COUNT
diff --git a/gdb/gnu-nat.c b/gdb/gnu-nat.c
index 5f9a405..558e357 100644
--- a/gdb/gnu-nat.c
+++ b/gdb/gnu-nat.c
@@ -2592,6 +2592,7 @@ init_gnu_ops (void)
gnu_ops.to_doc = "GNU Hurd process"; /* to_doc */
gnu_ops.to_open = gnu_open; /* to_open */
gnu_ops.to_attach = gnu_attach; /* to_attach */
+ gnu_ops.to_attach_no_wait = 1; /* to_attach_no_wait */
gnu_ops.to_detach = gnu_detach; /* to_detach */
gnu_ops.to_resume = gnu_resume; /* to_resume */
gnu_ops.to_wait = gnu_wait; /* to_wait */
diff --git a/gdb/infcmd.c b/gdb/infcmd.c
index d9c9cb7..fead6d1 100644
--- a/gdb/infcmd.c
+++ b/gdb/infcmd.c
@@ -2015,38 +2015,40 @@ attach_command (char *args, int from_tty)
init_wait_for_inferior ();
clear_proceed_status ();
- /* No traps are generated when attaching to inferior under Mach 3
- or GNU hurd. */
-#ifndef ATTACH_NO_WAIT
- /* Careful here. See comments in inferior.h. Basically some OSes
- don't ignore SIGSTOPs on continue requests anymore. We need a
- way for handle_inferior_event to reset the stop_signal variable
- after an attach, and this is what STOP_QUIETLY_NO_SIGSTOP is for. */
- stop_soon = STOP_QUIETLY_NO_SIGSTOP;
-
- if (target_can_async_p ())
+ /* Some system don't generate traps when attaching to inferior.
+ E.g. Mach 3 or GNU hurd. */
+ if (!target_attach_no_wait)
{
- /* sync_execution mode. Wait for stop. */
- struct continuation_arg *arg1, *arg2, *arg3;
+ /* Careful here. See comments in inferior.h. Basically some
+ OSes don't ignore SIGSTOPs on continue requests anymore. We
+ need a way for handle_inferior_event to reset the stop_signal
+ variable after an attach, and this is what
+ STOP_QUIETLY_NO_SIGSTOP is for. */
+ stop_soon = STOP_QUIETLY_NO_SIGSTOP;
- arg1 =
- (struct continuation_arg *) xmalloc (sizeof (struct continuation_arg));
- arg2 =
- (struct continuation_arg *) xmalloc (sizeof (struct continuation_arg));
- arg3 =
- (struct continuation_arg *) xmalloc (sizeof (struct continuation_arg));
- arg1->next = arg2;
- arg2->next = arg3;
- arg3->next = NULL;
- arg1->data.pointer = args;
- arg2->data.integer = from_tty;
- arg3->data.integer = async_exec;
- add_continuation (attach_command_continuation, arg1);
- return;
- }
+ if (target_can_async_p ())
+ {
+ /* sync_execution mode. Wait for stop. */
+ struct continuation_arg *arg1, *arg2, *arg3;
+
+ arg1 =
+ (struct continuation_arg *) xmalloc (sizeof (struct continuation_arg));
+ arg2 =
+ (struct continuation_arg *) xmalloc (sizeof (struct continuation_arg));
+ arg3 =
+ (struct continuation_arg *) xmalloc (sizeof (struct continuation_arg));
+ arg1->next = arg2;
+ arg2->next = arg3;
+ arg3->next = NULL;
+ arg1->data.pointer = args;
+ arg2->data.integer = from_tty;
+ arg3->data.integer = async_exec;
+ add_continuation (attach_command_continuation, arg1);
+ return;
+ }
- wait_for_inferior (0);
-#endif
+ wait_for_inferior (0);
+ }
attach_command_post_wait (args, from_tty, async_exec);
}
diff --git a/gdb/target.c b/gdb/target.c
index 7c1d6fd..af430bd 100644
--- a/gdb/target.c
+++ b/gdb/target.c
@@ -405,6 +405,7 @@ update_current_target (void)
INHERIT (to_close, t);
INHERIT (to_attach, t);
INHERIT (to_post_attach, t);
+ INHERIT (to_attach_no_wait, t);
INHERIT (to_detach, t);
/* Do not inherit to_disconnect. */
INHERIT (to_resume, t);
diff --git a/gdb/target.h b/gdb/target.h
index 9641f0d..a7b2bb9 100644
--- a/gdb/target.h
+++ b/gdb/target.h
@@ -412,6 +412,7 @@ struct target_ops
int to_has_registers;
int to_has_execution;
int to_has_thread_control; /* control thread execution */
+ int to_attach_no_wait;
struct section_table
*to_sections;
struct section_table
@@ -564,6 +565,13 @@ void target_close (struct target_ops *targ, int quitting);
#define target_attach(args, from_tty) \
(*current_target.to_attach) (args, from_tty)
+/* Some targets don't generate traps when attaching to the inferior,
+ or their target_attach implementation takes care of the waiting.
+ These targets must set to_attach_no_wait. */
+
+#define target_attach_no_wait \
+ (current_target.to_attach_no_wait)
+
/* The target_attach operation places a process under debugger control,
and stops the process.
diff --git a/gdb/win32-nat.c b/gdb/win32-nat.c
index 07c7b60..eb15c97 100644
--- a/gdb/win32-nat.c
+++ b/gdb/win32-nat.c
@@ -2085,6 +2085,7 @@ init_win32_ops (void)
win32_ops.to_open = win32_open;
win32_ops.to_close = win32_close;
win32_ops.to_attach = win32_attach;
+ win32_ops.to_attach_no_wait = 1;
win32_ops.to_detach = win32_detach;
win32_ops.to_resume = win32_resume;
win32_ops.to_wait = win32_wait;
diff --git a/gdb/windows-nat.c b/gdb/windows-nat.c
index 07c7b60..eb15c97 100644
--- a/gdb/windows-nat.c
+++ b/gdb/windows-nat.c
@@ -2085,6 +2085,7 @@ init_win32_ops (void)
win32_ops.to_open = win32_open;
win32_ops.to_close = win32_close;
win32_ops.to_attach = win32_attach;
+ win32_ops.to_attach_no_wait = 1;
win32_ops.to_detach = win32_detach;
win32_ops.to_resume = win32_resume;
win32_ops.to_wait = win32_wait;