This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [commit] Improved linker-debugger interface
- From: Sergio Durigan Junior <sergiodj at redhat dot com>
- To: Joel Brobecker <brobecker at adacore dot com>
- Cc: gdb-patches at sourceware dot org, Tom Tromey <tromey at redhat dot com>, Jan Kratochvil <jan dot kratochvil at redhat dot com>, Pedro Alves <palves at redhat dot com>
- Date: Tue, 25 Jun 2013 18:13:04 -0300
- Subject: Re: [commit] Improved linker-debugger interface
- References: <20130516144340 dot GA2105 at blade dot nx> <20130604133819 dot GA25892 at blade dot nx> <20130625205350 dot GA28973 at adacore dot com>
On Tuesday, June 25 2013, Joel Brobecker wrote:
>> I just committed the improved linker-debugger interface patch series
>> as attached. For those of you using the patches in distributions,
>> some of the attached files have different filenames than previously
>> posted:
>
> I am seeing some issues on ia64-linux (rhES5):
>
> | (gdb) start
> | Temporary breakpoint 1 at 0x40000000000005a1: file bar.c, line 4.
> | Starting program: /saco.a/users/brobecke/ex/simple/bar
> | warning: Probes-based dynamic linker interface failed.
> | Reverting to original interface.
> |
> | Unknown numeric token on expression `0 8@r35'.
>
> I will investigate the problem with the unknown numeric token, but
> any recommendation before I start?
Hi Joel,
Thanks for the report. There are two kinds of things happening here.
First, the current SDT parser (implemented on stap-probe.c) is not
recognizing one of the probe's arguments. This is where the "Unknown
numeric token" comes from. The initial "0" is making the parser
confused, and to be honest it is also making me confused :-). I did not
implement the support for IA-64, FWIW.
I am looking into this issue right away.
> Also, I am wondering we really want a warning in this case -
> I think this is going to make the average user think that there
> is something wrong and therefore that needs to be fixed. What
> do people think?
The second thing is the warning. It is issues by Gary's patch, and I
agree that it could confuse users. However, I think it is still a good
thing to have. Maybe one could create a debug flag that would enable
printing such warnings? Just an idea.
I will get back to you when I have more news about what's going on with
the SDT parser.
Thanks,
--
Sergio