From fa4ba8da6c28c972dd1b6b9971e29b51aabaafcc Mon Sep 17 00:00:00 2001 From: Pierre Muller Date: Mon, 4 Feb 2002 11:00:10 +0000 Subject: [PATCH] 2002-02-04 Pierre Muller Add support for hardware watchpoints on win32 native. * win32-nat.c (CONTEXT_DEBUG_DR macro): Add use of CONTEXT_DEBUG_REGISTERS. (dr variable): New variable. Static array containing a local copy of debug registers. (debug_registers_changed): New variable. Reflects when debug registers are changed and need to be written to inferior. (debug_registers_used): New variable. Reflects when any debug register was set, used when new threads are created. (cygwin_set_dr, cygwin_set_dr7, cygwin_get_dr6): New functions used by i386-nat code. (thread_rec): Set dr array if id is the thread of current_event . (child_continue, child_resume): Change the debug registers for all threads if debug_registers_changed. (child_add_thread): Change the debug registers if debug_registers_used. * config/i386/cygwin.mh: Add use of i386-nat.o file. Link nm.h to new nm-cygwin.h file. + config/i386/nm-cygwin.h: New file. Contains the macros used for use of hardware registers. --- gdb/ChangeLog | 22 +++++++ gdb/config/i386/cygwin.mh | 4 +- gdb/config/i386/nm-cygwin.h | 38 ++++++++++++ gdb/win32-nat.c | 119 +++++++++++++++++++++++++++++++++--- gdb/windows-nat.c | 119 +++++++++++++++++++++++++++++++++--- 5 files changed, 282 insertions(+), 20 deletions(-) create mode 100644 gdb/config/i386/nm-cygwin.h diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 4f54eb3d57b..135bdfc02ab 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,25 @@ +2002-02-04 Pierre Muller + + Add support for hardware watchpoints on win32 native. + * win32-nat.c (CONTEXT_DEBUG_DR macro): Add use of + CONTEXT_DEBUG_REGISTERS. + (dr variable): New variable. Static array containing a local copy + of debug registers. + (debug_registers_changed): New variable. Reflects when debug registers + are changed and need to be written to inferior. + (debug_registers_used): New variable. Reflects when any debug register + was set, used when new threads are created. + (cygwin_set_dr, cygwin_set_dr7, cygwin_get_dr6): New functions used by + i386-nat code. + (thread_rec): Set dr array if id is the thread of current_event . + (child_continue, child_resume): Change the debug registers for all + threads if debug_registers_changed. + (child_add_thread): Change the debug registers if debug_registers_used. + * config/i386/cygwin.mh: Add use of i386-nat.o file. + Link nm.h to new nm-cygwin.h file. + + config/i386/nm-cygwin.h: New file. Contains the macros used for use + of hardware registers. + 2002-02-03 Andrew Cagney * valprint.c (print_floating): Allow non TYPE_CODE_FLT types. diff --git a/gdb/config/i386/cygwin.mh b/gdb/config/i386/cygwin.mh index 3680fc7dcd9..ce4656f6f51 100644 --- a/gdb/config/i386/cygwin.mh +++ b/gdb/config/i386/cygwin.mh @@ -1,5 +1,5 @@ MH_CFLAGS= XM_FILE=xm-cygwin.h -NATDEPFILES= win32-nat.o corelow.o -NAT_FILE=../none/nm-none.h +NATDEPFILES= i386-nat.o win32-nat.o corelow.o +NAT_FILE=nm-cygwin.h XM_CLIBS= diff --git a/gdb/config/i386/nm-cygwin.h b/gdb/config/i386/nm-cygwin.h new file mode 100644 index 00000000000..b5e48907013 --- /dev/null +++ b/gdb/config/i386/nm-cygwin.h @@ -0,0 +1,38 @@ +/* Native definitions for Intel x86 running CYGWIN. + Copyright (C) 2002 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 2 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, write to the Free Software + Foundation, Inc., 59 Temple Place - Suite 330, + Boston, MA 02111-1307, USA. */ + +#define NO_PTRACE_H + +#define I386_USE_GENERIC_WATCHPOINTS + +#include "i386/nm-i386.h" + +/* Support for hardware-assisted breakpoints and watchpoints. */ + +#define I386_DR_LOW_SET_CONTROL(VAL) cygwin_set_dr7 (VAL) +extern void cygwin_set_dr7 (unsigned); + +#define I386_DR_LOW_SET_ADDR(N,ADDR) cygwin_set_dr (N,ADDR) +extern void cygwin_set_dr (int, CORE_ADDR); + +#define I386_DR_LOW_RESET_ADDR(N) + +#define I386_DR_LOW_GET_STATUS() cygwin_get_dr6 () +extern unsigned cygwin_get_dr6 (void); diff --git a/gdb/win32-nat.c b/gdb/win32-nat.c index 5265aefc4a3..f6895766e7a 100644 --- a/gdb/win32-nat.c +++ b/gdb/win32-nat.c @@ -71,11 +71,15 @@ enum #include #ifdef HAVE_SSE_REGS -#define CONTEXT_DEBUGGER_DR CONTEXT_DEBUGGER | CONTEXT_EXTENDED_REGISTERS +#define CONTEXT_DEBUGGER_DR CONTEXT_DEBUGGER | CONTEXT_DEBUG_REGISTERS \ + | CONTEXT_EXTENDED_REGISTERS #else -#define CONTEXT_DEBUGGER_DR CONTEXT_DEBUGGER +#define CONTEXT_DEBUGGER_DR CONTEXT_DEBUGGER | CONTEXT_DEBUG_REGISTERS #endif +static unsigned dr[8]; +static int debug_registers_changed = 0; +static int debug_registers_used = 0; /* The string sent by cygwin when it processes a signal. FIXME: This should be in a cygwin include file. */ @@ -216,6 +220,15 @@ static const struct xlate_exception {EXCEPTION_SINGLE_STEP, TARGET_SIGNAL_TRAP}, {-1, -1}}; +static void +check (BOOL ok, const char *file, int line) +{ + if (!ok) + printf_filtered ("error return %s:%d was %lu\n", file, line, + GetLastError ()); +} + + /* Find a thread record given a thread id. If get_context then also retrieve the context for this thread. */ @@ -236,6 +249,16 @@ thread_rec (DWORD id, int get_context) th->context.ContextFlags = CONTEXT_DEBUGGER_DR; GetThreadContext (th->h, &th->context); + if (id == current_event.dwThreadId) + { + /* Copy dr values from that thread. */ + dr[0] = th->context.Dr0; + dr[1] = th->context.Dr1; + dr[2] = th->context.Dr2; + dr[3] = th->context.Dr3; + dr[6] = th->context.Dr6; + dr[7] = th->context.Dr7; + } } return th; } @@ -259,6 +282,22 @@ child_add_thread (DWORD id, HANDLE h) th->next = thread_head.next; thread_head.next = th; add_thread (pid_to_ptid (id)); + /* Set the debug registers for the new thread in they are used. */ + if (debug_registers_used) + { + /* Only change the value of the debug registers. */ + th->context.ContextFlags = CONTEXT_DEBUG_REGISTERS; + CHECK (GetThreadContext (th->h, &th->context)); + th->context.Dr0 = dr[0]; + th->context.Dr1 = dr[1]; + th->context.Dr2 = dr[2]; + th->context.Dr3 = dr[3]; + /* th->context.Dr6 = dr[6]; + FIXME: should we set dr6 also ?? */ + th->context.Dr7 = dr[7]; + CHECK (SetThreadContext (th->h, &th->context)); + th->context.ContextFlags = 0; + } return th; } @@ -304,13 +343,6 @@ child_delete_thread (DWORD id) } } -static void -check (BOOL ok, const char *file, int line) -{ - if (!ok) - printf_filtered ("error return %s:%d was %lu\n", file, line, GetLastError ()); -} - static void do_child_fetch_inferior_registers (int r) { @@ -878,11 +910,27 @@ child_continue (DWORD continue_status, int id) for (th = &thread_head; (th = th->next) != NULL;) if (((id == -1) || (id == (int) th->id)) && th->suspend_count) { + for (i = 0; i < th->suspend_count; i++) (void) ResumeThread (th->h); th->suspend_count = 0; + if (debug_registers_changed) + { + /* Only change the value of the debug reisters */ + th->context.ContextFlags = CONTEXT_DEBUG_REGISTERS; + th->context.Dr0 = dr[0]; + th->context.Dr1 = dr[1]; + th->context.Dr2 = dr[2]; + th->context.Dr3 = dr[3]; + /* th->context.Dr6 = dr[6]; + FIXME: should we set dr6 also ?? */ + th->context.Dr7 = dr[7]; + CHECK (SetThreadContext (th->h, &th->context)); + th->context.ContextFlags = 0; + } } + debug_registers_changed = 0; return res; } @@ -1062,10 +1110,15 @@ static void do_initial_child_stuff (DWORD pid) { extern int stop_after_trap; + int i; last_sig = 0; event_count = 0; exception_count = 0; + debug_registers_changed = 0; + debug_registers_used = 0; + for (i = 0; i < sizeof (dr) / sizeof (dr[0]); i++) + dr[i] = 0; current_event.dwProcessId = pid; memset (¤t_event, 0, sizeof (current_event)); push_target (&child_ops); @@ -1345,6 +1398,7 @@ static void child_mourn_inferior (void) { (void) child_continue (DBG_CONTINUE, -1); + i386_cleanup_dregs(); unpush_target (&child_ops); generic_mourn_inferior (); } @@ -1432,6 +1486,16 @@ child_resume (ptid_t ptid, int step, enum target_signal sig) if (th->context.ContextFlags) { + if (debug_registers_changed) + { + th->context.Dr0 = dr[0]; + th->context.Dr1 = dr[1]; + th->context.Dr2 = dr[2]; + th->context.Dr3 = dr[3]; + /* th->context.Dr6 = dr[6]; + FIXME: should we set dr6 also ?? */ + th->context.Dr7 = dr[7]; + } CHECK (SetThreadContext (th->h, &th->context)); th->context.ContextFlags = 0; } @@ -1565,6 +1629,43 @@ _initialize_inftarg (void) add_target (&child_ops); } +/* Hardware watchpoint support, adapted from go32-nat.c code. */ + +/* Pass the address ADDR to the inferior in the I'th debug register. + Here we just store the address in dr array, the registers will be + actually set up when child_continue is called. */ +void +cygwin_set_dr (int i, CORE_ADDR addr) +{ + if (i < 0 || i > 3) + internal_error (__FILE__, __LINE__, + "Invalid register %d in cygwin_set_dr.\n", i); + dr[i] = (unsigned) addr; + debug_registers_changed = 1; + debug_registers_used = 1; +} + +/* Pass the value VAL to the inferior in the DR7 debug control + register. Here we just store the address in D_REGS, the watchpoint + will be actually set up in child_wait. */ +void +cygwin_set_dr7 (unsigned val) +{ + dr[7] = val; + debug_registers_changed = 1; + debug_registers_used = 1; +} + +/* Get the value of the DR6 debug status register from the inferior. + Here we just return the value stored in dr[6] + by the last call to thread_rec for current_event.dwThreadId id. */ +unsigned +cygwin_get_dr6 (void) +{ + return dr[6]; +} + + /* Determine if the thread referenced by "pid" is alive by "polling" it. If WaitForSingleObject returns WAIT_OBJECT_0 it means that the pid has died. Otherwise it is assumed to be alive. */ diff --git a/gdb/windows-nat.c b/gdb/windows-nat.c index 5265aefc4a3..f6895766e7a 100644 --- a/gdb/windows-nat.c +++ b/gdb/windows-nat.c @@ -71,11 +71,15 @@ enum #include #ifdef HAVE_SSE_REGS -#define CONTEXT_DEBUGGER_DR CONTEXT_DEBUGGER | CONTEXT_EXTENDED_REGISTERS +#define CONTEXT_DEBUGGER_DR CONTEXT_DEBUGGER | CONTEXT_DEBUG_REGISTERS \ + | CONTEXT_EXTENDED_REGISTERS #else -#define CONTEXT_DEBUGGER_DR CONTEXT_DEBUGGER +#define CONTEXT_DEBUGGER_DR CONTEXT_DEBUGGER | CONTEXT_DEBUG_REGISTERS #endif +static unsigned dr[8]; +static int debug_registers_changed = 0; +static int debug_registers_used = 0; /* The string sent by cygwin when it processes a signal. FIXME: This should be in a cygwin include file. */ @@ -216,6 +220,15 @@ static const struct xlate_exception {EXCEPTION_SINGLE_STEP, TARGET_SIGNAL_TRAP}, {-1, -1}}; +static void +check (BOOL ok, const char *file, int line) +{ + if (!ok) + printf_filtered ("error return %s:%d was %lu\n", file, line, + GetLastError ()); +} + + /* Find a thread record given a thread id. If get_context then also retrieve the context for this thread. */ @@ -236,6 +249,16 @@ thread_rec (DWORD id, int get_context) th->context.ContextFlags = CONTEXT_DEBUGGER_DR; GetThreadContext (th->h, &th->context); + if (id == current_event.dwThreadId) + { + /* Copy dr values from that thread. */ + dr[0] = th->context.Dr0; + dr[1] = th->context.Dr1; + dr[2] = th->context.Dr2; + dr[3] = th->context.Dr3; + dr[6] = th->context.Dr6; + dr[7] = th->context.Dr7; + } } return th; } @@ -259,6 +282,22 @@ child_add_thread (DWORD id, HANDLE h) th->next = thread_head.next; thread_head.next = th; add_thread (pid_to_ptid (id)); + /* Set the debug registers for the new thread in they are used. */ + if (debug_registers_used) + { + /* Only change the value of the debug registers. */ + th->context.ContextFlags = CONTEXT_DEBUG_REGISTERS; + CHECK (GetThreadContext (th->h, &th->context)); + th->context.Dr0 = dr[0]; + th->context.Dr1 = dr[1]; + th->context.Dr2 = dr[2]; + th->context.Dr3 = dr[3]; + /* th->context.Dr6 = dr[6]; + FIXME: should we set dr6 also ?? */ + th->context.Dr7 = dr[7]; + CHECK (SetThreadContext (th->h, &th->context)); + th->context.ContextFlags = 0; + } return th; } @@ -304,13 +343,6 @@ child_delete_thread (DWORD id) } } -static void -check (BOOL ok, const char *file, int line) -{ - if (!ok) - printf_filtered ("error return %s:%d was %lu\n", file, line, GetLastError ()); -} - static void do_child_fetch_inferior_registers (int r) { @@ -878,11 +910,27 @@ child_continue (DWORD continue_status, int id) for (th = &thread_head; (th = th->next) != NULL;) if (((id == -1) || (id == (int) th->id)) && th->suspend_count) { + for (i = 0; i < th->suspend_count; i++) (void) ResumeThread (th->h); th->suspend_count = 0; + if (debug_registers_changed) + { + /* Only change the value of the debug reisters */ + th->context.ContextFlags = CONTEXT_DEBUG_REGISTERS; + th->context.Dr0 = dr[0]; + th->context.Dr1 = dr[1]; + th->context.Dr2 = dr[2]; + th->context.Dr3 = dr[3]; + /* th->context.Dr6 = dr[6]; + FIXME: should we set dr6 also ?? */ + th->context.Dr7 = dr[7]; + CHECK (SetThreadContext (th->h, &th->context)); + th->context.ContextFlags = 0; + } } + debug_registers_changed = 0; return res; } @@ -1062,10 +1110,15 @@ static void do_initial_child_stuff (DWORD pid) { extern int stop_after_trap; + int i; last_sig = 0; event_count = 0; exception_count = 0; + debug_registers_changed = 0; + debug_registers_used = 0; + for (i = 0; i < sizeof (dr) / sizeof (dr[0]); i++) + dr[i] = 0; current_event.dwProcessId = pid; memset (¤t_event, 0, sizeof (current_event)); push_target (&child_ops); @@ -1345,6 +1398,7 @@ static void child_mourn_inferior (void) { (void) child_continue (DBG_CONTINUE, -1); + i386_cleanup_dregs(); unpush_target (&child_ops); generic_mourn_inferior (); } @@ -1432,6 +1486,16 @@ child_resume (ptid_t ptid, int step, enum target_signal sig) if (th->context.ContextFlags) { + if (debug_registers_changed) + { + th->context.Dr0 = dr[0]; + th->context.Dr1 = dr[1]; + th->context.Dr2 = dr[2]; + th->context.Dr3 = dr[3]; + /* th->context.Dr6 = dr[6]; + FIXME: should we set dr6 also ?? */ + th->context.Dr7 = dr[7]; + } CHECK (SetThreadContext (th->h, &th->context)); th->context.ContextFlags = 0; } @@ -1565,6 +1629,43 @@ _initialize_inftarg (void) add_target (&child_ops); } +/* Hardware watchpoint support, adapted from go32-nat.c code. */ + +/* Pass the address ADDR to the inferior in the I'th debug register. + Here we just store the address in dr array, the registers will be + actually set up when child_continue is called. */ +void +cygwin_set_dr (int i, CORE_ADDR addr) +{ + if (i < 0 || i > 3) + internal_error (__FILE__, __LINE__, + "Invalid register %d in cygwin_set_dr.\n", i); + dr[i] = (unsigned) addr; + debug_registers_changed = 1; + debug_registers_used = 1; +} + +/* Pass the value VAL to the inferior in the DR7 debug control + register. Here we just store the address in D_REGS, the watchpoint + will be actually set up in child_wait. */ +void +cygwin_set_dr7 (unsigned val) +{ + dr[7] = val; + debug_registers_changed = 1; + debug_registers_used = 1; +} + +/* Get the value of the DR6 debug status register from the inferior. + Here we just return the value stored in dr[6] + by the last call to thread_rec for current_event.dwThreadId id. */ +unsigned +cygwin_get_dr6 (void) +{ + return dr[6]; +} + + /* Determine if the thread referenced by "pid" is alive by "polling" it. If WaitForSingleObject returns WAIT_OBJECT_0 it means that the pid has died. Otherwise it is assumed to be alive. */ -- 2.30.2