* PATCH: Disable ARM RDP @ 2006-03-18 2:40 Mark Mitchell 2006-03-24 21:36 ` Daniel Jacobowitz 0 siblings, 1 reply; 6+ messages in thread From: Mark Mitchell @ 2006-03-18 2:40 UTC (permalink / raw) To: gdb-patches This patch removes remote-rdp.o from ARM's TDEPFILES since Daniel and Paul say that it's useless to most people. I also found that with --disable-sim GDB fails to link because remote-rdp.c references a function defined in the simulator. I'm not sure if this is the right patch for mainline, or whether people would rather I remove remote-rdp.c entirely, or do nothing, or ... So, I guess this is a request for approval, but a pretty tentative one. Thanks, -- Mark Mitchell CodeSourcery mark@codesourcery.com (650) 331-3385 x713 2006-03-17 Mark Mitchell <mark@codesourcery.com> * gdb/config/arm/embed.mt (TDEPFILES): Remove remote-rdp.o. Index: gdb/config/arm/embed.mt =================================================================== RCS file: /cvs/src/src/gdb/config/arm/embed.mt,v retrieving revision 1.3 diff -c -5 -p -r1.3 embed.mt *** gdb/config/arm/embed.mt 21 Jan 2006 01:29:04 -0000 1.3 --- gdb/config/arm/embed.mt 17 Mar 2006 21:15:49 -0000 *************** *** 1,6 **** # Target: ARM embedded system ! TDEPFILES= arm-tdep.o remote-rdp.o DEPRECATED_TM_FILE= tm-embed.h SIM_OBS = remote-sim.o SIM = ../sim/arm/libsim.a --- 1,6 ---- # Target: ARM embedded system ! TDEPFILES= arm-tdep.o DEPRECATED_TM_FILE= tm-embed.h SIM_OBS = remote-sim.o SIM = ../sim/arm/libsim.a ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: PATCH: Disable ARM RDP 2006-03-18 2:40 PATCH: Disable ARM RDP Mark Mitchell @ 2006-03-24 21:36 ` Daniel Jacobowitz 2006-04-11 20:00 ` Daniel Jacobowitz 0 siblings, 1 reply; 6+ messages in thread From: Daniel Jacobowitz @ 2006-03-24 21:36 UTC (permalink / raw) To: Mark Mitchell; +Cc: gdb-patches On Fri, Mar 17, 2006 at 02:25:42PM -0800, Mark Mitchell wrote: > > This patch removes remote-rdp.o from ARM's TDEPFILES since Daniel and > Paul say that it's useless to most people. I also found that with > --disable-sim GDB fails to link because remote-rdp.c references a > function defined in the simulator. > > I'm not sure if this is the right patch for mainline, or whether > people would rather I remove remote-rdp.c entirely, or do nothing, or > ... So, I guess this is a request for approval, but a pretty > tentative one. > * gdb/config/arm/embed.mt (TDEPFILES): Remove remote-rdp.o. Well, this patch isn't OK. Afterwards remote-rdp is orphaned - no other configuration uses it. So, either we are removing it, or we aren't. I'm in favor of removing it. Strikes against it: - It doesn't build with --disable-sim. - It doesn't build on some versions of MinGW32, because it references sleep(). - It's an old remote protocol and I haven't seen any reports of it being used. - Using RDP directly is even older than using RDI to talk to the same devices, which we've already removed. I have no real preference on removing it immediately versus marking it obsolete in the next release; the first two bullets are problems for CodeSourcery, but we can always turn it off in our release branch. But I'd like to do one or the other. Anyone have a preference? -- Daniel Jacobowitz CodeSourcery ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: PATCH: Disable ARM RDP 2006-03-24 21:36 ` Daniel Jacobowitz @ 2006-04-11 20:00 ` Daniel Jacobowitz 2006-05-05 19:58 ` Daniel Jacobowitz 0 siblings, 1 reply; 6+ messages in thread From: Daniel Jacobowitz @ 2006-04-11 20:00 UTC (permalink / raw) To: Mark Mitchell, gdb-patches On Fri, Mar 24, 2006 at 03:30:53PM -0500, Daniel Jacobowitz wrote: > On Fri, Mar 17, 2006 at 02:25:42PM -0800, Mark Mitchell wrote: > > > > This patch removes remote-rdp.o from ARM's TDEPFILES since Daniel and > > Paul say that it's useless to most people. I also found that with > > --disable-sim GDB fails to link because remote-rdp.c references a > > function defined in the simulator. > > > > I'm not sure if this is the right patch for mainline, or whether > > people would rather I remove remote-rdp.c entirely, or do nothing, or > > ... So, I guess this is a request for approval, but a pretty > > tentative one. > > > * gdb/config/arm/embed.mt (TDEPFILES): Remove remote-rdp.o. > > Well, this patch isn't OK. Afterwards remote-rdp is orphaned - no > other configuration uses it. So, either we are removing it, or we > aren't. > > I'm in favor of removing it. Strikes against it: > > - It doesn't build with --disable-sim. > - It doesn't build on some versions of MinGW32, because it > references sleep(). > - It's an old remote protocol and I haven't seen any reports of > it being used. > - Using RDP directly is even older than using RDI to talk to > the same devices, which we've already removed. > > I have no real preference on removing it immediately versus marking it > obsolete in the next release; the first two bullets are problems for > CodeSourcery, but we can always turn it off in our release branch. > But I'd like to do one or the other. Anyone have a preference? 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. (I had to apply a global interface change to remote-rdp.c today, which reminded me. Many of these remote targets I'm patching deserve to be removed. Let's make sure to get a complete list before the next release...) -- Daniel Jacobowitz CodeSourcery ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: PATCH: Disable ARM RDP 2006-04-11 20:00 ` Daniel Jacobowitz @ 2006-05-05 19:58 ` Daniel Jacobowitz 2006-05-17 14:53 ` Daniel Jacobowitz 0 siblings, 1 reply; 6+ messages in thread From: Daniel Jacobowitz @ 2006-05-05 19:58 UTC (permalink / raw) To: gdb-patches; +Cc: Mark Mitchell 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: PATCH: Disable ARM RDP 2006-05-05 19:58 ` Daniel Jacobowitz @ 2006-05-17 14:53 ` Daniel Jacobowitz 2006-05-17 18:36 ` Eli Zaretskii 0 siblings, 1 reply; 6+ messages in thread From: Daniel Jacobowitz @ 2006-05-17 14:53 UTC (permalink / raw) To: gdb-patches, Mark Mitchell On Fri, May 05, 2006 at 03:58:46PM -0400, Daniel Jacobowitz wrote: > Lacking objections, I plan to apply this patch next week. Does anyone > have a last call for RDP? NEWS addition OK? > 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. Let's have a moment of silence for RDP... I have applied this to HEAD. I don't have any strong preference as to whether it goes to 6.5 also, so for now I'm not going to touch the branch. Here's the version as committed. (Eli, sorry for not copying you on the original. If you would like any changes to the NEWS entry, please let me know!) -- Daniel Jacobowitz CodeSourcery 2006-05-17 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.816 diff -u -p -r1.816 Makefile.in --- Makefile.in 15 May 2006 21:53:20 -0000 1.816 +++ Makefile.in 17 May 2006 14:39:36 -0000 @@ -1468,7 +1468,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 \ @@ -2504,9 +2504,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.187 diff -u -p -r1.187 NEWS --- NEWS 16 May 2006 19:43:57 -0000 1.187 +++ NEWS 17 May 2006 14:39:37 -0000 @@ -3,6 +3,10 @@ *** Changes since GDB 6.5 +* REMOVED features + +The ARM Demon monitor support (RDP protocol, "target rdp"). + *** Changes in GDB 6.5 * New targets 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 17 May 2006 14:39:37 -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 17 May 2006 14:39:37 -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 17 May 2006 14:39:38 -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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: PATCH: Disable ARM RDP 2006-05-17 14:53 ` Daniel Jacobowitz @ 2006-05-17 18:36 ` Eli Zaretskii 0 siblings, 0 replies; 6+ messages in thread From: Eli Zaretskii @ 2006-05-17 18:36 UTC (permalink / raw) To: gdb-patches, Mark Mitchell > Date: Wed, 17 May 2006 10:42:06 -0400 > From: Daniel Jacobowitz <drow@false.org> > > (Eli, sorry for not copying you on the original. If you would like any > changes to the NEWS entry, please let me know!) No, it's fine. ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2006-05-17 18:13 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2006-03-18 2:40 PATCH: Disable ARM RDP Mark Mitchell 2006-03-24 21:36 ` Daniel Jacobowitz 2006-04-11 20:00 ` Daniel Jacobowitz 2006-05-05 19:58 ` Daniel Jacobowitz 2006-05-17 14:53 ` Daniel Jacobowitz 2006-05-17 18:36 ` Eli Zaretskii
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox