From: Daniel Jacobowitz <drow@false.org>
To: gdb-patches@sourceware.org
Cc: Mark Mitchell <mark@codesourcery.com>
Subject: Re: PATCH: Disable ARM RDP
Date: Fri, 05 May 2006 19:58:00 -0000 [thread overview]
Message-ID: <20060505195846.GP31029@nevyn.them.org> (raw)
In-Reply-To: <20060411200009.GA3235@nevyn.them.org>
On Tue, Apr 11, 2006 at 04:00:09PM -0400, Daniel Jacobowitz wrote:
> I didn't see any comments on this. I'm pretty confident that no one
> uses this protocol any more, so I would like to remove it, unless
> anyone objects.
>
> So, please speak up, if you object to "target rdp" being removed from
> the arm-elf target. No other triplet enables it.
Lacking objections, I plan to apply this patch next week. Does anyone
have a last call for RDP? NEWS addition OK?
--
Daniel Jacobowitz
CodeSourcery
2006-05-05 Daniel Jacobowitz <dan@codesourcery.com>
* remote-rdp.c: Deleted.
* NEWS: Mention removal of remote-rdp.c.
* Makefile.in (ALLDEPFILES): Remove remote-rdp.c.
(remote-rdp.o): Delete.
* README: Remove description of remote-rdp.c.
* arm-tdep.c (arm_breakpoint_from_pc): Remove obsolete comment.
* config/arm/embed.mt (TDEPFILES): Remove remote-rdp.o.
Index: Makefile.in
===================================================================
RCS file: /cvs/src/src/gdb/Makefile.in,v
retrieving revision 1.811
diff -u -p -r1.811 Makefile.in
--- Makefile.in 23 Apr 2006 14:15:01 -0000 1.811
+++ Makefile.in 5 May 2006 19:55:53 -0000
@@ -1449,7 +1449,7 @@ ALLDEPFILES = \
procfs.c \
remote-e7000.c \
remote-hms.c remote-m32r-sdi.c remote-mips.c \
- remote-rdp.c remote-sim.c \
+ remote-sim.c \
remote-st.c remote-utils.c dcache.c \
rs6000-nat.c rs6000-tdep.c \
s390-tdep.c s390-nat.c \
@@ -2485,9 +2485,6 @@ remote-mips.o: remote-mips.c $(defs_h) $
$(gdbcmd_h) $(gdbcore_h) $(serial_h) $(target_h) $(exceptions_h) \
$(remote_utils_h) $(gdb_string_h) $(gdb_stat_h) $(regcache_h) \
$(mips_tdep_h)
-remote-rdp.o: remote-rdp.c $(defs_h) $(inferior_h) $(value_h) \
- $(gdb_callback_h) $(command_h) $(symfile_h) $(remote_utils_h) \
- $(gdb_string_h) $(gdbcore_h) $(regcache_h) $(serial_h) $(arm_tdep_h)
remote-sds.o: remote-sds.c $(defs_h) $(gdb_string_h) $(frame_h) \
$(inferior_h) $(exceptions_h) $(bfd_h) $(symfile_h) $(target_h) \
$(gdbcmd_h) $(objfiles_h) $(gdb_stabs_h) $(gdbthread_h) $(gdbcore_h) \
Index: NEWS
===================================================================
RCS file: /cvs/src/src/gdb/NEWS,v
retrieving revision 1.186
diff -u -p -r1.186 NEWS
--- NEWS 20 Apr 2006 23:18:48 -0000 1.186
+++ NEWS 5 May 2006 19:55:53 -0000
@@ -59,6 +59,8 @@ The ARM rdi-share module.
The Netware NLM debug server.
+The ARM Demon monitor support ("target rdp").
+
*** Changes in GDB 6.4
* New native configurations
Index: README
===================================================================
RCS file: /cvs/src/src/gdb/README,v
retrieving revision 1.35
diff -u -p -r1.35 README
--- README 21 Jan 2006 01:29:03 -0000 1.35
+++ README 5 May 2006 19:55:54 -0000
@@ -454,7 +454,6 @@ monitors and other hardware:
remote-est.c EST emulator
remote-hms.c Renesas Micro Systems H8/300 monitor
remote-mips.c MIPS remote debugging protocol
- remote-rdp.c ARM with Demon monitor
remote-sds.c PowerPC SDS monitor
remote-sim.c Generalized simulator protocol
remote-st.c Tandem ST-2000 monitor
Index: arm-tdep.c
===================================================================
RCS file: /cvs/src/src/gdb/arm-tdep.c,v
retrieving revision 1.208
diff -u -p -r1.208 arm-tdep.c
--- arm-tdep.c 18 Apr 2006 19:20:05 -0000 1.208
+++ arm-tdep.c 5 May 2006 19:55:54 -0000
@@ -1964,11 +1964,6 @@ static const char arm_default_thumb_be_b
necessary) to point to the actual memory location where the
breakpoint should be inserted. */
-/* XXX ??? from old tm-arm.h: if we're using RDP, then we're inserting
- breakpoints and storing their handles instread of what was in
- memory. It is nice that this is the same size as a handle -
- otherwise remote-rdp will have to change. */
-
static const unsigned char *
arm_breakpoint_from_pc (CORE_ADDR *pcptr, int *lenptr)
{
Index: config/arm/embed.mt
===================================================================
RCS file: /cvs/src/src/gdb/config/arm/embed.mt,v
retrieving revision 1.3
diff -u -p -r1.3 embed.mt
--- config/arm/embed.mt 21 Jan 2006 01:29:04 -0000 1.3
+++ config/arm/embed.mt 5 May 2006 19:55:54 -0000
@@ -1,5 +1,5 @@
# Target: ARM embedded system
-TDEPFILES= arm-tdep.o remote-rdp.o
+TDEPFILES= arm-tdep.o
DEPRECATED_TM_FILE= tm-embed.h
SIM_OBS = remote-sim.o
next prev parent reply other threads:[~2006-05-05 19:58 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-03-18 2:40 Mark Mitchell
2006-03-24 21:36 ` Daniel Jacobowitz
2006-04-11 20:00 ` Daniel Jacobowitz
2006-05-05 19:58 ` Daniel Jacobowitz [this message]
2006-05-17 14:53 ` Daniel Jacobowitz
2006-05-17 18:36 ` Eli Zaretskii
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20060505195846.GP31029@nevyn.them.org \
--to=drow@false.org \
--cc=gdb-patches@sourceware.org \
--cc=mark@codesourcery.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox