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. 4a40f85a84f08f7fc6abe06115df9342efdc53c5


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  4a40f85a84f08f7fc6abe06115df9342efdc53c5 (commit)
      from  86db008d5af464bb3e13113d92a21fcac5965ff1 (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=4a40f85a84f08f7fc6abe06115df9342efdc53c5

commit 4a40f85a84f08f7fc6abe06115df9342efdc53c5
Author: Maciej W. Rozycki <macro@codesourcery.com>
Date:   Tue Sep 9 16:39:16 2014 +0100

    GDB/testsuite: Avoid timeout lowering
    
    The recent change to introduce `gdb_reverse_timeout' turned out
    ineffective for board setups that set the `gdb,timeout' target variable.
    A lower `gdb,timeout' setting takes precedence and defeats the effect of
    `gdb_reverse_timeout'.  This is because the global timeout is overridden
    in gdb_test_multiple and then again in gdb_expect.
    
    Three timeout variables are taken into account in these two places, in
    this precedence:
    
    1. The `gdb,timeout' target variable.
    
    2. The caller's local `timeout' variable (upvar timeout)
    
    3. The global `timeout' variable.
    
    This precedence is obeyed by gdb_test_multiple strictly.  OTOH
    gdb_expect will select the higher of the two formers and will only take
    the latter into account if none of the formers is present.  However the
    two timeout selections are conceptually the same and gdb_test_multiple
    does its only for the purpose of passing it down to gdb_expect.
    
    Therefore I decided there is no point to keep carrying on this
    duplication and removed the sequence from gdb_test_multiple, however
    retaining the `upvar timeout' variable definition.  This way gdb_expect
    will still access gdb_test_multiple's caller `timeout' variable (if any)
    via its own `upvar timeout' reference.
    
    Now as to the sequence in gdb_expect.  In addition to the three
    variables described above it also takes a timeout argument into account,
    as the fourth value to choose from.  It is currently used if it is
    higher than the timeout selected from the variables as described above.
    
    With the timeout selection code from gdb_test_multiple gone, gone is
    also the most prominent use of this timeout argument, it's now used in
    a couple of places only, mostly within this test framework library code
    itself for preparatory commands or suchlike.  With this being the case
    this timeout selection code can be simplified as follows:
    
    1. Among the three timeout variables, the highest is always chosen.
       This is so that a test case doesn't inadvertently lower a high value
       timeout needed by slow target boards.  This is what all test cases
       use.
    
    2. Any timeout argument takes precedence.  This is for special cases
       such as within the framework library code, e.g. it doesn't make sense
       to send `set height 0' with a timeout of 7200 seconds.  This is a
       local command that does not interact with the target and setting a
       high timeout here only risks a test suite run taking ages if it goes
       astray for some reason.
    
    3. The fallback timeout of 60s remains.
    
    	* lib/gdb.exp (gdb_test_multiple): Remove code to select the
    	timeout, don't pass one down to gdb_expect.
    	(gdb_expect): Rework timeout selection.

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

Summary of changes:
 gdb/testsuite/ChangeLog   |    6 ++++
 gdb/testsuite/lib/gdb.exp |   57 +++++++++++++-------------------------------
 2 files changed, 23 insertions(+), 40 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]