* gdb.mi/mi-nonstop.exp: Don't check isnative. Use
authorPedro Alves <palves@redhat.com>
Thu, 2 Apr 2009 15:43:10 +0000 (15:43 +0000)
committerPedro Alves <palves@redhat.com>
Thu, 2 Apr 2009 15:43:10 +0000 (15:43 +0000)
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
gdb/testsuite/gdb.mi/mi-nonstop.exp
gdb/testsuite/gdb.mi/mi-nsintrall.exp
gdb/testsuite/gdb.mi/mi-nsmoribund.exp
gdb/testsuite/lib/mi-support.exp

index 1e131c9f3d48245de45982d47d5ac5d2888edd55..de2205aee823ec7f94d20910f65115cbe9e9d558 100644 (file)
@@ -1,3 +1,19 @@
+2009-04-02  Pedro Alves  <pedro@codesourcery.com>
+
+       * 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  <pedro@codesourcery.com>
 
        * gdb.threads/pending-step.c, gdb.threads/pending-step.exp: New.
index 08952b0a610f0404b73f57edb0a2eca97a517eb4..18c08df6e04a2f777d3dca88dd0b6356120da468 100644 (file)
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-# 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
-
index d34b08d4e7849e781b59cea83c8ca2e052323065..0b904176bb4fb57f397444368bbf3ac075805813 100644 (file)
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-# 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"
index 505fc849bf178c9b02a776fd3b0c8a3a5b5a00fa..8ed53e083bebb9c56cc1bf7f9a7dd6e4d643f834 100644 (file)
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-# 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
index f62c2402c96aa17fd1f5d4ac8f9c95de2448b5b3..c31111b3ebebcd11d4319a489e7f34ede3740be9 100644 (file)
@@ -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