[gdb/testsuite] Fix timeout in gdb.base/valgrind-infcall-2.exp
authorTom de Vries <tdevries@suse.de>
Thu, 29 Apr 2021 21:32:20 +0000 (23:32 +0200)
committerTom de Vries <tdevries@suse.de>
Thu, 29 Apr 2021 21:32:20 +0000 (23:32 +0200)
Since commit 6d5702a5eb3 "Fix test case gdb.base/valgrind-bt.exp" I run into:
...
FAIL: gdb.base/valgrind-infcall-2.exp: target remote for vgdb (timeout)
FAIL: gdb.base/valgrind-infcall-2.exp: monitor v.set gdb_output (timeout)
...

The commit adds this line in proc vgdb_start:
...
    set vgdbcmd "set remotetimeout 3"
...
which has no effect given that the value of var vgdbcmd is not used before
it's overwritten.  We can fix this by doing instead:
...
    set_remotetimeout 3
...

The FAIL I'm observing is fixed by increasing the remotetimeout value to 4.

Tested on x86_64-linux.

gdb/testsuite/ChangeLog:

2021-04-29  Tom de Vries  <tdevries@suse.de>

PR testsuite/27786
* lib/valgrind.exp (vgdb_start): Use set_remotetimeout.  Increase
remotetimeout to 4.

gdb/testsuite/ChangeLog
gdb/testsuite/lib/valgrind.exp

index d4ed915881ba2678fea916856859fe711c39c76e..42ffb651f28de7ae78b6cb459c8b9a2e3660e91b 100644 (file)
@@ -1,3 +1,9 @@
+2021-04-29  Tom de Vries  <tdevries@suse.de>
+
+       PR testsuite/27786
+       * lib/valgrind.exp (vgdb_start): Use set_remotetimeout.  Increase
+       remotetimeout to 4.
+
 2021-04-28  Tom Tromey  <tromey@adacore.com>
 
        * gdb.ada/null_overload.exp: New file.
index caabeda9730441326925f4aa2afb8e329ea9c258..bba338f880c54d572b24116a42f804167ba310ae 100644 (file)
@@ -82,7 +82,7 @@ proc vgdb_start { {active_at_startup 1} } {
 
     clean_restart $testfile
 
-    set vgdbcmd "set remotetimeout 3"
+    set_remotetimeout 4
 
     # Make sure we're disconnected, in case we're testing with the
     # native-extended-gdbserver board, where gdb_start/gdb_load spawn