This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] remote-sim add call to init_thread_list


On 11/10/2012 3:31 PM, Pedro Alves wrote:
> On 10/03/2012 01:05 PM, Andrew Burgess wrote:
>> When running using a basic exec style target (on linux) I see the following behaviour:
>>
>>> gdb ./test-program
>> (gdb) start
>> (gdb) info threads
>>   Id   Target Id         Frame 
>> * 1    process 12285     main () at hello.c:5
>> (gdb) start
>> The program being debugged has been started already.
>> Start it from the beginning? (y or n) y
>>
>> (gdb) info threads 
>>   Id   Target Id         Frame 
>> * 1    process 16295     main () at hello.c:5
>>
>> ## END ##
>>
>> Notice that the thread ID remains as 1 after the restart.  This is due to the call to init_thread_list in gdb/fork-child.c:fork_inferior.
>>
>> Currently, when using a simulator target, the ID will increment on each restart.
>>
>> The patch below adds a similar call to init_thread_list to remote-sim.c:create_inferior so the simulator behaviour will match the native exec style target behaviour.
>>
>> OK to apply?
> 
> OK.

Committed.

Thanks for the review.
Andrew





Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]