This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Don't need fast tracepoint in tstatus.exp
On 03/05/2013 12:46 AM, Pedro Alves wrote:
>> tstatus.exp is used to test the output of 'tstatus' but there are some
>> >tests on fast tracepoint in it, which is not related to tstatus. This
>> >patch is to remove these fast tracepoint related stuff from tstatus.exp.
> Can we assume
>
The reason why fast tracepoint stuff was added into tstatus.exp is
unclear to me. It was not mentioned in the mail posting this patch.
However, I can't see how fast tracepoint affects the output of
'tstatus', so I propose this patch to remove them.
> If we aren't doing any fast tracepoint tests, then is
> there a reason to keep this prepare_for_testing and
> runto_main instead of removing them altogether?
OK, the prepare_for_testing and runto_main are removed.
--
Yao (éå)
gdb/testsuite:
2013-03-05 Yao Qi <yao@codesourcery.com>
* gdb.trace/tstatus.exp: Remove the invocation to
gdb_load_shlibs, gdb_compile, clean_restart and runto_main.
(test_tracepoints): Don't set fast tracepoint.
(top level): Don't check agent library is loaded or not.
---
gdb/testsuite/gdb.trace/tstatus.exp | 44 -----------------------------------
1 files changed, 0 insertions(+), 44 deletions(-)
diff --git a/gdb/testsuite/gdb.trace/tstatus.exp b/gdb/testsuite/gdb.trace/tstatus.exp
index 8a0bbdf..3be3b86 100644
--- a/gdb/testsuite/gdb.trace/tstatus.exp
+++ b/gdb/testsuite/gdb.trace/tstatus.exp
@@ -34,25 +34,6 @@ if ![gdb_target_supports_trace] {
return -1
}
-set libipa [get_in_proc_agent]
-gdb_load_shlibs $libipa
-
-# Can't use prepare_for_testing, because that splits compiling into
-# building objects and then linking, and we'd fail with "linker input
-# file unused because linking not done" when building the object.
-
-if { [gdb_compile "$srcdir/$subdir/$srcfile" $binfile \
- executable [list debug shlib=$libipa] ] != "" } {
- untested "failed to compile ftrace tests"
- return -1
-}
-clean_restart ${executable}
-
-if ![runto_main] {
- fail "Can't run to main for ftrace tests"
- return 0
-}
-
proc run_trace_experiment {} {
global gdb_prompt
global decimal
@@ -141,34 +122,9 @@ proc test_tracepoints {} {
gdb_trace_setactions "collect at set_point: define actions" \
"" \
"collect parm" "^$"
- set fastgood 0
-
- gdb_test_multiple "ftrace gdb_recursion_test" "set fast tracepoint" {
- -re "May not have a fast tracepoint at .*\r\n$gdb_prompt $" {
- pass "4-byte fast tracepoint could not be set"
- }
- -re "Fast tracepoint .*\r\n$gdb_prompt $" {
- pass "4-byte fast tracepoint is set"
- set fastgood 1
- }
- }
-
- if { $fastgood } {
-
- gdb_trace_setactions "collect at four_byter: define actions" \
- "" \
- "collect globvar, anarg" "^$"
- }
run_trace_experiment
}
-gdb_reinitialize_dir $srcdir/$subdir
-
-if { [gdb_test "info sharedlibrary" ".*${libipa}.*" "IPA loaded"] != 0 } {
- untested "Could not find IPA lib loaded"
- return 1
-}
-
test_tracepoints
--
1.7.7.6