Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: "André Pönitz" <andre.poenitz@mathematik.tu-chemnitz.de>
To: Yao Qi <yao@codesourcery.com>
Cc: Tom Tromey <tromey@redhat.com>, gdb-patches@sourceware.org
Subject: Re: [RFC] MI notification on register changes
Date: Tue, 06 Nov 2012 20:00:00 -0000	[thread overview]
Message-ID: <20121106200012.GA2591@klara.mpi.htwm.de> (raw)
In-Reply-To: <5098A88B.5070005@codesourcery.com>

On Tue, Nov 06, 2012 at 02:04:59PM +0800, Yao Qi wrote:
> On 11/06/2012 04:10 AM, Tom Tromey wrote:
> >I think the implication is that nearly any change can require the front
> >end to need to reset everything and so trying to make gdb differentiate
> >between the various events will never work out.  So, gdb might as well
> >emit something very generic rather than =register-changed.
> >
> 
> If GDB emits a very generic notification (for register change,
> memory change, etc), I am not sure how useful it is.  It would be
> noisy to frondend, IMO.

Why?

As soon as a single bit changes somewhere, the only formally correct
response from a frontend is to re-fetch everything.

One can collect several such notification before triggering a full
refetch (no user will insist on updates everyt 50ms, and gdb won't
deliver that anyway), or one can take some risk and not refetch
everything, which in most cases would even be "good enough", but
in general a refetch is needed. 

[...]
> My rationale here is like this: for the gdb internal state update,
> false positive (notify observers, but nothing is changed in fact) is
> fine and false negative (something is changed but observers are not
> notified) is not allowed.  For the external notification to clients,
> false negative is fine (it is a limitation of gdb, and will only
> happen in some corner cases), but false positive is noisy.

I wouldn't call "false external positives" noisy, unless they generate
so much traffic that there is a performance bottleneck. A frontend
can consciously decide to ignore them.

Andre'


  reply	other threads:[~2012-11-06 20:00 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-23  7:24 Yao Qi
2012-11-01  0:36 ` ping: " Yao Qi
2012-11-01 20:48 ` Tom Tromey
2012-11-02  9:56   ` Yao Qi
2012-11-02 16:42     ` Tom Tromey
2012-11-05 10:26       ` Yao Qi
2012-11-05 20:10         ` Tom Tromey
2012-11-06  6:05           ` Yao Qi
2012-11-06 20:00             ` André Pönitz [this message]
2012-11-07 15:35               ` Marc Khouzam
2012-11-07 17:01                 ` Pedro Alves
2012-11-07 18:18                   ` Marc Khouzam
2012-11-07 18:54                     ` Tom Tromey
2012-11-07 21:09                       ` André Pönitz
2012-11-09  1:47                   ` Yao Qi

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=20121106200012.GA2591@klara.mpi.htwm.de \
    --to=andre.poenitz@mathematik.tu-chemnitz.de \
    --cc=gdb-patches@sourceware.org \
    --cc=tromey@redhat.com \
    --cc=yao@codesourcery.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