From 56a8e183d289735ab75fa73260ec60523522ba18 Mon Sep 17 00:00:00 2001 From: Pedro Alves Date: Thu, 2 Apr 2009 15:43:10 +0000 Subject: [PATCH] * gdb.mi/mi-nonstop.exp: Don't check isnative. Use mi_run_to_main. Skip thread exit test on remote targets. * gdb.mi/mi-nsintrall.exp: Don't check isnative. Use mi_run_to_main. * gdb.mi/mi-nsmoribund.exp: Don't check isnative. Use mi_run_to_main. * lib/mi-support.exp (mi_gdb_target_cmd): Remove trailing anchor when expecting ^connected. Detect when the target doesn't support non-stop mode. (mi_run_cmd): Detect if non-stop mode was requested by isn't supported. Return -1 on error, 0 on success. (mi_runto_helper): Don't expect a stop if mi_run_cmd failed. (mi_runto): Return mi_runto_helper's result explicitly. --- gdb/testsuite/ChangeLog | 16 ++++++ gdb/testsuite/gdb.mi/mi-nonstop.exp | 73 +++++++++----------------- gdb/testsuite/gdb.mi/mi-nsintrall.exp | 36 ++----------- gdb/testsuite/gdb.mi/mi-nsmoribund.exp | 36 ++----------- gdb/testsuite/lib/mi-support.exp | 28 +++++++--- 5 files changed, 67 insertions(+), 122 deletions(-) diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 1e131c9f3d4..de2205aee82 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,19 @@ +2009-04-02 Pedro Alves + + * gdb.mi/mi-nonstop.exp: Don't check isnative. Use + mi_run_to_main. Skip thread exit test on remote targets. + * gdb.mi/mi-nsintrall.exp: Don't check isnative. Use + mi_run_to_main. + * gdb.mi/mi-nsmoribund.exp: Don't check isnative. Use + mi_run_to_main. + * lib/mi-support.exp (mi_gdb_target_cmd): Remove trailing anchor + when expecting ^connected. Detect when the target doesn't support + non-stop mode. + (mi_run_cmd): Detect if non-stop mode was requested by isn't + supported. Return -1 on error, 0 on success. + (mi_runto_helper): Don't expect a stop if mi_run_cmd failed. + (mi_runto): Return mi_runto_helper's result explicitly. + 2009-04-01 Pedro Alves * gdb.threads/pending-step.c, gdb.threads/pending-step.exp: New. diff --git a/gdb/testsuite/gdb.mi/mi-nonstop.exp b/gdb/testsuite/gdb.mi/mi-nonstop.exp index 08952b0a610..18c08df6e04 100644 --- a/gdb/testsuite/gdb.mi/mi-nonstop.exp +++ b/gdb/testsuite/gdb.mi/mi-nonstop.exp @@ -14,11 +14,6 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . -# This only works with native configurations -if {![isnative]} { - return -} - load_lib mi-support.exp set MIFLAGS "-i=mi" @@ -46,9 +41,9 @@ set supported 0 send_gdb "-gdb-show non-stop\n" gdb_expect { -re ".*\\^done,value=\"off\",supported=\"(\[^\"\]+)\"\r\n$mi_gdb_prompt$" { - if { $expect_out(1,string) == "1" } { - set supported 1 - } + if { $expect_out(1,string) == "1" } { + set supported 1 + } } -re ".$mi_gdb_prompt$" { } @@ -58,34 +53,10 @@ mi_gdb_test "-gdb-set non-stop 1" ".*" mi_gdb_test "-gdb-set target-async 1" ".*" detect_async -mi_gdb_test "200-break-insert -t main" ".*" - -set created "=thread-created,id=\"$decimal\"\r\n" -set running "\\*running,thread-id=\"$decimal\"\r\n" - -set notifs "($created)*($running)*($library_loaded_re)*" - -# Note: presently, we skip this test on non-native targets, -# so 'run' is OK. As soon as we start to run this on remote -# target, the logic from mi_run_cmd will have to be refactored. -send_gdb "-exec-run\n" -gdb_expect { - -re "\\^running\r\n$notifs$mi_gdb_prompt" { - } - -re "\\^error,msg=\"The target does not support running in non-stop mode.\"" { - verbose -log "Non-stop mode not supported, skipping all tests" - return - } - -re "\r\n$mi_gdb_prompt" { - perror "Cannot start target (unknown output after running)" - return -1 - } - timeout { - perror "Cannot start target (timeout)" - return -1 - } +if { [mi_run_to_main] < 0 } { + perror "mi-nonstop.exp tests suppressed" + continue } -mi_expect_stop "breakpoint-hit" main ".*" ".*" "\[0-9\]+" { "" "disp=\"del\"" } "run to main" mi_create_breakpoint break_at_me 2 keep break_at_me .* .* .* "breakpoint at marker" @@ -114,13 +85,13 @@ mi_varobj_update * {I_W1 I_W0} "update varobj, 1" mi_check_varobj_value I_W0 1 "check varobj, w0, 1" mi_check_varobj_value I_W1 1 "check varobj, w1, 1" -send_gdb "-exec-interrupt --thread 1\n" +send_gdb "-exec-interrupt --thread 1\n" gdb_expect { -re "\\^done\r\n$mi_gdb_prompt\\*stopped\[^\r\n\]+\r\n$" { - pass "interrupted" + pass "interrupted" } timeout { - fail "interrupted (timeout)" + fail "interrupted (timeout)" } } # The interrupt command sends SIGINT to the target, and therefore the @@ -132,10 +103,10 @@ mi_check_thread_states {"stopped" "stopped" "stopped"} "thread state, stop 4" send_gdb "-exec-continue --all\n" gdb_expect { -re ".*\\*running,thread-id=\"3\"\r\n\\*running,thread-id=\"2\"\r\n\\*running,thread-id=\"1\"\r\n$mi_gdb_prompt" { - pass "resume all" + pass "resume all" } timeout { - fail "resume all (timeout)" + fail "resume all (timeout)" } } @@ -172,12 +143,20 @@ mi_gdb_test "-stack-list-frames --thread 3" \ mi_gdb_test "-thread-select 2" "\\^done.*" "select first worker thread" # Since thread 2 is running, we need to set variable via another thread. mi_gdb_test "-gdb-set --thread 3 variable exit_first_thread=1" ".*\\^done" "ask the second thread to exit" -gdb_expect { - -re ".*=thread-exited,id=\"2\",group-id=\"\[0-9\]+\"\r\n$" { - pass "wait for thread exit" - } - timeout { - fail "wait for thread exit (timeout)" + +set test "wait for thread exit" +if { [is_remote target] } { + # The remote protocol doesn't have support for thread exit + # notifications. + unsupported $test +} else { + gdb_expect { + -re ".*=thread-exited,id=\"2\",group-id=\"\[0-9\]+\"\r\n$" { + pass $test + } + timeout { + fail "$test (timeout)" + } } } @@ -186,6 +165,4 @@ mi_gdb_test "-stack-list-frames --thread 3" \ "\\^done,stack=\\\[frame={level=\"0\",addr=\".*\",func=\"break_at_me\".*" \ "stacktrace of stopped thread" - mi_gdb_exit - diff --git a/gdb/testsuite/gdb.mi/mi-nsintrall.exp b/gdb/testsuite/gdb.mi/mi-nsintrall.exp index d34b08d4e78..0b904176bb4 100644 --- a/gdb/testsuite/gdb.mi/mi-nsintrall.exp +++ b/gdb/testsuite/gdb.mi/mi-nsintrall.exp @@ -13,11 +13,6 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . -# This only works with native configurations -if {![isnative]} { - return -} - load_lib mi-support.exp set MIFLAGS "-i=mi" @@ -57,35 +52,10 @@ mi_gdb_test "-gdb-set non-stop 1" ".*" mi_gdb_test "-gdb-set target-async 1" ".*" detect_async -mi_gdb_test "200-break-insert -t main" ".*" - -set created "=thread-created,id=\"$decimal\"\r\n" -set running "\\*running,thread-id=\"$decimal\"\r\n" - -set notifs "($created)*($running)*($library_loaded_re)*" - -# Note: presently, we skip this test on non-native targets, -# so 'run' is OK. As soon as we start to run this on remote -# target, the logic from mi_run_cmd will have to be refactored. -send_gdb "-exec-run\n" -gdb_expect { - -re "\\^running\r\n$notifs$mi_gdb_prompt" { - } - -re "\\^error,msg=\"The target does not support running in non-stop mode.\"" { - verbose -log "Non-stop mode not supported, skipping all tests" - return - } - -re "\r\n$mi_gdb_prompt" { - perror "Cannot start target (unknown output after running)" - return -1 - } - timeout { - perror "Cannot start target (timeout)" - return -1 - } +if { [mi_run_to_main] < 0 } { + perror "mi-nsintrall.exp tests suppressed" + continue } -mi_expect_stop "breakpoint-hit" main ".*" ".*" "\[0-9\]+" \ - { "" "disp=\"del\"" } "run to main" mi_create_breakpoint thread_function 2 keep thread_function .* .* .* \ "breakpoint at thread_function" diff --git a/gdb/testsuite/gdb.mi/mi-nsmoribund.exp b/gdb/testsuite/gdb.mi/mi-nsmoribund.exp index 505fc849bf1..8ed53e083be 100644 --- a/gdb/testsuite/gdb.mi/mi-nsmoribund.exp +++ b/gdb/testsuite/gdb.mi/mi-nsmoribund.exp @@ -13,11 +13,6 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . -# This only works with native configurations -if {![isnative]} { - return -} - load_lib mi-support.exp set MIFLAGS "-i=mi" @@ -57,35 +52,10 @@ mi_gdb_test "-gdb-set non-stop 1" ".*" mi_gdb_test "-gdb-set target-async 1" ".*" detect_async -mi_gdb_test "200-break-insert -t main" ".*" - -set created "=thread-created,id=\"$decimal\"\r\n" -set running "\\*running,thread-id=\"$decimal\"\r\n" - -set notifs "($created)*($running)*($library_loaded_re)*" - -# Note: presently, we skip this test on non-native targets, -# so 'run' is OK. As soon as we start to run this on remote -# target, the logic from mi_run_cmd will have to be refactored. -send_gdb "-exec-run\n" -gdb_expect { - -re "\\^running\r\n$notifs$mi_gdb_prompt" { - } - -re "\\^error,msg=\"The target does not support running in non-stop mode.\"" { - verbose -log "Non-stop mode not supported, skipping all tests" - return - } - -re "\r\n$mi_gdb_prompt" { - perror "Cannot start target (unknown output after running)" - return -1 - } - timeout { - perror "Cannot start target (timeout)" - return -1 - } +if { [mi_run_to_main] < 0 } { + perror "mi-nsmoribund.exp tests suppressed" + continue } -mi_expect_stop "breakpoint-hit" main ".*" ".*" "\[0-9\]+" \ - { "" "disp=\"del\"" } "run to main" # Keep this in sync with THREADS in the the $srcfile. set nthreads 10 diff --git a/gdb/testsuite/lib/mi-support.exp b/gdb/testsuite/lib/mi-support.exp index f62c2402c96..c31111b3ebe 100644 --- a/gdb/testsuite/lib/mi-support.exp +++ b/gdb/testsuite/lib/mi-support.exp @@ -333,7 +333,7 @@ proc mi_gdb_target_cmd { targetname serialport } { for {set i 1} {$i <= 3} {incr i} { send_gdb "47-target-select $targetname $serialport\n" gdb_expect 60 { - -re "47\\^connected.*$mi_gdb_prompt$" { + -re "47\\^connected.*$mi_gdb_prompt" { verbose "Set target to $targetname"; return 0; } @@ -365,6 +365,10 @@ proc mi_gdb_target_cmd { targetname serialport } { sleep 5 continue } + -re "Non-stop mode requested, but remote does not support non-stop.*$mi_gdb_prompt" { + unsupported "Non-stop mode not supported" + return 1 + } -re "Timeout reading from remote system.*$mi_gdb_prompt$" { verbose "Got timeout error from gdb."; } @@ -787,13 +791,13 @@ proc mi_run_cmd {args} { -re "$mi_gdb_prompt$" { } default { perror "gdb_init_command for target failed"; - return; + return -1; } } } if { [mi_gdb_target_load] < 0 } { - return + return -1 } if [target_info exists use_gdb_stub] { @@ -803,7 +807,7 @@ proc mi_run_cmd {args} { -re "220\\^running\[\r\n\]+\\*running,thread-id=\"\[^\"\]+\"\r\n$mi_gdb_prompt" {} default {} } - return; + return 0; } if [target_info exists gdb,start_symbol] { @@ -816,19 +820,25 @@ proc mi_run_cmd {args} { # to better handle RUN. send_gdb "jump *$start\n" warning "Using CLI jump command, expect run-to-main FAIL" - return + return 0 } send_gdb "220-exec-run $args\n" gdb_expect { -re "220\\^running\r\n(\\*running,thread-id=\"\[^\"\]+\"\r\n|=thread-created,id=\"1\",group-id=\"\[0-9\]+\"\r\n)*(${library_loaded_re})*(${thread_selected_re})?${mi_gdb_prompt}" { } + -re "\\^error,msg=\"The target does not support running in non-stop mode.\"" { + unsupported "Non-stop mode not supported" + return -1 + } timeout { perror "Unable to start target" - return + return -1 } } # NOTE: Shortly after this there will be a ``000*stopped,...(gdb)'' + + return 0 } # @@ -884,7 +894,9 @@ proc mi_runto_helper {func run_or_continue} { } if {$run_or_continue == "run"} { - mi_run_cmd + if { [mi_run_cmd] < 0 } { + return -1 + } } else { mi_send_resuming_command "exec-continue" "$test" } @@ -893,7 +905,7 @@ proc mi_runto_helper {func run_or_continue} { } proc mi_runto {func} { - mi_runto_helper $func "run" + return [mi_runto_helper $func "run"] } # Next to the next statement -- 2.30.2