[binutils-gdb] mi-sym-info.exp: Increase timeout for 114-symbol-info-functions

Kevin Buettner kevinb@sourceware.org
Fri Jun 11 21:58:39 GMT 2021


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=b8bd29a1575dcac0207ae6f20e7a83533093db7d

commit b8bd29a1575dcac0207ae6f20e7a83533093db7d
Author: Kevin Buettner <kevinb@redhat.com>
Date:   Wed Jun 9 19:52:08 2021 -0700

    mi-sym-info.exp: Increase timeout for 114-symbol-info-functions
    
    Loading libc.so's symbols increased the amount of time needed for
    114-symbol-info-function to fetch symbols, causing a timeout during my
    testing.  I enclosed the entire block with a "with_timeout_factor 4",
    which fixes the problem for me.  (Using 2 also fixed it for me, but it
    might not be enough when running this test on slower machines.)
    
    gdb/testsuite/ChangeLog:
    
            * gdb.mi/mi-sym-info.exp (114-symbol-info-function test): Increase
            timeout.

Diff:
---
 gdb/testsuite/ChangeLog              |  5 ++++
 gdb/testsuite/gdb.mi/mi-sym-info.exp | 56 +++++++++++++++++++-----------------
 2 files changed, 34 insertions(+), 27 deletions(-)

diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index 5f9db54ad02..6cb2ef2acb6 100644
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2021-06-11  Kevin Buettner  <kevinb@redhat.com>
+
+	* gdb.mi/mi-sym-info.exp (114-symbol-info-function test): Increase
+	timeout.
+
 2021-06-11  Kevin Buettner  <kevinb@redhat.com>
 
 	* gdb.base/print-symbol-loading.exp (proc test_load_shlib):
diff --git a/gdb/testsuite/gdb.mi/mi-sym-info.exp b/gdb/testsuite/gdb.mi/mi-sym-info.exp
index 18f85182a7b..dcd6f7d9187 100644
--- a/gdb/testsuite/gdb.mi/mi-sym-info.exp
+++ b/gdb/testsuite/gdb.mi/mi-sym-info.exp
@@ -123,33 +123,35 @@ gdb_test_multiple $cmd $testname -prompt "${mi_gdb_prompt}$" {
 # (from the symbol table).  There's often so much output output from
 # this command that we overflow expect's buffers, avoid this by
 # fetching the output piece by piece.
-set testname "List all functions"
-set cmd "114-symbol-info-functions --include-nondebug"
-set state 0
-gdb_test_multiple $cmd ${testname} -prompt "${mi_gdb_prompt}$" {
-    -re "114\\^done,symbols=\{" {
-	if { $state == 0 } { set state 1 }
-	exp_continue
-    }
-    -re "debug=\\\[${symtab_re}" {
-	if { $state == 1 } { set state 2 }
-	exp_continue
-    }
-    -re ",${symtab_re}" {
-	exp_continue
-    }
-    -re "\\\],nondebug=\\\[" {
-	if { $state == 2 } { set state 3 }
-	exp_continue
-    }
-    -re "\{address=${qstr},name=${qstr}\}," {
-	exp_continue
-    }
-    -re "\{address=${qstr},name=${qstr}\}\\\]\}\r\n${mi_gdb_prompt}$" {
-	if { $state == 3 } {
-	    pass $gdb_test_name
-	} else {
-	    fail $gdb_test_name
+with_timeout_factor 4 {
+    set testname "List all functions"
+    set cmd "114-symbol-info-functions --include-nondebug"
+    set state 0
+    gdb_test_multiple $cmd ${testname} -prompt "${mi_gdb_prompt}$" {
+	-re "114\\^done,symbols=\{" {
+	    if { $state == 0 } { set state 1 }
+	    exp_continue
+	}
+	-re "debug=\\\[${symtab_re}" {
+	    if { $state == 1 } { set state 2 }
+	    exp_continue
+	}
+	-re ",${symtab_re}" {
+	    exp_continue
+	}
+	-re "\\\],nondebug=\\\[" {
+	    if { $state == 2 } { set state 3 }
+	    exp_continue
+	}
+	-re "\{address=${qstr},name=${qstr}\}," {
+	    exp_continue
+	}
+	-re "\{address=${qstr},name=${qstr}\}\\\]\}\r\n${mi_gdb_prompt}$" {
+	    if { $state == 3 } {
+		pass $gdb_test_name
+	    } else {
+		fail $gdb_test_name
+	    }
 	}
     }
 }


More information about the Gdb-cvs mailing list