Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Andrew Cagney <cagney@gnu.org>
To: Jonathan Larmour <jifl@eCosCentric.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: powerpc remote target registers
Date: Mon, 01 Dec 2003 18:52:00 -0000	[thread overview]
Message-ID: <3FCB8DDC.30603@gnu.org> (raw)
In-Reply-To: <3FC7FF5D.7060906@eCosCentric.com>

> Now, arguably this is a problem with GCC/GAS/GLD: there is a machine type specifically for mpc860 with the correct register definitions, instead of "common" with the correct register definitions. However GCC/GAS don't allow you to set that when compiling objects, and LD appears to ignore my request to set it explicitly when linking (using -A powerpc:mpc860). That shouldn't really be needed anyway, as it should be implied by a compile with -mcpu=860, but GCC doesn't pass anything to GAS/the linker to reflect that. Sigh.
> 
> However I would say that GDB is also mistaken for not initializing the packet buffer in remote_fetch_registers() to 0 first, so that registers that aren't supplied by the remote target don't have uninitialised data, which may include an "x" in them. 

I don't think this fixes the bug.  It will fill each nibble of the 
altivec registers with (0 - '0') instead of zero (you should see this 
with "maint print raw-registers").

Suggest instead changing the for loop filling in regs[] so that it 
doesn't run off the end of the NUL terminated buf[] (I think this is the 
real bug).

Andrew



  reply	other threads:[~2003-12-01 18:52 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-11-29  2:07 Jonathan Larmour
2003-12-01 18:52 ` Andrew Cagney [this message]
2003-12-02  5:23   ` Jonathan Larmour
2003-12-03  4:23     ` Andrew Cagney
2003-12-04  7:32       ` Jonathan Larmour
2003-12-04 15:11         ` Andrew Cagney
2003-12-06 22:08           ` Jonathan Larmour
2003-12-06 22:58             ` 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=3FCB8DDC.30603@gnu.org \
    --to=cagney@gnu.org \
    --cc=gdb-patches@sources.redhat.com \
    --cc=jifl@eCosCentric.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