Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
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



  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