Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Daniel Jacobowitz <drow@false.org>
To: Ulrich Weigand <uweigand@de.ibm.com>
Cc: gdb-patches@sourceware.org
Subject: Re: [rfc/rft] [3/4] Untangle register_addr - v2 - mips-linux
Date: Wed, 25 Apr 2007 19:45:00 -0000	[thread overview]
Message-ID: <20070425192458.GA2235@caradoc.them.org> (raw)
In-Reply-To: <200704250016.l3P0GifL014998@d12av02.megacenter.de.ibm.com>

On Wed, Apr 25, 2007 at 02:16:44AM +0200, Ulrich Weigand wrote:
> I had assumed people would be building GDB for a different target in
> such cases -- if the mips-linux target is used, I agree my proposed
> patch is broken.

I can't say for sure whether they should be, but I know I and others
use a foo-linux gdb to talk to kgdb, or to talk to associated JTAG
stubs.  Of course, we might be able to make GDB consider that a
different OSABI.  But it's not easy to tell a static linked program
from a kernel image.

> 
> So I guess we're back to distinguishing between a gdbarch method of
> providing registers that cannot be fetched and stored, and in addition
> a target_ops method -- which in the case of a target using the 
> trad_ptrace helpers should be made available somehow ...
> 
> In any case, I still like to get the bulk of the register_addr patch
> set committed soon -- the whole cannot_fetch_register discussion is
> really an independent topic.  The following patch contains just the
> part to move register_addr to mips-linux-nat.c and leaves the
> CANNOT_FETCH_REGISTER situation completely unchanged.  Would that
> be OK with you for now?

Yes, this patch looks fine to me (with the Makefile.in dependency
update, of course).

-- 
Daniel Jacobowitz
CodeSourcery


  reply	other threads:[~2007-04-25 19:25 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-23 15:47 Ulrich Weigand
2007-04-24 21:53 ` Daniel Jacobowitz
2007-04-25  0:26   ` Ulrich Weigand
2007-04-25 19:45     ` Daniel Jacobowitz [this message]
2007-04-25 22:40       ` Ulrich Weigand

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=20070425192458.GA2235@caradoc.them.org \
    --to=drow@false.org \
    --cc=gdb-patches@sourceware.org \
    --cc=uweigand@de.ibm.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