This is the mail archive of the gdb-cvs@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]

gdb and binutils branch master updated. 03d469572472d5c59d8a35030bb1b8072c217dc7


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  03d469572472d5c59d8a35030bb1b8072c217dc7 (commit)
      from  e558d7c1095545832a90e76f72e6db6c98fdee0f (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=03d469572472d5c59d8a35030bb1b8072c217dc7

commit 03d469572472d5c59d8a35030bb1b8072c217dc7
Author: Pedro Alves <palves@redhat.com>
Date:   Mon Sep 22 11:12:30 2014 +0100

    infrun.c:user_visible_resume_ptid: Don't check singlestep_breakpoints_inserted_p
    
    What matters for this function, is whether the user requested a
    "step", for "set scheduler-locking step", not whether GDB is doing an
    internal step for some reason.
    
     /* Return a ptid representing the set of threads that we will proceed,
        in the perspective of the user/frontend.  */
     extern ptid_t user_visible_resume_ptid (int step);
    
    Therefore, the check for singlestep_breakpoints_inserted_p is actually
    incorrect, and we end up applying schedlock more often on sss targets
    than on non-sss targets.
    
    Found by inspection while working on a patch that eliminates the
    singlestep_breakpoints_inserted_p global.
    
    Tested on x86_64 Fedora 20 on top of my 'software single-step on x86'
    series.
    
    gdb/
    2014-09-25  Pedro Alves  <palves@redhat.com>
    
    	* infrun.c (user_visible_resume_ptid): Don't check
    	singlestep_breakpoints_inserted_p.

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

Summary of changes:
 gdb/ChangeLog |    5 +++++
 gdb/infrun.c  |    3 +--
 2 files changed, 6 insertions(+), 2 deletions(-)


hooks/post-receive
-- 
gdb and binutils


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