espressif/binutils-gdb
1
0
Fork 0
mirror of https://github.com/espressif/binutils-gdb.git synced 2025-06-22 11:00:01 +08:00
Code Issues Packages Projects Releases Wiki Activity

Skip attach-many-short-lived-threads.exp on known-broken DejaGnu versions

Browse Source 
If the testsuite is run with a DejaGnu version that predates the fix
from last year:

  [PATCH] DejaGnu kills the wrong process due to PID-reuse races
  http://lists.gnu.org/archive/html/dejagnu/2015-07/msg00005.html

... gdb.threads/attach-many-short-lived-threads.exp fails randomly,
often.  Other tests randomly fail due to that issue too, but this one
is _much_ more exposed.

DejaGnu 1.6 was released meanwhile, which includes that DejaGnu fix,
and also some distros backported the fix too.

So skip the test when run with older/broken DejaGnus.

gdb/testsuite/ChangeLog:
2016-05-27  Pedro Alves  <palves@redhat.com>

	* gdb.threads/attach-many-short-lived-threads.exp (bad_dejagnu):
	New procedure.
	(top level): Call it, and bail out of DejaGnu is known to be bad.
This commit is contained in:
Pedro Alves
2016-05-27 16:18:28 +01:00
parent e92bae6260
commit 744608cc85
2 changed files with 40 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
gdb/testsuite/ChangeLog
View File

@ -1,3 +1,9 @@
2016-05-27 Pedro Alves <palves@redhat.com>
* gdb.threads/attach-many-short-lived-threads.exp (bad_dejagnu):
New procedure.
(top level): Call it, and bail out of DejaGnu is known to be bad.
2016-05-27 Andrew Burgess <andrew.burgess@embecosm.com>
* gdb.base/whatis.c: Extend the test case.

34
gdb/testsuite/gdb.threads/attach-many-short-lived-threads.exp
View File

@ -21,6 +21,40 @@
# end up leaving stale state behind that confuse the following
# attach).
# Return true if the running version of DejaGnu is known to not be
# able to run this test.
proc bad_dejagnu {} {
global frame_version
verbose -log "DejaGnu version: $frame_version"
verbose -log "Expect version: [exp_version]"
verbose -log "Tcl version: [info tclversion]"
set dj_ver [split $frame_version .]
set dj_ver_major [lindex $dj_ver 0]
set dj_ver_minor [lindex $dj_ver 1]
# DejaGnu versions prior to 1.6 manage to kill the wrong process
# due to PID-reuse races. Since this test spawns many threads, it
# widens the race window a whole lot, enough that the inferior is
# often killed, and thus the test randomly fails. See:
# http://lists.gnu.org/archive/html/dejagnu/2015-07/msg00005.html
# The fix added a close_wait_program procedure. If that procedure
# is defined, and DejaGnu is older than 1.6, assume that means the
# fix was backported.
if {$dj_ver_major == 1
&& ($dj_ver_minor < 6 && [info procs close_wait_program] == "")} {
return 1
}
return 0
}
if {[bad_dejagnu]} {
unsupported "broken DejaGnu"
return 0
}
if {![can_spawn_for_attach]} {
return 0
}