This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [PATCH]: Extend SVR4 shlib debug changes to SH3 and other targets
- To: Mark Kettenis <kettenis at wins dot uva dot nl>
- Subject: Re: [PATCH]: Extend SVR4 shlib debug changes to SH3 and other targets
- From: Michael Snyder <msnyder at redhat dot com>
- Date: Tue, 31 Oct 2000 11:51:26 -0800
- CC: gdb-patches at sourceware dot cygnus dot com, kevinb at cygnus dot com
- Organization: Red Hat, Inc.
- References: <200010310102.RAA24876@cleaver.cygnus.com> <200010310937.e9V9b4604953@debye.wins.uva.nl>
Tweaked per feedback and checked in.
Mark Kettenis wrote:
>
> From: Michael Snyder <msnyder@cygnus.com>
> Date: Mon, 30 Oct 2000 17:02:01 -0800 (PST)
>
> This change moves the definition of SVR4_SHARED_LIBS into
> config/tm-linux.h, so that it will affect all linuxen.
> I then add a new file config/sh/tm-linux.h which includes
> config/tm-linux.h (as well as sh/tm-sh.h). Finally I add
> target-specific definitions of fetch_link_map_offsets, so
> that shared libraries can be cross-debugged.
>
> 2000-10-30 Michael Snyder <msnyder@cleaver.cygnus.com>
>
> * config/sh/tm-linux.h: New file. Include generic tm-linux.h,
> plus tm-sh.h, then define SVR4_FETCH_LINK_MAP_OFFSETS to use
> the sh target function instead of the default link map offsets.
> * config/sh/sh.mt: Add solib.o and solib-svr4.o to TDEPFILES.
> Use sh/tm-linux.h instead of sh/tm-sh.h.
> * sh-tdep.c (sh_linux_svr4_fetch_link_map_offsets):
> New function. Construct target-specific link map offsets.
> * i386-linux-tdep.c (i386_linux_svr4_fetch_link_map_offsets:
> New function. Construct target-specific link map offsets.
> * config/i386/tm-linux.h: Use above function instead of default.
>
> 2000-10-30 Michael Snyder <msnyder@cleaver.cygnus.com>
>
> * config/i386/tm-linux.h: Remove definition of SVR4_SHARED_LIBS,
> and inclusion of solib.h. Move up into ../tm-linux.h.
> config/tm-linux.h: Define SVR4_SHARED_LIBS, include solib.h.
>
> Looks good to me. One minor nit though:
>
> Index: config/tm-linux.h
> ===================================================================
> RCS file: /cvs/src/src/gdb/config/tm-linux.h,v
> retrieving revision 1.1.1.1
> diff -c -3 -p -r1.1.1.1 tm-linux.h
> *** tm-linux.h 1999/12/07 03:56:08 1.1.1.1
> --- tm-linux.h 2000/10/31 00:57:12
> ***************
> *** 34,36 ****
> --- 34,42 ----
> /* We need this file for the SOLIB_TRAMPOLINE stuff. */
>
> #include "tm-sysv4.h"
> +
> + /* We define this if link.h is available, because with ELF we use SVR4 style
> + shared libraries. FIXME: move up into config/tm-linux.h? */
> +
> + #define SVR4_SHARED_LIBS
> + #include "solib.h" /* Support for shared libraries. */
>
> Please update the comment. The FIXME can go now, and the blurb about
> link.h isn't true anymore isn't it?