Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Tom Tromey <tromey@redhat.com>
To: Marc Khouzam <marc.khouzam@ericsson.com>
Cc: "'Pedro Alves'" <palves@redhat.com>,
	"'André Pönitz'" <andre.poenitz@mathematik.tu-chemnitz.de>,
	"'Yao Qi'" <yao@codesourcery.com>,
	"'gdb-patches@sourceware.org'" <gdb-patches@sourceware.org>
Subject: Re: [RFC] MI notification on register changes
Date: Wed, 07 Nov 2012 18:54:00 -0000	[thread overview]
Message-ID: <87sj8li6r2.fsf@fleche.redhat.com> (raw)
In-Reply-To: <E59706EF8DB1D147B15BECA3322E4BDC0183CC@EUSAAMB103.ericsson.se>	(Marc Khouzam's message of "Wed, 7 Nov 2012 18:18:18 +0000")

>>>>> "Marc" == Marc Khouzam <marc.khouzam@ericsson.com> writes:

Marc> Since GDB allows the frontend to try to avoid a full
Marc> refresh by giving extra information in its notifications,
Marc> the idea of continuing with that approach seemed the right one.
Marc> For instance, GDB could have a single =breakpoints-changed
Marc> notif which would trigger a full refresh a breakpoints.
Marc> Same for variable objects (see -var-update).  Even
Marc> =threads-changed (although this may be a little more
Marc> justified if we think about it more deeply).

In some cases, like breakpoints, gdb has perfect information.
In these situations, sending small changes is easy and efficient.

For memory or registers, I think the PR established pretty clearly that
gdb doesn't have perfect information, or at the very least, that trying
to compute it perfectly would be a lot of work, and so it is preferable
to avoid doing so if possible.

For varobjs, gdb caches the previously sent data so that it can send
deltas.  I think this could be done with other the entities in question
-- registers, memory, stack frames -- if the communications overhead of
a full update is the rate-limiting step.  On the other hand, if the
computation in gdb (including communication with the inferior) is
rate-limiting, then this won't help at all.

Threads are a kind of funny case, see the "finding new threads in the
inferior" item here: http://sourceware.org/gdb/wiki/LocalRemoteFeatureParity

Tom


  reply	other threads:[~2012-11-07 18:54 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
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 [this message]
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=87sj8li6r2.fsf@fleche.redhat.com \
    --to=tromey@redhat.com \
    --cc=andre.poenitz@mathematik.tu-chemnitz.de \
    --cc=gdb-patches@sourceware.org \
    --cc=marc.khouzam@ericsson.com \
    --cc=palves@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