This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[patch] Fix testsuite annotate-quit race (PR 544)
- From: Jan Kratochvil <jan dot kratochvil at redhat dot com>
- To: gdb-patches at sourceware dot org
- Date: Tue, 18 Mar 2008 23:54:36 +0100
- Subject: [patch] Fix testsuite annotate-quit race (PR 544)
Hi,
PR 544 des gdb.cp/annota2.exp and gdb.cp/annota3.exp sometimes FAIL with:
FAIL: gdb.cp/annota3.exp: annotate-quit (pattern 1)
It gets reliably reproducible for me (Fedora 8 x86_64) if GDB gets straced:
mv gdb gdb-orig
wget -O gdb http://cvs.jankratochvil.net/viewcvs/nethome/src/gdb-wrap?rev=HEAD
The problem is that if usually CTRL-C is hit during poll(2) and everything is
fine. But if CTRL-C is hit inside readline processing its RL_SIGNAL_HANDLER
gets called which calls RL_FREE_LINE_STATE and $gdb_prompt gets displayed twice
and GDB_EXPECT or GDB_EXPECT_LIST are not patient enough to see later
^Z^Zerror-begin+Quit+^Z^Zquit.
Unaware how to easily detect a failure without timing out.
Regards,
Jan
2008-03-18 Jan Kratochvil <jan.kratochvil@redhat.com>
PR 544
* gdb.cp/annota2.exp, gdb.cp/annota3.exp: Fix a race while recognizing
`annotate-quit'.
--- ./gdb/testsuite/gdb.cp/annota2.exp 28 Feb 2008 16:24:25 -0000 1.10
+++ ./gdb/testsuite/gdb.cp/annota2.exp 18 Mar 2008 22:41:28 -0000
@@ -205,14 +205,13 @@ gdb_expect {
# test:
# annotate-quit
#
-# This test sometimes fails, but not reproducibly. See gdb/544.
+# We may get "$gdb_prompt$" earlier as it may get printed several times during
+# CTRL-C if we catch its output early enough.
#
send_gdb "\003"
gdb_expect {
-re "\r\n\032\032error-begin\r\nQuit\r\n\r\n\032\032quit\r\n$gdb_prompt$" \
{ pass "annotate-quit" }
- -re "$gdb_prompt$" { kfail "gdb/544" "annotate-quit" }
- -re ".*$gdb_prompt$" { fail "annotate-quit" }
timeout { fail "annotate-quit (timeout)" }
}
--- ./gdb/testsuite/gdb.cp/annota3.exp 1 Jan 2008 22:53:19 -0000 1.11
+++ ./gdb/testsuite/gdb.cp/annota3.exp 18 Mar 2008 22:41:28 -0000
@@ -208,13 +208,14 @@ gdb_expect {
# test:
# annotate-quit
#
-# This test sometimes fails, but not reproducibly. See gdb/544.
+# We may get "$gdb_prompt$" earlier as it may get printed several times during
+# CTRL-C if we catch its output early enough.
#
send_gdb "\003"
-gdb_expect_list "annotate-quit" "$gdb_prompt$" {
- "\r\n\032\032error-begin\r\n"
- "Quit\r\n"
- "\r\n\032\032quit\r\n"
+gdb_expect {
+ -re "\r\n\032\032error-begin\r\nQuit\r\n\r\n\032\032quit\r\n$gdb_prompt$" \
+ { pass "annotate-quit" }
+ timeout { fail "annotate-quit (timeout)" }
}
#