binutils-gdb/gdb/testsuite/gdb.trace/tspeed.exp
Andrew Burgess c708f4d256 gdb: Don't call gdb_load_shlib unless GDB is running
The gdb_load_shlib function will, on remote targets, try to run some
GDB commands.  This obviously isn't going to work unless GDB is
running.

The gdb.trace/tspeed.exp test calls gdb_load_shlib before starting
GDB.  Don't do that.

The failure that's triggered is actually DeJaGNU complaining that the
variable $use_gdb_stub doesn't exist, this is only created when GDB is
started.  Something like this should trigger a failure:

  make check-gdb \
    RUNTESTFLAGS="--target_board=remote-gdbserver-on-localhost \
                  gdb.trace/tspeed.exp"

This commit also adds a check to gdb_load_shlib that GDB is running.
The check is always performed, so this should catch cases where a GDB
developer adds a use of gdb_load_shlib but doesn't test their code
with a remote target.

gdb/testsuite/ChangeLog:

	* gdb.trace/tspeed.exp: Only call gdb_load_shlib after gdb has
	started.
	* lib/gdb.exp (gdb_load_shlib): Call perror if GDB is not running.
2018-07-28 20:21:50 +01:00

140 lines
3.3 KiB
Plaintext

# Copyright 2011-2018 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 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, see <http://www.gnu.org/licenses/>.
load_lib "trace-support.exp"
standard_testfile
set executable $testfile
set ipalib [get_in_proc_agent]
if { [gdb_compile "$srcdir/$subdir/$srcfile" $binfile \
executable [concat {debug nowarnings c} shlib=$ipalib]] != "" } {
untested "failed to compile"
return -1
}
# Typically we need a little extra time for this test.
set timeout 180
set ws "\[\r\n\t \]+"
set cr "\[\r\n\]+"
#
# Utility procs
#
proc prepare_for_trace_test {} {
global executable
clean_restart $executable
gdb_load_shlib $ipalib
runto_main
set testline [gdb_get_line_number "set pre-run breakpoint here"]
gdb_test "break $testline" ".*" ""
set testline [gdb_get_line_number "set post-run breakpoint here"]
gdb_test "break $testline" ".*" ""
}
proc run_trace_experiment {} {
gdb_test "continue" \
".*Breakpoint \[0-9\]+, main .*" \
"advance to trace begin"
gdb_test_no_output "tstart" "start trace experiment"
gdb_test "continue" \
".*Breakpoint \[0-9\]+, main .*" \
"advance through tracing"
gdb_test "tstatus" ".*Trace .*" "check on trace status"
gdb_test "tstop" "" ""
}
proc gdb_slow_trace_speed_test { } {
gdb_delete_tracepoints
gdb_test "print iters = init_iters" ".* = .*"
set testline [gdb_get_line_number "set tracepoint here"]
gdb_test "trace $testline if (globfoo != 12 && globfoo2 == 45)" \
"Tracepoint \[0-9\]+ at .*" \
"set slow tracepoint"
# Begin the test.
run_trace_experiment
}
proc gdb_fast_trace_speed_test { } {
global gdb_prompt
gdb_delete_tracepoints
gdb_test "print iters = init_iters" ".* = .*"
set run_ftrace 0
set testline [gdb_get_line_number "set tracepoint here"]
gdb_test_multiple "ftrace $testline if (globfoo != 12 && globfoo2 == 45)" \
"set conditional fast tracepoint" {
-re "Fast tracepoint \[0-9\]+ at .*\r\n$gdb_prompt $" {
pass "set conditional fast tracepoint, done"
set run_ftrace 1
}
-re "May not have a fast tracepoint at .*\r\n$gdb_prompt $" {
pass "set conditional fast tracepoint, not allowed at line"
}
}
# If the fast tracepoint couldn't be set, don't bother with the run.
if { $run_ftrace == 1 } then {
# Begin the test.
run_trace_experiment
}
}
proc gdb_trace_collection_test {} {
prepare_for_trace_test
gdb_slow_trace_speed_test
gdb_fast_trace_speed_test
}
clean_restart $executable
gdb_load_shlib $ipalib
runto_main
if { ![gdb_target_supports_trace] } then {
unsupported "current target does not support trace"
return 1
}
# Body of test encased in a proc so we can return prematurely.
gdb_trace_collection_test