gdb and binutils branch master updated. 32990adaadc1b119700cd0dfd2dd8849114e0135

[palves at sourceware dot org]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "gdb and binutils".

The branch, master has been updated
       via  32990adaadc1b119700cd0dfd2dd8849114e0135 (commit)
      from  dac3fe877844e76de9cf102151837998aa1d2a8f (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=32990adaadc1b119700cd0dfd2dd8849114e0135

commit 32990adaadc1b119700cd0dfd2dd8849114e0135
Author: Pedro Alves <palves@redhat.com>
Date:   Thu Oct 2 09:55:38 2014 +0100

    Reduce Hg packet (select remote general thread) bouncing
    
    A patch I wrote made GDB pull the thread list sooner when debugging
    with target remote, and I noticed an intended consequence.  GDB
    started bouncing around the currently selected remote/general thread
    more frequently.  E.g.:
    
      Sending packet: $qTMinFTPILen#3b...Packet received: 5
     +Sending packet: $Hgp726d.726d#53...Packet received: OK
      Sending packet: $m400680,40#2f...Packet received: 85c0741455bff00d60004889e5ffd05de97bffffff0f1f00e973ffffff0f1f00554889e5c745fc00000000c745fc01000000e900000000c745fc02000000b800
     +Sending packet: $Hgp726d.7278#28...Packet received: OK
      Sending packet: $m4006b2,1#28...Packet received: e9
      Fast tracepoint 2 at 0x4006b2: file gdb/testsuite/gdb.trace/range-stepping.c, line 53.
      Sending packet: $qTStatus#49...Packet received: T0;tnotrun:0;tframes:0;tcreated:0;tfree:500000;tsize:500000;circular:0;disconn:0;starttime:0;stoptime:0;username:;notes::
    
    This ended up breaking "tstart" when one has fast tracepoints set,
    because gdbserver isn't expecting an Hg packet in response to
    qRelocInsn:
    
     (gdb) ftrace *set_point
     Fast tracepoint 3 at 0x4006b2: file gdb/testsuite/gdb.trace/range-stepping.c, line 53.
     (gdb) PASS: gdb.trace/range-stepping.exp: ftrace: ftrace *set_point
     tstart
     gdbserver: Malformed response to qRelocInsn, ignoring: Hgp2783.2783
    
     Target does not support this command.
     (gdb) FAIL: gdb.trace/range-stepping.exp: ftrace: tstart
    
    remote_trace_start should probably start by making sure the remote
    current thread matches inferior_ptid (calling set_general_thread), but
    still, reducing unnecessary bouncing is a good idea.  It happens
    because the memory/symbol/breakpoint routines use
    switch_to_program_space_and_thread to do something in the right
    context and then revert back to the previously current thread.
    
    The fix is to simply make any_thread_of_process,
    find_inferior_for_program_space, etc. give preference to the current
    thread/inferior it if matches.
    
    gdb/
    2014-10-02  Pedro Alves  <palves@redhat.com>
    
    	* gdbthread.h (any_thread_of_process, any_live_thread_of_process):
    	Adjust comments.
    	* inferior.c (find_inferior_for_program_space): Give preference to
    	the current inferior.
    	* inferior.h (find_inferior_for_program_space): Update comment.
    	* progspace.c (switch_to_program_space_and_thread): Prefer the
    	current inferior if it's bound to the program space requested.  If
    	the inferior found doesn't have a PID yet, don't bother looking up
    	a thread.
    	* progspace.h (switch_to_program_space_and_thread): Adjust
    	comment.
    	* thread.c (any_thread_of_process, any_live_thread_of_process):
    	Give preference to the current thread.

-----------------------------------------------------------------------

Summary of changes:
 gdb/ChangeLog   |   16 ++++++++++++++++
 gdb/gdbthread.h |    7 ++++---
 gdb/inferior.c  |    7 +++++--
 gdb/inferior.h  |    3 ++-
 gdb/progspace.c |    5 ++---
 gdb/progspace.h |    3 ++-
 gdb/thread.c    |   40 ++++++++++++++++++++++++++++++++++------
 7 files changed, 65 insertions(+), 16 deletions(-)


hooks/post-receive
-- 
gdb and binutils