Subject: Re: CVS commit: src/gnu/dist/gdb6/gdb
To: None <source-changes@netbsd.org>
From: Christos Zoulas <christos@astron.com>
List: source-changes
Date: 12/08/2006 18:18:40
In article <200612081801.TAA0000053167@zel459.zel.kfa-juelich.de>,
Matthias Drochner  <M.Drochner@fz-juelich.de> wrote:
>
>nick.hudson@dsl.pipex.com said:
>> > Update code for recognition of signal trampolines to the new
>> > "setcontext"
>> [...]
>> Shouldn't we keep the recognition of the old style signal handler and 
>> possibliy recognise the 3 versions of __sigtramp_siginfo_2 that have
>> existed at various points?
>
>I haven't seen how to do this in the gdb code cleanly -- the
>initialisation of register offsets etc is done only once in
>target specific initialisation.
>
>
>thorpej@shagadelic.org said:
>> That's what I intended with the versioning. 
>
>The version information is not there when it is needed. It
>is not present in the stackframe, and even if it was, it would
>be hard to make use of it - see above.
>
>best regards
>Matthias

We discussed this briefly with Matthias before he committed the
code. Unfortunately there is no clean way for gdb to recognize
multiple trampolines, so we decided that it was better to make it
recognize the current trampoline code, instead of the one that has
not been used for a while. It is noteworthy that nobody has stepped
up so far to fix gdb for all the rest of the platforms that need
this, and also the kernel stack crawling fixes.

christos