From: Andrew Cagney <cagney@gnu.org>
To: Kevin Buettner <kevinb@redhat.com>
Cc: Andrew Cagney <ac131313@redhat.com>, gdb-patches@sources.redhat.com
Subject: Re: [rfa:solib] Handle start-address descriptors
Date: Wed, 29 Oct 2003 15:47:00 -0000 [thread overview]
Message-ID: <3F9E8F40.10205@gnu.org> (raw)
In-Reply-To: <1031029042111.ZM4799@localhost.localdomain>
>> > Hmm, a possible problem... What happens when the target uses function
>> > descriptors, but not for the exec file's start address? I'm wondering
>> > (ugh) if a separate gdbarch method is required for obtaining the start
>> > address.
>
>>
>> Fortunatly a previous patch addressed that problem. The convert
>> function is only applied to positively identified descriptors.
>
>
> If I'm not mistaken, this problem has been addressed for ppc64 and
> ia64, but not in a generic fashion. I'm concerned about some other
> ABI (for some other architecture) which has function descriptors, but
> has no easy way to discriminate between descriptors and code
> addresses. If this ABI specifies that the entry point should refer to
> a code address instead of a descriptor, then we need a more
> complicated mechanism -- perhaps that separate gdbarch method that I
> mentioned earlier.
> I suppose we can wait to do something about this problem until it
> actually arises. But... since we've identified a potential problem,
> we should at least document it in some appropriate location. If
> the convert_from_func_ptr_addr() method *must* (due to the way it
> is used elsewhere) contain a discrimination mechanism, then this
> should be mentioned in the documentation. (BTW, I don't see this
> method documented in gdbint.texinfo.)
Can I encourage you to write this up and add it to the doco? The method
most affects the ia64 and ppc/rs6000 targets that you maintain. There
is no value-add in taking it any further though.
I'll commit the solib-svr4.c change stuff once I've drained a few other
pending patches.
Andrew
next prev parent reply other threads:[~2003-10-29 15:47 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-10-27 18:45 Andrew Cagney
2003-10-27 18:57 ` Kevin Buettner
2003-10-27 19:11 ` Andrew Cagney
2003-10-28 21:55 ` Kevin Buettner
2003-10-28 23:50 ` Andrew Cagney
2003-10-29 4:21 ` Kevin Buettner
2003-10-29 15:47 ` Andrew Cagney [this message]
2003-10-31 21:15 ` Andrew Cagney
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=3F9E8F40.10205@gnu.org \
--to=cagney@gnu.org \
--cc=ac131313@redhat.com \
--cc=gdb-patches@sources.redhat.com \
--cc=kevinb@redhat.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