From: Kevin Buettner <kevinb@redhat.com>
To: Daniel Jacobowitz <drow@mvista.com>, "J. Johnston" <jjohnstn@redhat.com>
Cc: Kevin Buettner <kevinb@redhat.com>,
Marcel Moolenaar <marcel@xcllnt.net>,
ac131313@redhat.com, gdb-patches@sources.redhat.com
Subject: Re: RFA: ia64 portion of libunwind patch
Date: Wed, 29 Oct 2003 04:48:00 -0000 [thread overview]
Message-ID: <1031029044805.ZM4842@localhost.localdomain> (raw)
In-Reply-To: Daniel Jacobowitz <drow@mvista.com> "Re: RFA: ia64 portion of libunwind patch" (Oct 28, 8:28pm)
On Oct 28, 8:28pm, Daniel Jacobowitz wrote:
> On Tue, Oct 28, 2003 at 06:53:36PM -0500, J. Johnston wrote:
> > I have addressed your comments below. The bfd stuff (map_segment,
> > map_info) has been replaced by a simple call to bfd_bread(). I have moved
> > the getunwind syscall stuff into ia64-linux-tdep.c and I access it via the
> > gdbarch_tdep structure.
>
> Nothing which involves a syscall is acceptable in a tdep file. That's
> what the t means - target support.
>
> Move it to the -nat file.
Yes, I agree with Daniel. Earlier, I had said:
... and the code for ia64_getunwind() will have to go in
ia64-linux-nat.c. Presumably, if a remote target wanted to use
the unwind library, there'd need to be some remote protocol
modifications.
I also asked the following questions:
Hmm... thinking about this some more, I'd like to know why a
syscall is required. Is there any way this functionality could be
implemented in an OS independent fashion?
I'm guessing that there's kernel state that the unwinder needs to be
made aware of and that it won't be possible to implement ia64_getunwind()
in an OS independent manner. (I'd like someone to verify this though...)
> I don't know how you should integrate it with the gdbarch tdep.
If my speculation above is correct, I don't think the gdbarch tdep
ought to be used for accessing ia64_getunwind(). This sort of smells
like something that belongs in the target vector. (Since we need
different mechanisms for the remote and native cases.) But, since
this functionality is highly ia64-centric, I'd really hate to pollute
the target vector with such a method. The only approach that comes
to mind (at the moment) is something similar to the
native_find_global_pointer hook in ia64-tdep.c. (AIX/rs6000 has
something similar.) Note though that this kind of hook doesn't
help with the native / remote problem.
Again, it'd be really, *really*, REALLY nice if we could come up with
an OS independent way of determining the necessary information. E.g.
Is there any way to do memory reads to obtain the necessary information?
(I'm guessing not, but it doesn't hurt to ask...)
Kevin
next prev parent reply other threads:[~2003-10-29 4:48 UTC|newest]
Thread overview: 65+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-10-24 0:11 J. Johnston
2003-10-24 17:57 ` Kevin Buettner
2003-10-24 18:20 ` J. Johnston
2003-10-24 18:56 ` Kevin Buettner
2003-10-24 21:53 ` Marcel Moolenaar
2003-10-24 23:58 ` Kevin Buettner
2003-10-28 23:53 ` J. Johnston
2003-10-29 1:28 ` Daniel Jacobowitz
2003-10-29 4:48 ` Kevin Buettner [this message]
2003-10-29 18:43 ` J. Johnston
2003-10-29 22:48 ` Andrew Cagney
2003-11-04 19:09 ` J. Johnston
2003-11-04 20:48 ` Kevin Buettner
2003-11-06 17:21 ` RFA: ia64 portion of libunwind patch updated J. Johnston
2003-11-14 21:24 ` J. Johnston
2003-11-17 17:01 ` Kevin Buettner
2003-11-17 17:03 ` Kevin Buettner
2003-11-17 17:05 ` Kevin Buettner
2003-11-17 21:40 ` J. Johnston
2003-11-17 22:32 ` Marcel Moolenaar
2003-11-06 20:03 ` [commit] Fix two xfer partial bugs; Was; RFA: ia64 portion of libunwind patch Andrew Cagney
2003-11-06 20:12 ` J. Johnston
2003-11-06 22:32 ` [patch/rfc] Add child_to_xfer_partial; Was: " Andrew Cagney
2003-11-06 22:47 ` J. Johnston
2003-11-07 21:40 ` Andrew Cagney
2003-11-07 1:04 ` Kevin Buettner
2003-11-14 0:26 ` RFA: " J. Johnston
2003-11-14 1:17 ` Kevin Buettner
2003-11-14 20:49 ` J. Johnston
2003-10-29 23:28 ` Andrew Cagney
2003-11-02 20:39 ` Elena Zannoni
2003-10-29 15:18 ` Andrew Cagney
2003-10-31 19:25 J. Johnston
2003-10-31 20:46 ` Andrew Cagney
2003-10-31 22:55 ` David Mosberger
2003-11-07 21:47 ` Andrew Cagney
2003-11-07 22:43 ` David Mosberger
2003-11-07 23:01 ` Andrew Cagney
2003-11-07 23:12 ` David Mosberger
2003-11-07 23:38 ` Andrew Cagney
2003-11-07 23:55 ` David Mosberger
2003-11-08 0:07 ` Andrew Cagney
2003-11-08 0:13 ` Kevin Buettner
2003-11-08 0:27 ` Andrew Cagney
2003-11-08 7:21 ` David Mosberger
2003-11-09 0:13 ` Andrew Cagney
2003-11-10 22:10 ` David Mosberger
2003-11-10 22:43 ` Andrew Cagney
2003-11-10 23:01 ` David Mosberger
2003-11-26 0:11 ` David Mosberger
2003-12-04 2:15 ` David Mosberger
2003-12-04 3:15 ` Kevin Buettner
2003-12-04 23:57 ` J. Johnston
2003-12-05 0:39 ` David Mosberger
2003-12-10 20:58 ` J. Johnston
2003-12-10 22:15 ` David Mosberger
2003-12-12 22:25 ` Kevin Buettner
[not found] ` <davidm@napali.hpl.hp.com>
2003-12-13 4:01 ` Kevin Buettner
2003-11-09 1:34 ` Marcel Moolenaar
2003-11-10 21:54 ` David Mosberger
2003-11-10 23:18 ` Marcel Moolenaar
2003-10-31 21:36 ` Marcel Moolenaar
2003-10-31 23:00 ` David Mosberger
2003-10-31 23:42 ` Andrew Cagney
2003-10-31 23:59 ` David Mosberger
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=1031029044805.ZM4842@localhost.localdomain \
--to=kevinb@redhat.com \
--cc=ac131313@redhat.com \
--cc=drow@mvista.com \
--cc=gdb-patches@sources.redhat.com \
--cc=jjohnstn@redhat.com \
--cc=marcel@xcllnt.net \
/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