Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: Richard Earnshaw <rearnsha@arm.com>
To: Andrew Cagney <ac131313@cygnus.com>
Cc: Richard.Earnshaw@arm.com, gdb@sources.redhat.com
Subject: Re: ARM and virtual/raw registers
Date: Sat, 11 May 2002 07:05:00 -0000	[thread overview]
Message-ID: <200205111405.PAA15949@cam-mail2.cambridge.arm.com> (raw)
In-Reply-To: Your message of "Fri, 10 May 2002 15:07:49 EDT." <3CDC1A85.4090603@cygnus.com>

> >> Now obviously, in order to do all this correctly the stack-frame groveller will have to record the information as it unwinds the stack in some private data; but creating this information is part of the stack-unwinding process.
> 
> > 
> > Yes.  The arm would need to implement a custom get_saved_register() architecture method.
> > 
> > Rather than normalize the registers on the way in.  Consider normalizing them them on the way out via a pseudo.  Doing this would mean that the raw register wouldn't be directly visible and the layer below regcache wouldn't need to normalize anything.
> 
> [to expand a little]
> 
> Two target side interfaces (at least) don't have mechanisms for 
> re-aranging (normalizing) registers on the way in:
> 	- sim
> 	- remote
> I suspect that at least short term, doing everything on the gdb-core 
> side will be easier.

No, it wouldn't.  Because then the gdb-core side would have to know *how* 
the target was connected.  That is, the normalization function becomes

	f(value,target-vector);

which complicates things vastly.

> Having the very raw data in the cache may also prove better for 
> debugging - both the raw (in cache) and the normalized (via register 
> read/write) formats are available.  Perhaphs also look at the i387 which 
> normalizes things on the way in (and its bug count).

Let me try and clarify the model that I think we have in gdb; maybe that 
will help to explain why I think doing it on the gdb-core side is 
incorrect.

GDB has three main vectors, but only two are interesting in this case (the 
third being the UI).  The target vector is the lowest level, it provides a 
virtual abstraction of the processor -- registers, memory etc and methods 
for recovering/changing their contents and generally manipulating the 
machine (stepping, breakpoints, etc).  The target vector should present a 
substantially uniform view of the processor regardless of the method used 
to connect to it.  Putting it another way, the target vector presents 
substantially the basic programmers' model of the CPU to the debugger.

The gdbarch vector is, effectively, the ABI model applied to that virtual 
machine; it covers the mapping rules that were used by the compiler when 
it produced the machine code for the user's program: procedure call 
standards, layout of record types, symbolic information etc.  It knows, 
for example, when a double for the variable X is in register F0: it 
doesn't want to have to worry about how F0 is being recovered from the 
physical machine -- in particular it doesn't want to know about how that 
might vary across the various target connections -- that's the 
responsibility of the target vector.

As to whether the sim or remote targets can be made to support this model 
cleanly, I'm not to worried about sim -- we can always fix this in 
sim/arm/wrapper.c, the layer that interfaces between gdb and the simulator 
so that it presents the virtual CPU model directly.  I don't know enough 
about remote.c at this time to know what the problems are there, but I 
suspect that this one will be the least of my worries when I want to start 
adding support for the banked registers as well.  However, if these 
targets can't present the processor abstraction cleanly, then they are 
technically broken ;-)

R.


  reply	other threads:[~2002-05-11 14:05 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-05-09  7:31 Richard Earnshaw
2002-05-09  9:45 ` Andrew Cagney
2002-05-09 10:01   ` Richard Earnshaw
2002-05-09 11:52     ` Andrew Cagney
2002-05-10  3:45       ` Richard Earnshaw
2002-05-10  7:48         ` Andrew Cagney
2002-05-10 12:07           ` Andrew Cagney
2002-05-11  7:05             ` Richard Earnshaw [this message]
2002-05-11 14:52               ` Andrew Cagney
2002-05-12  7:20                 ` Richard Earnshaw
2002-05-12  8:25                   ` Andrew Cagney
2002-05-12  8:30                     ` Richard Earnshaw
2002-05-12  8:51                       ` Andrew Cagney
2002-05-10  9:29       ` Richard Earnshaw
2002-05-10 11:42         ` Andrew Cagney
2002-05-11  6:16           ` Richard Earnshaw
2002-05-11 11:41           ` Richard Earnshaw
2002-05-11 13:36             ` Andrew Cagney
2002-05-12  7:11               ` Richard Earnshaw
2002-05-12  7:40                 ` Richard Earnshaw
2002-05-12  9:03                   ` Andrew Cagney
2002-05-12 11:31                     ` Andrew Cagney
2002-05-12  8:07                 ` Andrew Cagney
2002-05-12  8:25                   ` Richard Earnshaw
2002-05-12  8:41                     ` Andrew Cagney
2002-05-13  5:35                   ` Richard Earnshaw
2002-05-13  6:13                     ` Andrew Cagney
2002-05-13  6:18                       ` Richard Earnshaw
2002-05-09 10:08 ` Andrew Cagney
2002-05-09 10:36   ` Richard Earnshaw

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=200205111405.PAA15949@cam-mail2.cambridge.arm.com \
    --to=rearnsha@arm.com \
    --cc=Richard.Earnshaw@arm.com \
    --cc=ac131313@cygnus.com \
    --cc=gdb@sources.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