Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: Pedro Alves <pedro@codesourcery.com>
To: Vladimir Prus <vladimir@codesourcery.com>
Cc: gdb@sourceware.org,  Pawel Piech <pawel.piech@windriver.com>,
	 Marc Khouzam <marc.khouzam@ericsson.com>
Subject: Re: MI non-stop interface details
Date: Thu, 01 May 2008 17:52:00 -0000	[thread overview]
Message-ID: <200805011852.25316.pedro@codesourcery.com> (raw)
In-Reply-To: <200805012121.51174.vladimir@codesourcery.com>

Vladimir,

A Thursday 01 May 2008 18:21:51, Vladimir Prus wrote:

> [Pedro and I discussed this a bit on IRC, and...]
>
> On IRC, you said:
>
>    [20:40] <pedro> in CLI it's lame to have to know the thread id, and to
> have to pass it to "continue". [20:43] <pedro> break 20; <hit breakpoint>;
> next; next; <hits other bkreapoint in another thread>; another next; one
> step; continue; uuups, other thread resumed too...
>
> I think this is valid point -- if user, in CLI, enables non-stop, then
> having "continue" just work, resuming one thread, seems right. Are you
> going to add "--all" to "continue"?

Sure!  I hereby volunteer to do that.  :-)

> Going back to -exec-continue, it's 
> probably best to make it behave the same way as "continue", in non-stop
> mode. This means that a frontend will have to conditionally pass "--all" to
> -exec-continue, but in any way, if a frontend enables non-stop mode, is
> surely has to adjust some of the logic, and changing -exec-continue logic
> does not seem too hard.
>

Exactly.  It's the same amount of work for a frontend writer either way.

If "-exec-continue" (without parameters) resumes all:

The frontend has to add new "resume one thread only button", which 
calls "-exec-continue --thread=x".  The current "resume" button,
already calling "-exec-continue", would resume all threads.

If "-exec-continue" (without parameters) resumes only current thread:

The frontend has to add new "resume all threads" button which 
calls "-exec-continue --thread=all".  The current "resume" button,
already calling "-exec-continue", would resume only the current
thread.

It's the same amount of work for a frontend writer, because
he'd have to write support for one of the new buttons.  In
an hypothetical world, where one could only have access to one
of these buttons, the most useful and general would be the
"resume (current thread)" one.  This means, that if the
frontend writer is lazy, it's better this way :-)

In my simplistic view of the frontend's world, I'd imagine that since
there's a *running notification telling the frontend which threads
resumed, the button actions doesn't have to care much which of the
behaviours (one thread or all-threads) it is triggering.  It's
the notification callback that has to care.

Thanks,
-- 
Pedro Alves


      reply	other threads:[~2008-05-01 17:52 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-26 18:09 Vladimir Prus
2008-04-26 18:16 ` Doug Evans
2008-04-26 19:49   ` Vladimir Prus
2008-04-30  7:18     ` Marc Khouzam
2008-04-28 16:21 ` Pedro Alves
2008-04-29 19:21   ` Vladimir Prus
2008-04-29 20:04     ` Pedro Alves
2008-04-30  7:00       ` Pawel Piech
2008-05-01 16:15         ` Vladimir Prus
2008-05-01 16:31           ` Pawel Piech
2008-05-01 16:38             ` Vladimir Prus
2008-05-01 16:58               ` Pawel Piech
     [not found]               ` <4819F4D4.4010305@windriver.com>
2008-05-01 17:00                 ` Vladimir Prus
2008-05-01 17:53                   ` Pawel Piech
2008-05-01 18:12                     ` Vladimir Prus
2008-05-01 18:37                       ` Pawel Piech
2008-05-02  1:23                       ` Evolution of GDB/MI [was Re: MI non-stop interface details] Nick Roberts
2008-04-30 14:23       ` MI non-stop interface details Marc Khouzam
2008-04-30 17:21         ` Pedro Alves
     [not found]         ` <200804301117.42633.vladimir@codesourcery.com>
     [not found]           ` <4818AA58.4040201@windriver.com>
2008-05-01 17:11             ` Vladimir Prus
2008-05-01 18:08               ` Pawel Piech
2008-05-02 14:21                 ` Vladimir Prus
2008-05-02 16:59                   ` Pawel Piech
2008-05-02 17:13                     ` Vladimir Prus
     [not found]                       ` <481B4FDC.4010802@windriver.com>
2008-05-02 17:36                         ` Vladimir Prus
2008-05-02 18:00                           ` Pawel Piech
2008-05-02 18:19                             ` Vladimir Prus
2008-05-02 18:36                               ` Pawel Piech
2008-04-29  3:14 ` Pawel Piech
     [not found] ` <200804301059.44112.vladimir@codesourcery.com>
     [not found]   ` <200804301534.48779.pedro@codesourcery.com>
2008-05-01 17:22     ` Vladimir Prus
2008-05-01 17:52       ` Pedro Alves [this message]

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=200805011852.25316.pedro@codesourcery.com \
    --to=pedro@codesourcery.com \
    --cc=gdb@sourceware.org \
    --cc=marc.khouzam@ericsson.com \
    --cc=pawel.piech@windriver.com \
    --cc=vladimir@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