Update copyright year range in all GDB files.
[external/binutils.git] / gdb / testsuite / gdb.base / attach.exp
index cf38d92..2343354 100644 (file)
@@ -1,49 +1,25 @@
-# Copyright 1997, 1999, 2002, 2003, 2004 Free Software Foundation, Inc.
+# Copyright 1997-2019 Free Software Foundation, Inc.
 
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
-# the Free Software Foundation; either version 2 of the License, or
+# the Free Software Foundation; either version 3 of the License, or
 # (at your option) any later version.
-# 
+#
 # This program is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 # GNU General Public License for more details.
-# 
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
-
-if $tracelevel then {
-    strace $tracelevel
-}
-
-set prms_id 0
-set bug_id 0
-
-# On HP-UX 11.0, this test is causing a process running the program
-# "attach" to be left around spinning.  Until we figure out why, I am
-# commenting out the test to avoid polluting tiamat (our 11.0 nightly
-# test machine) with these processes. RT
 #
-# Setting the magic bit in the target app should work.  I added a
-# "kill", and also a test for the R3 register warning.  JB
-if { [istarget "hppa*-*-hpux*"] } {
-    return 0
-}
+# You should have received a copy of the GNU General Public License
+# along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
 
-# are we on a target board
-if [is_remote target] then {
+if {![can_spawn_for_attach]} {
     return 0
 }
 
-set testfile "attach"
-set srcfile  ${testfile}.c
-set srcfile2 ${testfile}2.c
-set binfile  ${objdir}/${subdir}/${testfile}
-set binfile2 ${objdir}/${subdir}/${testfile}2
-set escapedbinfile  [string_to_regexp ${objdir}/${subdir}/${testfile}]
-set cleanupfile ${objdir}/${subdir}/${testfile}.awk
+standard_testfile attach.c attach2.c
+set binfile2 ${binfile}2
+set escapedbinfile  [string_to_regexp $binfile]
 
 #execute_anywhere "rm -f ${binfile} ${binfile2}"
 remote_exec build "rm -f ${binfile} ${binfile2}"
@@ -51,23 +27,21 @@ remote_exec build "rm -f ${binfile} ${binfile2}"
 #
 #log_user 1
 
-# Clean out any old files from past runs.
-#
-remote_exec build "${cleanupfile}"
-
 # build the first test case
 #
 if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } {
-    gdb_suppress_entire_file "Testcase compile failed, so all tests in this file will automatically fail."
+    untested "failed to compile"
+    return -1
 }
 
 # Build the in-system-call test
 
 if  { [gdb_compile "${srcdir}/${subdir}/${srcfile2}" "${binfile2}" executable {debug}] != "" } {
-    gdb_suppress_entire_file "Testcase compile failed, so all tests in this file will automatically fail."
+    untested "failed to compile in-system-call test"
+    return -1
 }
 
-if [get_compiler_info ${binfile}] {
+if [get_compiler_info] {
     return -1
 }
 
@@ -77,44 +51,64 @@ proc do_attach_tests {} {
     global escapedbinfile
     global srcfile
     global testfile
-    global objdir
     global subdir
     global timeout
+    global decimal
     
+    # Figure out a regular expression that will match the sysroot,
+    # noting that the default sysroot is "target:", and also noting
+    # that GDB will strip "target:" from the start of filenames when
+    # operating on the local filesystem.  However the default sysroot
+    # can be set via configure option --with-sysroot, which can be "/".
+    # If $binfile is a absolute path, so pattern
+    # "$sysroot$escapedbinfile" below is wrong.  Use [^\r\n]* to make
+    # $sysroot simple.
+    set sysroot "\[^\r\n\]*"
+
     # Start the program running and then wait for a bit, to be sure
     # that it can be attached to.
 
-    set testpid [eval exec $binfile &]
-    exec sleep 2
-    if { [istarget "*-*-cygwin*"] } {
-       # testpid is the Cygwin PID, GDB uses the Windows PID, which might be
-       # different due to the way fork/exec works.
-       set testpid [ exec ps -e | gawk "{ if (\$1 == $testpid) print \$4; }" ]
-    }
+    set test_spawn_id [spawn_wait_for_attach $binfile]
+    set testpid [spawn_id_get_pid $test_spawn_id]
 
     # Verify that we cannot attach to nonsense.
 
-    send_gdb "attach abc\n"
-    gdb_expect {
-       -re ".*Illegal process-id: abc.*$gdb_prompt $" {
-           pass "attach to nonsense is prohibited"
+    set test "attach to nonsense is prohibited"
+    gdb_test_multiple "attach abc" "$test" {
+       -re "Illegal process-id: abc\\.\r\n$gdb_prompt $" {
+           pass "$test"
        }
        -re "Attaching to.*, process .*couldn't open /proc file.*$gdb_prompt $" {
            # Response expected from /proc-based systems.
-           pass "attach to nonsense is prohibited
+           pass "$test
        }
-       -re ".*Can't attach to process..*$gdb_prompt $" {
+       -re "Can't attach to process..*$gdb_prompt $" {
            # Response expected on Cygwin
-           pass "attach to nonsense is prohibited"
+           pass "$test"
        }
        -re "Attaching to.*$gdb_prompt $" {
-           fail "attach to nonsense is prohibited (bogus pid allowed)"
+           fail "$test (bogus pid allowed)"
+       }
+    }
+
+    # Verify that we cannot attach to nonsense even if its initial part is
+    # a valid PID.
+
+    set test "attach to digits-starting nonsense is prohibited"
+    gdb_test_multiple "attach ${testpid}x" "$test" {
+       -re "Illegal process-id: ${testpid}x\\.\r\n$gdb_prompt $" {
+           pass "$test"
        }
-       -re "$gdb_prompt $" {
-           fail "attach to nonsense is prohibited"
+       -re "Attaching to.*, process .*couldn't open /proc file.*$gdb_prompt $" {
+           # Response expected from /proc-based systems.
+           pass "$test" 
        }
-       timeout {
-           fail "(timeout) attach to nonsense is prohibited"
+       -re "Can't attach to process..*$gdb_prompt $" {
+           # Response expected on Cygwin
+           pass "$test"
+       }
+       -re "Attaching to.*$gdb_prompt $" {
+           fail "$test (bogus pid allowed)"
        }
     }
 
@@ -130,35 +124,33 @@ proc do_attach_tests {} {
        # FreeBSD, and probably other *BSD's too).
        set boguspid -1
     }
-    send_gdb "attach $boguspid\n"
-    gdb_expect {
+    set test "attach to nonexistent process is prohibited"
+    gdb_test_multiple "attach $boguspid" "$test" {
        -re "Attaching to.*, process $boguspid.*No such process.*$gdb_prompt $" {
            # Response expected on ptrace-based systems (i.e. HP-UX 10.20).
-           pass "attach to nonexistent process is prohibited"
+           pass "$test"
        }
        -re "Attaching to.*, process $boguspid failed.*Hint.*$gdb_prompt $" {
            # Response expected on ttrace-based systems (i.e. HP-UX 11.0).
-           pass "attach to nonexistent process is prohibited"
+           pass "$test"
        }
        -re "Attaching to.*, process $boguspid.*denied.*$gdb_prompt $" {
-           pass "attach to nonexistent process is prohibited"
+           pass "$test"
        }
        -re "Attaching to.*, process $boguspid.*not permitted.*$gdb_prompt $" {
-           pass "attach to nonexistent process is prohibited"
+           pass "$test"
        }
        -re "Attaching to.*, process .*couldn't open /proc file.*$gdb_prompt $" {
            # Response expected from /proc-based systems.
-           pass "attach to nonexistent process is prohibited"
+           pass "$test"
        }
-       -re ".*Can't attach to process..*$gdb_prompt $" {
+       -re "Can't attach to process..*$gdb_prompt $" {
            # Response expected on Cygwin
-           pass "attach to nonexistent process is prohibited"
+           pass "$test"
        }
-       -re "$gdb_prompt $" {
-           fail "attach to nonexistent process is prohibited"
-       }
-       timeout {
-           fail "(timeout) attach to nonexistent process is prohibited"
+       -re "Attaching to.*, process $boguspid.*failed.*$gdb_prompt $" {
+           # Response expected on the extended-remote target.
+           pass "$test"
        }
     }
     
@@ -170,80 +162,38 @@ proc do_attach_tests {} {
     # us.  So, we must also be prepared to be asked if we want to
     # discard an existing set of symbols.)
     
-    send_gdb "file $binfile\n"
-    gdb_expect {
-       -re "Load new symbol table from.*y or n.*$" {
-           send_gdb "y\n"
-           gdb_expect {
-               -re "Reading symbols from $escapedbinfile\.\.\.*done.*$gdb_prompt $" {
-                   pass "(re)set file, before attach1"
-               }
-               -re "$gdb_prompt $" {
-                   fail "(re)set file, before attach1"
-               }
-               timeout {
-                   fail "(timeout) (re)set file, before attach1"
-               }
-           }
+    set test "set file, before attach1"
+    gdb_test_multiple "file $binfile" "$test" {
+       -re "Load new symbol table from.*y or n. $" {
+           gdb_test "y" "Reading symbols from $escapedbinfile\.\.\.*" \
+               "$test (re-read)"
        }
-       -re "Reading symbols from $escapedbinfile\.\.\.*done.*$gdb_prompt $" {
-           pass "set file, before attach1"
-       }
-       -re "$gdb_prompt $" {
-           fail "set file, before attach1"
-       }
-       timeout {
-           fail "(timeout) set file, before attach1"
+       -re "Reading symbols from $escapedbinfile\.\.\.*$gdb_prompt $" {
+           pass "$test"
        }
     }
 
-    send_gdb "attach $testpid\n"
-    gdb_expect {
+    set test "attach1, after setting file"
+    gdb_test_multiple "attach $testpid" "$test" {
        -re "Attaching to program.*`?$escapedbinfile'?, process $testpid.*main.*at .*$srcfile:.*$gdb_prompt $" {
-           pass "attach1, after setting file"
+           pass "$test"
        }
        -re "Attaching to program.*`?$escapedbinfile\.exe'?, process $testpid.*\[Switching to thread $testpid\..*\].*$gdb_prompt $" {
            # Response expected on Cygwin
-           pass "attach1, after setting file"
-       }
-       -re "$gdb_prompt $" {
-           fail "attach1, after setting file"
-       }
-       timeout {
-           fail "(timeout) attach1, after setting file"
+           pass "$test"
        }
     }
 
     # Verify that we can "see" the variable "should_exit" in the
     # program, and that it is zero.
    
-    send_gdb "print should_exit\n"
-    gdb_expect {
-       -re ".* = 0.*$gdb_prompt $" {
-           pass "after attach1, print should_exit"
-       }
-       -re "$gdb_prompt $" {
-           fail "after attach1, print should_exit"
-       }
-       timeout {
-           fail "(timeout) after attach1, print should_exit"
-       }
-    }
+    gdb_test "print should_exit" " = 0" "after attach1, print should_exit"
 
     # Detach the process.
    
-    send_gdb "detach\n"
-    gdb_expect {
-       -re "Detaching from program: .*$escapedbinfile.*$gdb_prompt $" {
-           pass "attach1 detach"
-       }
-       -re "$gdb_prompt $" {
-           fail "attach1 detach"
-       }
-       timeout {
-           fail "(timeout) attach1 detach"
-       }
-    }
+    gdb_test "detach" \
+       "Detaching from program: .*$escapedbinfile, process $testpid\r\n\\\[Inferior $decimal \\(.*\\) detached\\\]" \
+       "attach1 detach"
 
     # Wait a bit for gdb to finish detaching
     
@@ -255,30 +205,10 @@ proc do_attach_tests {} {
     
     set old_timeout $timeout
     set timeout 15 
-    send_gdb "file\n"
-    gdb_expect {
-       -re ".*gdb internal error.*$" { 
-           fail "Internal error, prob. Memory corruption" 
-       }
-       -re "No executable file now.*Discard symbol table.*y or n.*$" {
-           send_gdb "y\n"
-           gdb_expect {
-               -re "No symbol file now.*$gdb_prompt $" {
-                   pass "attach1, purging symbols after detach"
-               }
-               -re "$gdb_prompt $" {
-                   fail "attach1, purging symbols after detach"
-               }
-               timeout {
-                   fail "(timeout) attach1, purging symbols after detach"
-               }
-           }
-       }
-       -re "$gdb_prompt $" {
-           fail "attach1, purging file after detach"
-       }
-       timeout {
-           fail "(timeout) attach1, purging file after detach"
+    set test "attach1, purging symbols after detach"
+    gdb_test_multiple "file" "$test" {
+       -re "No executable file now.*Discard symbol table.*y or n. $" {
+           gdb_test "y" "No symbol file now." "$test"
        }
     }
     set timeout $old_timeout
@@ -286,277 +216,224 @@ proc do_attach_tests {} {
     # Verify that we can attach to the process just by giving the
     # process ID.
    
-    send_gdb "attach $testpid\n"
-    gdb_expect {
-       -re "Attaching to process $testpid.*Load new symbol table from \"$escapedbinfile\.exe\".*y or n.*$" {
+    set test "attach2, with no file"
+    set found_exec_file 0
+    gdb_test_multiple "attach $testpid" "$test" {
+       -re "Attaching to process $testpid.*Load new symbol table from \"$sysroot$escapedbinfile\.exe\".*y or n. $" {
            # On Cygwin, the DLL's symbol tables are loaded prior to the
            # executable's symbol table.  This in turn always results in
            # asking the user for actually loading the symbol table of the
            # executable.
-           send_gdb "y\n"
-           gdb_expect {
-               -re "Reading symbols from $escapedbinfile\.\.\.*done.*$gdb_prompt $" {
-                   pass "(re)set file, before attach2"
-               }
-               -re "$gdb_prompt $" {
-                   fail "(re)set file, before attach2"
-               }
-               timeout {
-                   fail "(timeout) (re)set file, before attach2"
-               }
-           }
-       }
-       -re "Attaching to process $testpid.*Reading symbols from $escapedbinfile.*main.*at .*$gdb_prompt $" {
-           pass "attach2"
+           gdb_test "y" "Reading symbols from $sysroot$escapedbinfile\.\.\.*" \
+               "$test (reset file)"
+
+           set found_exec_file 1
        }
-       -re "$gdb_prompt $" {
-           fail "attach2"
+       -re "Attaching to process $testpid.*Reading symbols from $sysroot$escapedbinfile.*main.*at .*$gdb_prompt $" {
+           pass "$test"
+           set found_exec_file 1
        }
-       timeout {
-           fail "(timeout) attach2"
+    }
+
+    if {$found_exec_file == 0} {
+       set test "load file manually, after attach2"
+       gdb_test_multiple "file $binfile" "$test" {
+           -re "A program is being debugged already..*Are you sure you want to change the file.*y or n. $" {
+               gdb_test "y" "Reading symbols from $escapedbinfile\.\.\.*" \
+                   "$test (re-read)"
+           }
+           -re "Reading symbols from $escapedbinfile\.\.\.*$gdb_prompt $" {
+               pass "$test"
+           }
        }
     }
 
     # Verify that we can modify the variable "should_exit" in the
     # program.
 
-    send_gdb "set should_exit=1\n"
-    gdb_expect {
-       -re "$gdb_prompt $" {
-           pass "after attach2, set should_exit"
-       }
-       timeout {
-           fail "(timeout) after attach2, set should_exit"
-       }
-    }
+    gdb_test_no_output "set should_exit=1" "after attach2, set should_exit"
 
     # Verify that the modification really happened.
 
-    send_gdb "tbreak 19\n"
-    gdb_expect {
-       -re "Breakpoint .*at.*$srcfile, line 19.*$gdb_prompt $" {
-           pass "after attach2, set tbreak postloop"
-       }
-       -re "$gdb_prompt $" {
-           fail "after attach2, set tbreak postloop"
-       }
-       timeout {
-           fail "(timeout) after attach2, set tbreak postloop"
-       }
-    }
-    send_gdb "continue\n"
-    gdb_expect {
-       -re "main.*at.*$srcfile:19.*$gdb_prompt $" {
-           pass "after attach2, reach tbreak postloop"
-       }
-       -re "$gdb_prompt $" {
-           fail "after attach2, reach tbreak postloop"
-       }
-       timeout {
-           fail "(timeout) after attach2, reach tbreak postloop"
-       }
-    }
+    gdb_breakpoint [gdb_get_line_number "postloop"] temporary
+    gdb_continue_to_breakpoint "postloop" ".* postloop .*"
 
     # Allow the test process to exit, to cleanup after ourselves.
 
-    send_gdb "continue\n"
-    gdb_expect {
-       -re "Program exited normally.*$gdb_prompt $" {
-           pass "after attach2, exit"
-       }
-       -re "$gdb_prompt $" {
-           fail "after attach2, exit"
-       }
-       timeout {
-           fail "(timeout) after attach2, exit"
-       }
-    }
+    gdb_continue_to_end "after attach2, exit"
 
     # Make sure we don't leave a process around to confuse
     # the next test run (and prevent the compile by keeping
     # the text file busy), in case the "set should_exit" didn't
     # work.
-   
-    remote_exec build "kill -9 ${testpid}"
-    # Start the program running and then wait for a bit, to be sure
-    # that it can be attached to.
-   
-    set testpid [eval exec $binfile &]
-    exec sleep 2
-    if { [istarget "*-*-cygwin*"] } {
-       # testpid is the Cygwin PID, GDB uses the Windows PID, which might be
-       # different due to the way fork/exec works.
-       set testpid [ exec ps -e | gawk "{ if (\$1 == $testpid) print \$4; }" ]
-    }
+
+    kill_wait_spawned_process $test_spawn_id
+
+    set test_spawn_id [spawn_wait_for_attach $binfile]
+    set testpid [spawn_id_get_pid $test_spawn_id]
 
     # Verify that we can attach to the process, and find its a.out
     # when we're cd'd to some directory that doesn't contain the
     # a.out.  (We use the source path set by the "dir" command.)
     
-    send_gdb "dir ${objdir}/${subdir}\n"
-    gdb_expect {
-       -re ".*Source directories searched: .*$gdb_prompt $" {
-           pass "set source path"
-       }
-       -re "$gdb_prompt $" {
-           fail "set source path"
-       }
-       timeout {
-           fail "(timeout) set source path"
-       }
-    }
+    gdb_test "dir [standard_output_file {}]" "Source directories searched: .*" \
+       "set source path"
 
-    send_gdb "cd /tmp\n"
-    gdb_expect {
-       -re ".*Working directory /tmp.*$gdb_prompt $" {
-           pass "cd away from process' a.out"
-       }
-       -re "$gdb_prompt $" {
-           fail "cd away from process' a.out"
-       }
-       timeout {
-           fail "(timeout) cd away from process' a.out"
-       }
-    }
+    gdb_test "cd /tmp" "Working directory /tmp." \
+       "cd away from process working directory"
 
     # Explicitly flush out any knowledge of the previous attachment.
-    send_gdb "symbol\n"
-    gdb_expect {
-       -re ".*Discard symbol table from.*y or n. $" {
-           send_gdb "y\n"
-           gdb_expect {
-               -re ".*No symbol file now.*$gdb_prompt $" {
-                   pass "before attach3, flush symbols"
-               }
-               -re "$gdb_prompt $" {
-                   fail "before attach3, flush symbols"
-               }
-               timeout {
-                   fail "(timeout) before attach3, flush symbols"
-               }
-           }
-       }
-       -re ".*No symbol file now.*$gdb_prompt $" {
-           pass "before attach3, flush symbols"
-       }
-       -re "$gdb_prompt $" {
-           fail "before attach3, flush symbols"
-       }
-       timeout {
-           fail "(timeout) before attach3, flush symbols"
-       }
-    }
-    send_gdb "exec\n"
-    gdb_expect {
-       -re ".*No executable file now.*$gdb_prompt $" {
-           pass "before attach3, flush exec"
-       }
-       -re "$gdb_prompt $" {
-           fail "before attach3, flush exec"
-       }
-       timeout {
-           fail "(timeout) before attach3, flush exec"
-       }
-    }
 
-    send_gdb "attach $testpid\n"
-    gdb_expect {
-       -re "Attaching to process $testpid.*Reading symbols from $escapedbinfile.*main.*at .*$gdb_prompt $" {
-           pass "attach when process' a.out not in cwd"
+    set test "before attach3, flush symbols"
+    gdb_test_multiple "symbol-file" "$test" {
+       -re "Discard symbol table from.*y or n. $" {
+           gdb_test "y" "No symbol file now." \
+               "$test"
        }
-       -re "$gdb_prompt $" {
-           fail "attach when process' a.out not in cwd"
-       }
-       timeout {
-           fail "(timeout) attach when process' a.out not in cwd"
+       -re "No symbol file now.*$gdb_prompt $" {
+           pass "$test"
        }
     }
 
-    send_gdb "kill\n"
-    gdb_expect {
-       -re ".*Kill the program being debugged.*y or n. $" {
-           send_gdb "y\n"
-           gdb_expect {
-               -re "$gdb_prompt $" {
-                   pass "after attach3, exit"
-               }
-               timeout {
-                   fail "(timeout) after attach3, exit"
-               }
-           }
-       }
-       -re "$gdb_prompt $" {
-           fail "after attach3, exit"
-       }
-       timeout {
-           fail "(timeout) after attach3, exit"
-       }
-    }
+    gdb_test "exec" "No executable file now." \
+       "before attach3, flush exec"
+
+    gdb_test "attach $testpid" \
+       "Attaching to process $testpid.*Reading symbols from $sysroot$escapedbinfile.*main.*at .*" \
+       "attach when process' a.out not in cwd"
+
+    set test "after attach3, exit"
+    gdb_test "kill" \
+       "" \
+       "$test" \
+       "Kill the program being debugged.*y or n. $" \
+       "y"
     
     # Another "don't leave a process around"
-    remote_exec build "kill -9 ${testpid}"
+    kill_wait_spawned_process $test_spawn_id
 }
 
 proc do_call_attach_tests {} {
     global gdb_prompt
     global binfile2
     
-    # Start the program running and then wait for a bit, to be sure
-    # that it can be attached to.
-   
-    set testpid [eval exec $binfile2 &]
-    exec sleep 2
-    if { [istarget "*-*-cygwin*"] } {
-       # testpid is the Cygwin PID, GDB uses the Windows PID, which might be
-       # different due to the way fork/exec works.
-       set testpid [ exec ps -e | gawk "{ if (\$1 == $testpid) print \$4; }" ]
-    }
+    set test_spawn_id [spawn_wait_for_attach $binfile2]
+    set testpid [spawn_id_get_pid $test_spawn_id]
 
     # Attach
    
-    gdb_test "file $binfile2" ".*" "force switch to gdb64, if necessary"
-    send_gdb "attach $testpid\n"
-    gdb_expect {
-       -re ".*warning: reading register.*I.*O error.*$gdb_prompt $" {
-           fail "attach call, read register 3 error"
+    gdb_test "file $binfile2" ".*" "load file"
+    set test "attach call"
+    gdb_test_multiple "attach $testpid" "$test" {
+       -re "warning: reading register.*I.*O error.*$gdb_prompt $" {
+           fail "$test (read register error)"
        }
        -re "Attaching to.*process $testpid.*libc.*$gdb_prompt $" {
-           pass "attach call"
+           pass "$test"
        }
        -re "Attaching to.*process $testpid.*\[Switching to thread $testpid\..*\].*$gdb_prompt $" {
-           pass "attach call"
-       }
-       -re "$gdb_prompt $" {
-           fail "attach call"
-       }
-       timeout {
-           fail "(timeout) attach call"
+           pass "$test"
        }
     }
 
     # See if other registers are problems
     
-    send_gdb "i r r3\n"
-    gdb_expect {
-       -re ".*warning: reading register.*$gdb_prompt $" {
-           pass "CHFts23490: known bug"
+    set test "info other register"
+    gdb_test_multiple "i r r3" "$test" {
+       -re "warning: reading register.*$gdb_prompt $" {
+           fail "$test"
        }
-       -re ".*r3.*$gdb_prompt $" {
-           pass "Bug fixed, Yayyy!"
+       -re "r3.*$gdb_prompt $" {
+           pass "$test"
        }
-       timeout { fail "timeout on info reg" }
     }
 
     # Get rid of the process
     
-    gdb_test "p should_exit = 1" ".*"
-    gdb_test "c" ".*Program exited normally.*"
+    gdb_test "p should_exit = 1"
+    gdb_continue_to_end
    
     # Be paranoid
    
-    remote_exec build "kill -9 ${testpid}"
+    kill_wait_spawned_process $test_spawn_id
 }
 
+proc do_command_attach_tests {} {
+    global gdb_prompt
+    global binfile
+    global verbose
+    global GDB
+    global INTERNAL_GDBFLAGS
+    global GDBFLAGS
+
+    if ![isnative] then {
+       unsupported "command attach test"
+       return 0
+    }
+
+    set test_spawn_id [spawn_wait_for_attach $binfile]
+    set testpid [spawn_id_get_pid $test_spawn_id]
+
+    gdb_exit
+
+    set res [gdb_spawn_with_cmdline_opts \
+                "-quiet -iex \"set height 0\" -iex \"set width 0\" --pid=$testpid"]
+    set test "starting with --pid"
+    gdb_test_multiple "" $test {
+       -re "Reading symbols from.*$gdb_prompt $" {
+           pass "$test"
+       }
+    }
+
+    # Get rid of the process
+    kill_wait_spawned_process $test_spawn_id
+}
+
+# Test ' gdb --pid PID -ex "run" '.  GDB used to have a bug where
+# "run" would run before the attach finished - PR17347.
+
+proc test_command_line_attach_run {} {
+    global gdb_prompt
+    global binfile
+
+    if ![isnative] then {
+       unsupported "commandline attach run test"
+       return 0
+    }
+
+    with_test_prefix "cmdline attach run" {
+       set test_spawn_id [spawn_wait_for_attach $binfile]
+       set testpid [spawn_id_get_pid $test_spawn_id]
+
+       set test "run to prompt"
+       gdb_exit
+
+       set res [gdb_spawn_with_cmdline_opts \
+                    "-quiet -iex \"set height 0\" -iex \"set width 0\" --pid=$testpid -ex \"start\""]
+       if { $res != 0} {
+           fail $test
+           kill_wait_spawned_process $test_spawn_id
+           return $res
+       }
+       gdb_test_multiple "" $test {
+           -re {Attaching to.*Start it from the beginning\? \(y or n\) } {
+               pass $test
+           }
+       }
+
+       send_gdb "y\n"
+
+       set test "run to main"
+       gdb_test_multiple "" $test {
+           -re "Temporary breakpoint .* main .*$gdb_prompt $" {
+               pass $test
+           }
+       }
+
+       # Get rid of the process
+       kill_wait_spawned_process $test_spawn_id
+    }
+}
 
 # Start with a fresh gdb
 
@@ -577,4 +454,10 @@ gdb_start
 gdb_reinitialize_dir $srcdir/$subdir
 do_call_attach_tests
 
+# Test "gdb --pid"
+
+do_command_attach_tests
+
+test_command_line_attach_run
+
 return 0