From 76e1ee85c1f22e95e4910b38c286d00f597859fd Mon Sep 17 00:00:00 2001 From: Daniel Jacobowitz Date: Mon, 14 May 2007 17:33:33 +0000 Subject: [PATCH] * gdb_proc_service.h (paddr_t): Delete typedef. * proc-service.c (ps_addr_to_core_addr, core_addr_to_ps_addr): New. (ps_xfer_memory): Take a psaddr_t. Use ps_addr_to_core_addr. (ps_pglobal_lookup): Take a psaddr_t *. Use core_addr_to_ps_addr. (ps_pdread, ps_pdwrite, ps_ptread, ps_ptwrite): Take a psaddr_t. * sol-thread.c (gdb_ps_addr_t): Use psaddr_t instead of paddr_t. * Makefile.in (proc-service.o): Update. --- gdb/ChangeLog | 10 +++++++++ gdb/Makefile.in | 3 ++- gdb/gdb_proc_service.h | 2 -- gdb/proc-service.c | 50 ++++++++++++++++++++++++++++++++---------- gdb/sol-thread.c | 2 +- 5 files changed, 51 insertions(+), 16 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 0394552c67a..d1486a0754b 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,13 @@ +2007-05-14 Daniel Jacobowitz + + * gdb_proc_service.h (paddr_t): Delete typedef. + * proc-service.c (ps_addr_to_core_addr, core_addr_to_ps_addr): New. + (ps_xfer_memory): Take a psaddr_t. Use ps_addr_to_core_addr. + (ps_pglobal_lookup): Take a psaddr_t *. Use core_addr_to_ps_addr. + (ps_pdread, ps_pdwrite, ps_ptread, ps_ptwrite): Take a psaddr_t. + * sol-thread.c (gdb_ps_addr_t): Use psaddr_t instead of paddr_t. + * Makefile.in (proc-service.o): Update. + 2007-05-14 Daniel Jacobowitz * Makefile.in (mips-tdep.o): Update. diff --git a/gdb/Makefile.in b/gdb/Makefile.in index 7574f0b682a..db74f877e1c 100644 --- a/gdb/Makefile.in +++ b/gdb/Makefile.in @@ -2462,7 +2462,8 @@ procfs.o: procfs.c $(defs_h) $(inferior_h) $(target_h) $(gdbcore_h) \ $(gdb_string_h) $(gdb_assert_h) $(inflow_h) $(auxv_h) \ $(gdb_dirent_h) $(gdb_stat_h) $(proc_utils_h) $(gregset_h) proc-service.o: proc-service.c $(defs_h) $(gdb_proc_service_h) $(inferior_h) \ - $(symtab_h) $(target_h) $(regcache_h) $(gregset_h) + $(symtab_h) $(target_h) $(regcache_h) $(gregset_h) $(gdbcore_h) \ + $(gdb_stdint_h) proc-why.o: proc-why.c $(defs_h) $(proc_utils_h) prologue-value.o: prologue-value.c $(defs_h) $(gdb_string_h) $(gdb_assert_h) \ $(prologue_value_h) $(regcache_h) diff --git a/gdb/gdb_proc_service.h b/gdb/gdb_proc_service.h index 7dae10fb3f6..f889920f10e 100644 --- a/gdb/gdb_proc_service.h +++ b/gdb/gdb_proc_service.h @@ -48,8 +48,6 @@ typedef enum typedef unsigned int lwpid_t; #endif -typedef unsigned long paddr_t; - #ifndef HAVE_PSADDR_T typedef unsigned long psaddr_t; #endif diff --git a/gdb/proc-service.c b/gdb/proc-service.c index 6d3eb886e32..d38ade538bb 100644 --- a/gdb/proc-service.c +++ b/gdb/proc-service.c @@ -21,14 +21,17 @@ #include "defs.h" -#include "gdb_proc_service.h" -#include - +#include "gdbcore.h" #include "inferior.h" #include "symtab.h" #include "target.h" #include "regcache.h" +#include "gdb_proc_service.h" +#include "gdb_stdint.h" + +#include + /* Prototypes for supply_gregset etc. */ #include "gregset.h" @@ -58,6 +61,28 @@ typedef size_t gdb_ps_size_t; /* Helper functions. */ +/* Convert a psaddr_t to a CORE_ADDR. */ + +static CORE_ADDR +ps_addr_to_core_addr (psaddr_t addr) +{ + if (exec_bfd && bfd_get_sign_extend_vma (exec_bfd)) + return (intptr_t) addr; + else + return (uintptr_t) addr; +} + +/* Convert a CORE_ADDR to a psaddr_t. */ + +static psaddr_t +core_addr_to_ps_addr (CORE_ADDR addr) +{ + if (exec_bfd && bfd_get_sign_extend_vma (exec_bfd)) + return (psaddr_t) (intptr_t) addr; + else + return (psaddr_t) (uintptr_t) addr; +} + /* Transfer LEN bytes of memory between BUF and address ADDR in the process specified by PH. If WRITE, transfer them to the process, else transfer them from the process. Returns PS_OK for success, @@ -67,18 +92,19 @@ typedef size_t gdb_ps_size_t; ps_ptwrite. */ static ps_err_e -ps_xfer_memory (const struct ps_prochandle *ph, paddr_t addr, +ps_xfer_memory (const struct ps_prochandle *ph, psaddr_t addr, gdb_byte *buf, size_t len, int write) { struct cleanup *old_chain = save_inferior_ptid (); int ret; + CORE_ADDR core_addr = ps_addr_to_core_addr (addr); inferior_ptid = pid_to_ptid (ph->pid); if (write) - ret = target_write_memory (addr, buf, len); + ret = target_write_memory (core_addr, buf, len); else - ret = target_read_memory (addr, buf, len); + ret = target_read_memory (core_addr, buf, len); do_cleanups (old_chain); @@ -173,7 +199,7 @@ ps_plog (const char *fmt, ...) ps_err_e ps_pglobal_lookup (gdb_ps_prochandle_t ph, const char *obj, - const char *name, paddr_t *sym_addr) + const char *name, psaddr_t *sym_addr) { struct minimal_symbol *ms; @@ -182,7 +208,7 @@ ps_pglobal_lookup (gdb_ps_prochandle_t ph, const char *obj, if (ms == NULL) return PS_NOSYM; - *sym_addr = SYMBOL_VALUE_ADDRESS (ms); + *sym_addr = core_addr_to_ps_addr (SYMBOL_VALUE_ADDRESS (ms)); return PS_OK; } @@ -190,7 +216,7 @@ ps_pglobal_lookup (gdb_ps_prochandle_t ph, const char *obj, them into BUF. */ ps_err_e -ps_pdread (gdb_ps_prochandle_t ph, paddr_t addr, +ps_pdread (gdb_ps_prochandle_t ph, psaddr_t addr, gdb_ps_read_buf_t buf, gdb_ps_size_t size) { return ps_xfer_memory (ph, addr, buf, size, 0); @@ -199,7 +225,7 @@ ps_pdread (gdb_ps_prochandle_t ph, paddr_t addr, /* Write SIZE bytes from BUF into the target process PH at address ADDR. */ ps_err_e -ps_pdwrite (gdb_ps_prochandle_t ph, paddr_t addr, +ps_pdwrite (gdb_ps_prochandle_t ph, psaddr_t addr, gdb_ps_write_buf_t buf, gdb_ps_size_t size) { return ps_xfer_memory (ph, addr, (gdb_byte *) buf, size, 1); @@ -209,7 +235,7 @@ ps_pdwrite (gdb_ps_prochandle_t ph, paddr_t addr, them into BUF. */ ps_err_e -ps_ptread (gdb_ps_prochandle_t ph, paddr_t addr, +ps_ptread (gdb_ps_prochandle_t ph, psaddr_t addr, gdb_ps_read_buf_t buf, gdb_ps_size_t size) { return ps_xfer_memory (ph, addr, (gdb_byte *) buf, size, 0); @@ -218,7 +244,7 @@ ps_ptread (gdb_ps_prochandle_t ph, paddr_t addr, /* Write SIZE bytes from BUF into the target process PH at address ADDR. */ ps_err_e -ps_ptwrite (gdb_ps_prochandle_t ph, paddr_t addr, +ps_ptwrite (gdb_ps_prochandle_t ph, psaddr_t addr, gdb_ps_write_buf_t buf, gdb_ps_size_t size) { return ps_xfer_memory (ph, addr, (gdb_byte *) buf, size, 1); diff --git a/gdb/sol-thread.c b/gdb/sol-thread.c index ee26b412478..4224d283262 100644 --- a/gdb/sol-thread.c +++ b/gdb/sol-thread.c @@ -910,7 +910,7 @@ typedef const struct ps_prochandle *gdb_ps_prochandle_t; typedef char *gdb_ps_read_buf_t; typedef char *gdb_ps_write_buf_t; typedef int gdb_ps_size_t; -typedef paddr_t gdb_ps_addr_t; +typedef psaddr_t gdb_ps_addr_t; #else typedef struct ps_prochandle *gdb_ps_prochandle_t; typedef void *gdb_ps_read_buf_t; -- 2.30.2