]>
Commit | Line | Data |
---|---|---|
4a44dc1a JK |
1 | Contents |
2 | -------- | |
dd3b648e | 3 | |
4a44dc1a JK |
4 | If you find inaccuracies in this list, please send mail to |
5 | [email protected]. | |
dd3b648e | 6 | |
4a44dc1a JK |
7 | * Things to do for Mach. |
8 | * General to do list. | |
dd3b648e | 9 | |
4a44dc1a JK |
10 | Things to do for Mach |
11 | --------------------- | |
12 | ||
01cd70b9 JK |
13 | Note: If mach_port_t is undefined, you have mach2 headers instead of |
14 | mach3 headers. Get the mach3 headers or typedef it to unsigned int. | |
4a44dc1a JK |
15 | |
16 | 0. Get it to compile and run again, especially for non-threaded | |
17 | programs (some of the following are sub-tasks for this). | |
18 | ||
19 | 1. attach_command still contains a call to wait_for_inferior which is | |
20 | wrong for Mach. Need to figure out a way to push this functionality | |
21 | into target_attach (perhaps by having target_attach, for non-Mach | |
22 | targets, call a function which does what is now in attach_command). | |
23 | ||
24 | 2. jtv's port contains an #ifdef which skips the call to | |
25 | insert_step_breakpoint right after SOLIB_CREATE_INFERIOR_HOOK, but | |
26 | goes ahead and calls insert_breakpoints. I don't understand this--the | |
27 | comment would appear to apply to all breakpoints. Perhaps it is an | |
28 | artifact from a previous version of the Mach port? (BTW, the modern | |
29 | equivalent is the call to proceed from m3_create_inferior; proceed | |
30 | inserts breakpoints). | |
31 | ||
32 | 3. Get the thread stuff to use the new generic thread code (enhancing | |
33 | the generic thread code to include any missing features). This is | |
34 | necessary to make thread-specific breakpoints work again. If someone | |
35 | wants to try to patch up the old Mach threads code, need to deal with | |
36 | the hooks for PREPARE_TO_PROCEED and ATTACH_TO_THREAD, which I haven't | |
37 | merged--can these go in target_resume()? | |
38 | ||
39 | 4. BFD problem--"Undefined symbol _aout_32_swap_exec_header_in". | |
df0393e3 | 40 | Believed to be fixed (fix not yet tested with GDB). |
4a44dc1a JK |
41 | |
42 | 5. The linker complains about mfree and so on being multiply defined. | |
df0393e3 | 43 | Believed to be fixed (fix not yet tested). |
4a44dc1a JK |
44 | |
45 | 6. i386_mach3_float_info and register_addr were undefined in the | |
46 | link. I haven't investigated, but probably just another easy | |
01cd70b9 | 47 | configuration thing or something. (possibly already fixed). |
4a44dc1a | 48 | |
01cd70b9 | 49 | 7. Implement the features which CMU gdb has which the main GDB does |
4a44dc1a JK |
50 | not. This could be done by getting paperwork from CMU and merging |
51 | their changes, or by reimplementing them. | |
52 | ||
53 | General To Do List | |
54 | ------------------ | |
55 | ||
56 | This to do list is probably not up to date, and opinions may vary | |
57 | about the importance or even desirability of some of the items. | |
dd3b648e | 58 | |
dd3b648e RP |
59 | It should be possible to use symbols from shared libraries before we know |
60 | exactly where the libraries will be loaded. E.g. "b perror" before running | |
61 | the program. This could maybe be done as an extension of the "breakpoint | |
62 | re-evaluation" after new symbols are loaded. | |
63 | ||
64 | Make single_step() insert and remove breakpoints in one operation. | |
65 | ||
66 | Speed up single stepping by avoiding extraneous ptrace calls. | |
67 | ||
68 | Speed up single stepping by not inserting and removing breakpoints | |
69 | each time the inferior starts and stops. | |
70 | ||
adf2bb58 JG |
71 | Breakpoints should not be inserted and deleted all the time. Only the |
72 | one(s) there should be removed when we have to step over one. Support | |
73 | breakpoints that don't have to be removed to step over them. | |
74 | ||
e05fe481 JK |
75 | Speed up watchpoints by using debug registers, page table diddling (on |
76 | SunOS4, can call mprotect() in the inferior; on other machines can do | |
3fd4045f JK |
77 | something simpler), etc. Note that you need to detect a |
78 | "fast-watchable expression" (i.e., if watching "*p", then either a | |
79 | change to the address pointed to by p or a change to p itself which | |
80 | causes the value of *p to change, is a watchpoint hit). It is | |
81 | possible we will also someday want extensions which are | |
82 | lower-level--"read from these addresses", "write to these addresses", | |
83 | etc., but there is no consensus about just how important these are and | |
84 | exactly what form they would take. There is a consensus that the | |
85 | existing watchpoint semantics should use hardware assists when | |
86 | available. | |
dd3b648e | 87 | |
b1dcd6ac | 88 | Update gdbint.texinfo to include doc on the directory structure and |
dd3b648e RP |
89 | the various tricks of building gdb. |
90 | ||
91 | Do a tutorial in gdb.texinfo on how to do simple things in gdb. | |
92 | E.g. how to set a breakpoint that just prints something and continues. | |
93 | How to break on aborts. Etc. | |
94 | ||
dd3b648e RP |
95 | Provide "voodoo" debugging of core files. This creates a zombie |
96 | process as a child of the debugger, and loads it up with the data, | |
97 | stack, and regs of the core file. This allows you to call functions | |
98 | in the executable, to manipulate the data in the core file. | |
99 | ||
100 | GDB reopens the source file on every line, as you "next" through it. | |
101 | ||
102 | Referencing the vtbl member of a struct doesn't work. It prints OK | |
103 | if you print the struct, but it gets 0 if you try to deref it. | |
104 | ||
105 | Persistent command history: A feature where you could save off a list | |
106 | of the commands you did, so you can edit it into something that will bring | |
adf2bb58 | 107 | the target to the same place every time you source it. |
dd3b648e RP |
108 | This would also be useful for automated fast watchpointing; if you go |
109 | past the place where it watchpoints, you just start it over again and | |
110 | do it more carefully. | |
111 | ||
adf2bb58 JG |
112 | Deal with the SunOS 4.0 and 4.1.1 ptrace bug that loses the registers if |
113 | the stack is paged out. | |
dd3b648e RP |
114 | |
115 | Finish the C++ exception handling stub routines. Lint points them out | |
116 | as unused statics functions. | |
117 | ||
adf2bb58 | 118 | Perhaps "i source" should take an argument like that of "list". |
dd3b648e RP |
119 | |
120 | See if coredep.c's fetch_core_registers can be used on more machines. | |
121 | E.g. MIPS (mips-xdep.c). | |
122 | ||
dd3b648e RP |
123 | unpack_double() does not handle IEEE float on the target unless the host |
124 | is also IEEE. Death on a vax. | |
125 | ||
dd3b648e RP |
126 | Set up interface between GDB and INFO so that you can hop into interactive |
127 | INFO and back out again. When running under Emacs, should use Emacs | |
128 | info, else fork the info program. Installation of GDB should install | |
129 | its texinfo files into the info tree automagically, including the readline | |
adf2bb58 | 130 | texinfo files. |
dd3b648e | 131 | |
adf2bb58 | 132 | "help address" ought to find the "help set print address" entry. |
dd3b648e RP |
133 | |
134 | Remove the VTBL internal guts from printouts of C++ structs, unless | |
135 | vtblprint is set. | |
136 | ||
b1dcd6ac | 137 | Remove "at 0xnnnn" from the "b foo" response, if `print address off' and if |
dd3b648e RP |
138 | it matches the source line indicated. |
139 | ||
140 | The prompt at end of screen should accept space as well as CR. | |
141 | ||
4a44dc1a JK |
142 | "List" should put you into a pseudo-"more" where you can hit space to |
143 | get more, forever to eof. (questionable--you can already hit return | |
144 | to get more, and modal user interfaces are evil -kingdon, 28 Oct | |
145 | 1993). | |
dd3b648e RP |
146 | |
147 | Check STORE_RETURN_VALUE on all architectures. Check near it in tm-sparc.h | |
148 | for other bogosities. | |
149 | ||
150 | Check for storage leaks in GDB, I'm sure there are a lot! | |
151 | ||
152 | vtblprint of a vtbl should demangle the names it's printing. | |
153 | ||
4a44dc1a JK |
154 | Backtrace should point out what the currently selected frame is, in |
155 | its display, perhaps showing "@3 foo (bar, ...)" or ">3 foo (bar, | |
156 | ...)" rather than "#3 foo (bar, ...)". | |
dd3b648e RP |
157 | |
158 | "i program" should work for core files, and display more info, like what | |
159 | actually caused it to die. | |
160 | ||
dd3b648e RP |
161 | "x/10i" should shorten the long name, if any, on subsequent lines. |
162 | ||
163 | Check through the code for FIXME comments and fix them. dbxread.c, | |
164 | blockframe.c, and plenty more. | |
165 | ||
166 | "next" over a function that longjumps, never stops until next time you happen | |
167 | to get to that spot by accident. E.g. "n" over execute_command which has | |
168 | an error. | |
169 | ||
adf2bb58 | 170 | Watchpoints seem not entirely reliable, though they haven't failed me recently. |
dd3b648e RP |
171 | |
172 | "set zeroprint off", don't bother printing members of structs which are entirely | |
173 | zero. Useful for those big structs with few useful members. | |
174 | ||
175 | GDB does four ioctl's for every command, probably switching terminal modes | |
176 | to/from inferior or for readline or something. | |
177 | ||
178 | terminal_ours versus terminal_inferior: cache state. Switch should be a noop | |
179 | if the state is the same, too. | |
180 | ||
181 | ptype $i6 = void??! | |
182 | ||
183 | Clean up invalid_float handling so gdb doesn't coredump when it tries to | |
184 | access a NaN. While this might work on SPARC, other machines are not | |
185 | configured right. | |
186 | ||
187 | "b value_at ; commands ; continue ; end" stops EVERY OTHER TIME! | |
188 | Then once you enter a command, it does the command, runs two more | |
189 | times, and then stops again! Bizarre... (This behaviour has been | |
190 | modified, but it is not yet 100% predictable when e.g. the commands | |
191 | call functions in the child, and while there, the child is interrupted | |
192 | with a signal, or hits a breakpoint.) | |
193 | ||
dd3b648e RP |
194 | help completion, help history should work. |
195 | ||
dd3b648e RP |
196 | Check that we can handle stack trace through varargs AND alloca in same |
197 | function, on 29K. | |
198 | ||
199 | wait_for_inferior loops forever if wait() gives it an error. | |
200 | ||
dd3b648e RP |
201 | "i frame" shows wrong "arglist at" location, doesn't show where the args |
202 | should be found, only their actual values. | |
203 | ||
dd3b648e RP |
204 | There should be a way for "set" commands to validate the new setting |
205 | before it takes effect. | |
206 | ||
dd3b648e RP |
207 | A mess of floating point opcodes are missing from sparc-opcode.h. |
208 | Also, a little program should test the table for bits that are | |
209 | overspecified or underspecified. E.g. if the must-be-ones bits | |
210 | and the must-be-zeroes bits leave some fields unexamined, and the format | |
211 | string leaves them unprinted, then point this out. If multiple | |
212 | non-alias patterns match, point this out too. Finally, there should | |
213 | be a sparc-optest.s file that tries each pattern out. This file | |
214 | should end up coming back the same (modulo transformation comments) | |
215 | if fed to "gas" then the .o is fed to gdb for disassembly. | |
216 | ||
dd3b648e RP |
217 | Eliminate all the core_file_command's in all the xdep files. |
218 | Eliminate separate declarations of registers[] everywhere. | |
219 | ||
220 | "ena d" is ambiguous, why? "ena delete" seems to think it is a command! | |
221 | ||
77eb2e8f JK |
222 | Perhaps move the tdep, xdep, and nat files, into the config |
223 | subdirectories. If not, at least straighten out their names so that | |
dd3b648e RP |
224 | they all start with the machine name. |
225 | ||
226 | inferior_status should include stop_print_frame. It won't need to be | |
227 | reset in wait_for_inferior after bpstat_stop_status call, then. | |
228 | ||
229 | i line VAR produces "Line number not known for symbol ``var''.". I | |
230 | thought we were stashing that info now! | |
231 | ||
dd3b648e RP |
232 | We should be able to write to random files at hex offsets like adb. |
233 | ||
dd3b648e RP |
234 | Make "target xxx" command interruptible. |
235 | ||
236 | Handle add_file with separate text, data, and bss addresses. Maybe | |
237 | handle separate addresses for each segment in the object file? | |
238 | ||
239 | Handle free_named_symtab to cope with multiply-loaded object files | |
240 | in a dynamic linking environment. Should remember the last copy loaded, | |
241 | but not get too snowed if it finds references to the older copy. | |
242 | ||
dd3b648e RP |
243 | The original BFD core dump reading routine would itself coredump when fed |
244 | a garbage file as a core file. Does the current one? | |
245 | ||
dd3b648e | 246 | Generalize and Standardize the RPC interface to a target program, |
adf2bb58 JG |
247 | improve it beyond the "ptrace" interface, and see if it can become a |
248 | standard for remote debugging. | |
dd3b648e RP |
249 | |
250 | Remove all references to: | |
251 | text_offset | |
252 | data_offset | |
253 | text_data_start | |
254 | text_end | |
255 | exec_data_offset | |
256 | ... | |
257 | now that we have BFD. All remaining are in machine dependent files. | |
258 | ||
259 | When quitting with a running program, if a core file was previously | |
260 | examined, you get "Couldn't read float regs from core file"...if | |
261 | indeed it can't. generic_mourn_inferior... | |
262 | ||
4a44dc1a JK |
263 | Have remote targets give a warning on a signal argument to |
264 | target_resume. Or better yet, extend the protocols so that it works | |
265 | like it does on the Unix-like systems. | |
dd3b648e | 266 | |
dd3b648e RP |
267 | Sort help and info output. |
268 | ||
269 | Re-organize help categories into things that tend to fit on a screen | |
270 | and hang together. | |
271 | ||
dd3b648e RP |
272 | renote-nindy.c handles interrupts poorly; it error()s out of badly |
273 | chosen places, e.g. leaving current_frame zero, which causes core dumps | |
274 | on the next command. | |
275 | ||
276 | Add in commands like ADB's for searching for patterns, etc. We should | |
277 | be able to examine and patch raw unsymboled binaries as well in gdb as | |
278 | we can in adb. (E.g. increase the timeout in /bin/login without source). | |
279 | ||
280 | Those xdep files that call register_addr without defining it are | |
281 | probably simply broken. When reconfiguring this part of gdb, I could | |
282 | only make guesses about how to redo some of those files, and I | |
283 | probably guessed wrong, or left them "for later" when I have a | |
284 | machine that can attempt to build them. | |
285 | ||
dd3b648e RP |
286 | When doing "step" or "next", if a few lines of source are skipped between |
287 | the previous line and the current one, print those lines, not just the | |
288 | last line of a multiline statement. | |
289 | ||
290 | When searching for C++ superclasses in value_cast in valops.c, we must | |
291 | not search the "fields", only the "superclasses". There might be a | |
292 | struct with a field name that matches the superclass name. This can | |
293 | happen when the struct was defined before the superclass (before the | |
294 | name became a typedef). | |
295 | ||
36b9d39c JG |
296 | Handling of "&" address-of operator needs some serious overhaul |
297 | for ANSI C and consistency on arrays and functions. | |
298 | For "float point[15];": | |
dd3b648e | 299 | ptype &point[4] ==> Attempt to take address of non-lvalue. |
36b9d39c JG |
300 | For "char *malloc();": |
301 | ptype malloc ==> "char *()"; should be same as | |
302 | ptype &malloc ==> "char *(*)()" | |
303 | call printf ("%x\n", malloc) ==> wierd value, should be same as | |
304 | call printf ("%x\n", &malloc) ==> correct value | |
dd3b648e | 305 | |
4a44dc1a JK |
306 | Fix dbxread.c symbol reading in the presence of interrupts. It |
307 | currently leaves a cleanup to blow away the entire symbol table when a | |
308 | QUIT occurs. (What's wrong with that? -kingdon, 28 Oct 1993). | |
ef98d5ac JG |
309 | |
310 | Mipsread.c reads include files depth-first, because the dependencies | |
311 | in the psymtabs are way too inclusive (it seems to me). Figure out what | |
312 | really depends on what, to avoid recursing 20 or 30 times while reading | |
313 | real symtabs. | |
314 | ||
315 | value_add() should be subtracting the lower bound of arrays, if known, | |
316 | and possibly checking against the upper bound for error reporting. | |
317 | ||
318 | mipsread.c symbol table allocation and deallocation should be checked. | |
319 | My suspicion is that it's full of memory leaks. | |
320 | ||
321 | SunOS should have a target_lookup_symbol() for common'd things allocated | |
322 | by the shared library linker ld.so. | |
323 | ||
324 | When listing source lines, check for a preceding \n, to verify that | |
325 | the file hasn't changed out from under us. | |
326 | ||
327 | When listing source lines, eat leading whitespace corresponding to the | |
328 | line-number prefix we print. This avoids long lines wrapping. | |
329 | ||
330 | mipsread.c needs to check for old symtabs and psymtabs for the same | |
331 | files, the way it happens for dbxread.c and coffread.c, for VxWorks | |
332 | incremental symbol table reloading. | |
333 | ||
4a44dc1a JK |
334 | Get all the remote systems (where the protocol allows it) to be able to |
335 | stop the remote system when the GDB user types ^C (like remote.c | |
336 | does). For ebmon, use ^Ak. | |
ce31a796 JK |
337 | |
338 | Possible feature: A version of the "disassemble" command which shows | |
2af4851f JK |
339 | both source and assembly code ("set symbol-filename on" is a partial |
340 | solution). |