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

Vladimir Prus wrote:
> On Friday 02 May 2008 21:59:14 Pawel Piech wrote:
>   
>> Vladimir Prus wrote:
>>     
>>> On Friday 02 May 2008 21:31:08 Pawel Piech wrote:
>>>
>>>   
>>>       
>>>> Right now, *stopped reports the thread which hit the breakpoint. In
>>>> all-stop mode, all threads are stopped. In non-stop mode, only that thread
>>>> is stopped.
>>>>
>>>> We surely can add an extra field to indicate which threads are actually stopped,
>>>> if this makes life easier.
>>>>   
>>>>  Thank You :-)  It would be more helpful if the thread-id field was 
>>>>  left alone and always reported the triggering thread as it does now.  
>>>>  It would be better (more consistent and logical) if the new field 
>>>>  was used to indicate whether or not the whole container changed state.    
>>>>     
>>>>         
>>> For avoidance of doubt, do you have any objections to *running using the "thread-id"
>>> as the field name? Since this is new notification, there's no backward compatibility
>>> issues, and there's no issue of which thread originally triggered anything.
>>>
>>> - Volodya
>>>   
>>>       
>> First of all, I think it's important to make the events in all-stop and 
>> non-stop mode to be consistent.  Additionally, it is helpful if the 
>> fields, such as thread-id have consistent meaning even if they are used 
>> in different events.  So for consistency sake it would be better to also 
>> use thread-id to indicate the triggering thread in the running event as 
>> well, and use a separate field to indicate whether the container changed 
>> state.  BTW the triggering thread is needed in the running event, for 
>> example when stepping in the all-stop mode.
>>     
>
> And what the triggering thread would report in that case? The thread where
> the step command was emitted? 
Right.
> Frontend knows it already, no?
>
> - Volodya
>   
Right, it typically does... unless the command was issued through the 
CLI console, in which case the UI has to do some work to get it.  But as 
long as GDB has the information, it's simpler and more reliable to get 
it from the event.

-Pawel


  reply	other threads:[~2008-05-02 18:36 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 [this message]
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

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