From afcad54a90445ba65ca91b13341b75facc96521c Mon Sep 17 00:00:00 2001 From: Andrew Cagney Date: Tue, 24 Nov 1998 13:49:00 +0000 Subject: [PATCH] CARP: *BREAKPOINT* Convert mn10300, MIPS and powerpc/rs6000 targets to use BREAKPOINT_FROM_PC Delete global variable memory_breakpoint_size. Use BREAKPOINT_FROM_PC instead. --- gdb/ChangeLog | 48 +++++++++++++++++++++++++++++++++ gdb/config/mips/tm-mips.h | 15 +---------- gdb/config/mn10300/tm-mn10300.h | 11 +++----- gdb/doc/gdbint.texinfo | 20 ++++++++++++++ gdb/mn10300-tdep.c | 17 ++++++++++++ gdb/remote-array.c | 8 +++--- gdb/remote-os9k.c | 6 +++-- gdb/remote-st.c | 7 +++-- gdb/rs6000-tdep.c | 19 +++++++++++++ 9 files changed, 123 insertions(+), 28 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 9bcc2cd0d50..5c5a74969f6 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,51 @@ +Tue Nov 24 14:13:10 1998 Andrew Cagney + + * breakpoint.c (memory_breakpoint_size): Delete global. + (read_memory_nobpt): Determine real breakpoint address and size + using BREAKPOINT_FROM_PC. + + * defs.h (breakpoint_from_pc_fn): BREAKPOINT_FROM_PC function + template. + * target.h, mem-break.c (memory_breakpoint_from_pc): + Rewrite. Always define. Return NULL when memory breakpoints are + not supported. + (memory_insert_breakpoint, memory_remove_breakpoint): Call + BREAKPOINT_FROM_PC. + * target.h (BREAKPOINT_FROM_PC): Provide default. + * gdbint.texinfo (BREAKPOINT_FROM_PC): Document. + + * config/rs6000/tm-rs6000.h (BREAKPOINT): Delete macro. + (BREAKPOINT_FROM_PC): Define. + ({BIG,LITTLE}_BREAKPOINT): Move macros from here. + * rs6000-tdep.c: To here. + (rs6000_breakpoint_from_pc): New function. + + * config/mn10300/tm-mn10300.h (BREAKPOINT): Delete macro. + (BREAKPOINT_FROM_PC): Define, call. + * mn10300-tdep.c (mn10300_breakpoint_from_pc): New function. + + * config/mips/tm-mips.h ({BIG,LITTLE}_BREAKPOINT, + IDT_{BIG,LITTLE}_BREAKPOINT, PMON_{BIG,LITTLE}_BREAKPOINT, + MIPS16_{BIG,LITTLE}_BREAKPOINT): Move macros from here. + * mips-tdep.c: To here. +start-sanitize-sky + * txvu-tdep.c (core_break_insn): Delete variable. + (txvu_insert_breakpoint): Update, use BREAKPOINT_FROM_PC. +end-sanitize-sky + + * config/arm/tm-arm.h ({BIG,LITTLE}_BREAKPOINT): Delete macros. + ({ARM,THUMB}_{BE,LE}_BREAKPOINT): Move macros from here. + * arm-tdep.c: To here. + + * remote-array.c (memory_breakpoint_size): Delete variable. + (array_insert_breakpoint): Obtain breakpoint size using + BREAKPOINT_FROM_PC. + * remote-st.c (memory_breakpoint_size, st2000_insert_breakpoint): + Ditto. + * remote-os9k.c (memory_breakpoint_size, + rombug_insert_breakpoint): Ditto. + * remote-e7000.c (memory_breakpoint_size): Ditto. + Mon Nov 23 11:38:40 1998 Michael Snyder * symfile.c (deduce_language_from_filename): rewrite so as to diff --git a/gdb/config/mips/tm-mips.h b/gdb/config/mips/tm-mips.h index 960e315de3a..922d218da28 100644 --- a/gdb/config/mips/tm-mips.h +++ b/gdb/config/mips/tm-mips.h @@ -123,19 +123,6 @@ extern int in_sigtramp PARAMS ((CORE_ADDR, char *)); #define BIG_ENDIAN 4321 -/* Old-style breakpoint macros. - The IDT board uses an unusual breakpoint value, and sometimes gets - confused when it sees the usual MIPS breakpoint instruction. */ - -#define BIG_BREAKPOINT {0, 0x5, 0, 0xd} -#define LITTLE_BREAKPOINT {0xd, 0, 0x5, 0} -#define PMON_BIG_BREAKPOINT {0, 0, 0, 0xd} -#define PMON_LITTLE_BREAKPOINT {0xd, 0, 0, 0} -#define IDT_BIG_BREAKPOINT {0, 0, 0x0a, 0xd} -#define IDT_LITTLE_BREAKPOINT {0xd, 0x0a, 0, 0} -#define MIPS16_BIG_BREAKPOINT {0xe8, 0xa5} -#define MIPS16_LITTLE_BREAKPOINT {0xa5, 0xe8} - /* BREAKPOINT_FROM_PC uses the program counter value to determine whether a 16- or 32-bit breakpoint should be used. It returns a pointer to a string of bytes that encode a breakpoint instruction, stores @@ -143,7 +130,7 @@ extern int in_sigtramp PARAMS ((CORE_ADDR, char *)); point to the actual memory location where the breakpoint should be inserted. */ -unsigned char *mips_breakpoint_from_pc PARAMS ((CORE_ADDR *pcptr, int *lenptr)); +extern breakpoint_from_pc_fn mips_breakpoint_from_pc; #define BREAKPOINT_FROM_PC(pcptr, lenptr) mips_breakpoint_from_pc(pcptr, lenptr) /* Amount PC must be decremented by after a breakpoint. diff --git a/gdb/config/mn10300/tm-mn10300.h b/gdb/config/mn10300/tm-mn10300.h index cce0470ff39..4898df40cfc 100644 --- a/gdb/config/mn10300/tm-mn10300.h +++ b/gdb/config/mn10300/tm-mn10300.h @@ -60,13 +60,10 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. */ /* Pseudo register that contains true address of executing stack frame */ #define FP_REGNUM 31 -/* The breakpoint instruction must be the same size as the smallest - instruction in the instruction set. - - The Matsushita mn10x00 processors have single byte instructions - so we need a single byte breakpoint. Matsushita hasn't defined - one, so we defined it ourselves. */ -#define BREAKPOINT {0xff} +/* BREAKPOINT_FROM_PC uses the program counter value to determine the + breakpoint that should be used */ +extern breakpoint_from_pc_fn mn10300_breakpoint_from_pc; +#define BREAKPOINT_FROM_PC(pcptr, lenptr) mn10300_breakpoint_from_pc (pcptr, lenptr) #define FUNCTION_START_OFFSET 0 diff --git a/gdb/doc/gdbint.texinfo b/gdb/doc/gdbint.texinfo index bbc5437d259..d7dfa1782c2 100644 --- a/gdb/doc/gdbint.texinfo +++ b/gdb/doc/gdbint.texinfo @@ -1140,6 +1140,26 @@ longer than the shortest instruction of the architecture. @item LITTLE_BREAKPOINT Similar to BREAKPOINT, but used for bi-endian targets. +@item REMOTE_BREAKPOINT +@item LITTLE_REMOTE_BREAKPOINT +@item BIG_REMOTE_BREAKPOINT +Similar to BREAKPOINT, but used for remote targets. + +@item BREAKPOINT_FROM_PC (pcptr, lenptr) + +Use the program counter to determine the contents and size of a +breakpoint instruction. It returns a pointer to a string of bytes that +encode a breakpoint instruction, stores the length of the string to +*lenptr, and adjusts pc (if necessary) to point to the actual memory +location where the breakpoint should be inserted. + +Although it is common to use a trap instruction for a breakpoint, it's +not required; for instance, the bit pattern could be an invalid +instruction. The breakpoint must be no longer than the shortest +instruction of the architecture. + +Replaces all the other BREAKPOINTs. + @item CALL_DUMMY valops.c @item CALL_DUMMY_LOCATION diff --git a/gdb/mn10300-tdep.c b/gdb/mn10300-tdep.c index 151331984d1..44e9f6b0263 100644 --- a/gdb/mn10300-tdep.c +++ b/gdb/mn10300-tdep.c @@ -58,6 +58,23 @@ mn10300_use_struct_convention (gcc_p, type) return (TYPE_NFIELDS (type) > 1 || TYPE_LENGTH (type) > 8); } +/* The breakpoint instruction must be the same size as the smallest + instruction in the instruction set. + + The Matsushita mn10x00 processors have single byte instructions + so we need a single byte breakpoint. Matsushita hasn't defined + one, so we defined it ourselves. */ + +unsigned char * +mn10300_breakpoint_from_pc (bp_addr, bp_size) + CORE_ADDR *bp_addr; + int *bp_size; +{ + static char breakpoint[] = {0xff}; + *bp_size = 1; + return breakpoint; +} + /* Fix fi->frame if it's bogus at this point. This is a helper function for mn10300_analyze_prologue. */ diff --git a/gdb/remote-array.c b/gdb/remote-array.c index c17535d5168..f6f174f4bbc 100644 --- a/gdb/remote-array.c +++ b/gdb/remote-array.c @@ -1,5 +1,5 @@ /* Remote debugging interface for Array Tech RAID controller.. - Copyright 90, 91, 92, 93, 94, 1995 Free Software Foundation, Inc. + Copyright 90, 91, 92, 93, 94, 1995, 1998 Free Software Foundation, Inc. Contributed by Cygnus Support. Written by Rob Savoye for Cygnus. This module talks to a debug monitor called 'MONITOR', which @@ -1050,7 +1050,6 @@ array_mourn_inferior () #define MAX_ARRAY_BREAKPOINTS 16 -extern int memory_breakpoint_size; static CORE_ADDR breakaddr[MAX_ARRAY_BREAKPOINTS] = {0}; /* @@ -1062,15 +1061,18 @@ array_insert_breakpoint (addr, shadow) char *shadow; { int i; + int bp_size = 0; + int bp_addr = addr; debuglogs (1, "array_insert_breakpoint() addr = 0x%x", addr); + BREAKPOINT_FROM_PC (&bp_addr, &bp_size); for (i = 0; i <= MAX_ARRAY_BREAKPOINTS; i++) { if (breakaddr[i] == 0) { breakaddr[i] = addr; if (sr_get_debug() > 4) printf ("Breakpoint at %x\n", addr); - array_read_inferior_memory(addr, shadow, memory_breakpoint_size); + array_read_inferior_memory (bp_addr, shadow, bp_size); printf_monitor("b 0x%x\n", addr); expect_prompt(1); return 0; diff --git a/gdb/remote-os9k.c b/gdb/remote-os9k.c index fa2e2bc2952..84a0f2b5382 100644 --- a/gdb/remote-os9k.c +++ b/gdb/remote-os9k.c @@ -886,7 +886,6 @@ rombug_mourn_inferior () #define MAX_MONITOR_BREAKPOINTS 16 -extern int memory_breakpoint_size; static CORE_ADDR breakaddr[MAX_MONITOR_BREAKPOINTS] = {0}; static int @@ -895,9 +894,12 @@ rombug_insert_breakpoint (addr, shadow) char *shadow; { int i; + CORE_ADDR bp_addr = addr; + int bp_size = 0; if (monitor_log) fprintf (log_file, "\nIn Insert_breakpoint (addr=%x)\n", addr); + BREAKPOINT_FROM_PC (&bp_addr, &bp_size); for (i = 0; i <= MAX_MONITOR_BREAKPOINTS; i++) if (breakaddr[i] == 0) @@ -905,7 +907,7 @@ rombug_insert_breakpoint (addr, shadow) breakaddr[i] = addr; if (sr_get_debug()) printf ("Breakpoint at %x\n", addr); - rombug_read_inferior_memory(addr, shadow, memory_breakpoint_size); + rombug_read_inferior_memory (bp_addr, shadow, bp_size); printf_monitor(SET_BREAK_CMD, addr); is_trace_mode = 0; expect_prompt(1); diff --git a/gdb/remote-st.c b/gdb/remote-st.c index 2e81dae41f3..3fd8cf17638 100644 --- a/gdb/remote-st.c +++ b/gdb/remote-st.c @@ -627,7 +627,6 @@ st2000_mourn_inferior () #define MAX_STDEBUG_BREAKPOINTS 16 -extern int memory_breakpoint_size; static CORE_ADDR breakaddr[MAX_STDEBUG_BREAKPOINTS] = {0}; static int @@ -636,13 +635,17 @@ st2000_insert_breakpoint (addr, shadow) char *shadow; { int i; + CORE_ADDR bp_addr = addr; + int bp_size = 0; + + BREAKPOINT_FROM_PC (&bp_addr, &bp_size); for (i = 0; i <= MAX_STDEBUG_BREAKPOINTS; i++) if (breakaddr[i] == 0) { breakaddr[i] = addr; - st2000_read_inferior_memory(addr, shadow, memory_breakpoint_size); + st2000_read_inferior_memory (bp_addr, shadow, bp_size); printf_stdebug("BR %x H\r", addr); expect_prompt(1); return 0; diff --git a/gdb/rs6000-tdep.c b/gdb/rs6000-tdep.c index 346b26cf559..36f5f614724 100644 --- a/gdb/rs6000-tdep.c +++ b/gdb/rs6000-tdep.c @@ -134,6 +134,25 @@ branch_dest (opcode, instr, pc, safety) } +/* Sequence of bytes for breakpoint instruction. */ + +#define BIG_BREAKPOINT { 0x7d, 0x82, 0x10, 0x08 } +#define LITTLE_BREAKPOINT { 0x08, 0x10, 0x82, 0x7d } + +unsigned char * +rs6000_breakpoint_from_pc (bp_addr, bp_size) + CORE_ADDR *bp_addr; + int *bp_size; +{ + static unsigned char *big_breakpoint = BIG_BREAKPOINT; + static unsigned char *little_breakpoint = LITTLE_BREAKPOINT; + *bp_size = 4; + if (TARGET_BYTE_ORDER == BIG_ENDIAN) + return big_breakpoint; + else + return little_breakpoint; +} + /* AIX does not support PT_STEP. Simulate it. */ -- 2.30.2