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

[commit] Fix timeouts in break-interp.exp on slow machines


Hello,

since Jan checked in his recent set of patches, break-interp.exp test cases
have started to succeed on my ARM machines.  However, I was still seeing
timeouts in the "info files" tests.

This is once again caused by a gdb_expect clause with a ".*" wildcard that
matches a large number of lines.  This can cause expect to slow down
significantly, to the extent that the tests time out on slow machines.

Fixed along the lines of similiar tests elsewhere by separating the
statement into two different gdb_expect patterns, which eliminates
the need for this particular wildcard.

Tested on i386-linux and armv7l-linux-gnueabi with no regressions,
fixes the timeouts on ARM.

Committed to mainline.

Bye,
Ulrich


ChangeLog:

	* break-interp.exp (test_ld): Use two separate gdb_expect statements
	for the "info files" test to avoid timeouts on slow machines.


Index: gdb/testsuite/gdb.base/break-interp.exp
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.base/break-interp.exp,v
retrieving revision 1.22
diff -u -p -r1.22 break-interp.exp
--- gdb/testsuite/gdb.base/break-interp.exp	11 Oct 2010 08:47:47 -0000	1.22
+++ gdb/testsuite/gdb.base/break-interp.exp	12 Oct 2010 13:38:57 -0000
@@ -442,14 +442,23 @@ proc test_ld {file ifmain trynosym displ
     if $ifmain {
 	reach "_dl_debug_state" run $displacement
 
+	# Use two separate gdb_expect statements to avoid timeouts due to
+	# slow processing of wildcard capturing long output
 	set test "info files"
 	set entrynohex ""
-	gdb_test_multiple $test $test {
-	    -re "\r\n\[\t \]*Entry point:\[\t \]*0x(\[0-9a-f\]+)\r\n.*$gdb_prompt $" {
+	send_gdb "$test\n"
+	gdb_expect  {
+	    -re "\r\n\[\t \]*Entry point:\[\t \]*0x(\[0-9a-f\]+)\r\n" {
 		set entrynohex $expect_out(1,string)
-		pass $test
+	        gdb_expect {
+		    -re "$gdb_prompt $" { pass $test }
+		    timeout { fail "$test (timeout)" }
+		}
 	    }
+	    -re ".*$gdb_prompt $"  { fail $test }
+	    timeout         { fail "$test (timeout)" }
 	}
+
 	# `info sym' cannot be tested for .opd as the binary may not have
 	# symbols.
 	if {[istarget powerpc64-*] && [is_lp64_target]} {
-- 
  Dr. Ulrich Weigand
  GNU Toolchain for Linux on System z and Cell BE
  Ulrich.Weigand@de.ibm.com


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