From: Andrew Cagney <ac131313@ges.redhat.com>
To: Keith Seitz <keiths@redhat.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: [RFA] register_modified_event
Date: Tue, 20 Aug 2002 13:46:00 -0000 [thread overview]
Message-ID: <3D62AA88.5080508@ges.redhat.com> (raw)
In-Reply-To: <Pine.LNX.4.44.0208200841510.1421-100000@valrhona.uglyboxes.com>
> On Mon, 19 Aug 2002, Andrew Cagney wrote:
>
>
>> I don't the register_modified (the changelog called it register_update)
>> is correct. Trying to supply a register number, for instance, is just
>> misleading.
>
>
> (ChangeLog is wrong, of course. It should be register_modified, not
> register_update.) As for supplying register numbers, I think you'd
> better take another look at MI's command set. Everything (except
> data-list-register-names) returns register numbers. In fact, there is no
> way to find out what the numbers mean at all with the current MI command
> set!
(We're not talking about register numbers vs names. You can figure out
a name/number map from the command that returns the register names. The
current implementation is broken in that on some targets, it will leave
gaps. That can be fixed by either changing the interface or the code :-)
The problem here is that trying to include the ``modified register'' in
the event is misleading to the point of being dangerous. The best MI
can offer the GUI is ``hey, target-changed!''. Every time the target
changes, the GUI needs to refresh everything via varobj.
> I presume that you mean this thread:
>
> http://sources.redhat.com/ml/gdb/2002-03/msg00154.html
>
> I guess Jim thought there were bigger fish to fry. So if a global
> "target-changed" event is wanted, when is it to be sent?
My understanding of JimI's comments (and I agree) is that nothing
matters except how long it takes for the GUI to recover from a target
changed (eg from inferior execution).
Given that single step refresh performance sets an upper bound on on
memory/register write refresh performance, people will always notice a
slow single-step long before they notice slow writes.
> When:
These change the target:
> o register changed by user (either via command-line or other UI)
> o memory changed by user (")
> o inferior execution
These do not change the target:
> o user changes threads (thread_command)
> o user changes stack levels (frame_command, up_command, down_command)
See also:
http://sources.redhat.com/gdb/papers/multi-arch/real-multi-arch/index.html#SEC33
enjoy,
Andrew
next prev parent reply other threads:[~2002-08-20 20:46 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-08-19 15:17 Keith Seitz
2002-08-19 15:49 ` Andrew Cagney
2002-08-20 9:36 ` Keith Seitz
2002-08-20 13:46 ` Andrew Cagney [this message]
2002-08-20 15:36 ` Keith Seitz
2002-08-20 17:23 ` Andrew Cagney
2002-08-21 8:34 ` Keith Seitz
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=3D62AA88.5080508@ges.redhat.com \
--to=ac131313@ges.redhat.com \
--cc=gdb-patches@sources.redhat.com \
--cc=keiths@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