diff options
author | Yao Qi <yao.qi@linaro.org> | 2015-10-21 16:16:25 +0100 |
---|---|---|
committer | Yao Qi <yao.qi@linaro.org> | 2015-10-21 16:16:25 +0100 |
commit | 80f0110c98d4134409f38e7d498e89fe96773819 (patch) | |
tree | 43db0ae3e83ac23c5058a99405082b2e6998833a /gdb/testsuite/lib | |
parent | d7161de46af80e460d432d8dbb1c42f8cbacf6dc (diff) | |
download | gdb-80f0110c98d4134409f38e7d498e89fe96773819.zip gdb-80f0110c98d4134409f38e7d498e89fe96773819.tar.gz gdb-80f0110c98d4134409f38e7d498e89fe96773819.tar.bz2 |
Remove checking vCont;s in exec_cmd_expect_vCont_count
Nowadays, in the range-stepping tests, we check not only the number of
vCont;r packets but also the number of vCont;s packets, because we think
the remote target which can do range stepping must support single step.
However, if we turn displaced stepping on, the remote target (GDBserver)
can do range stepping, and support single step, but GDB may decide to
resume instructions in the scratchpad rather than single step them one
by one for displaced stepping. For example, when aarch64 GDB debugs
arm linux program with aarch64 GDBserver, GDBserver supports both range
stepping and single step, but GDB (with the gdbarch for arm-linux)
decides resume instructions in the scratchpad, so in the RSP traffic,
there is no vCont;s packet at all, and some range-stepping.exp tests
fail,
FAIL: gdb.base/range-stepping.exp: multi insns: next: vCont;s=1 vCont;r=1
This patch is to get rid of the checking to the number of vCont;s in
exec_cmd_expect_vCont_count.
gdb/testsuite:
2015-10-21 Yao Qi <yao.qi@linaro.org>
* lib/range-stepping-support.exp (exec_cmd_expect_vCont_count):
Remove argument exp_vCont_s.
* gdb.base/range-stepping.exp: Callers updated.
* gdb.trace/range-stepping.exp: Likewise.
Diffstat (limited to 'gdb/testsuite/lib')
-rw-r--r-- | gdb/testsuite/lib/range-stepping-support.exp | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/gdb/testsuite/lib/range-stepping-support.exp b/gdb/testsuite/lib/range-stepping-support.exp index d6c0e85..1b27309 100644 --- a/gdb/testsuite/lib/range-stepping-support.exp +++ b/gdb/testsuite/lib/range-stepping-support.exp @@ -14,15 +14,14 @@ # along with this program. If not, see <http://www.gnu.org/licenses/>. # Execute command CMD and check that GDB sends the expected number of -# vCont;s and vCont;r packets. Returns 0 if the test passes, -# otherwise returns 1. +# vCont;r packet. Returns 0 if the test passes, otherwise returns 1. -proc exec_cmd_expect_vCont_count { cmd exp_vCont_s exp_vCont_r } { +proc exec_cmd_expect_vCont_count { cmd exp_vCont_r } { global gdb_prompt gdb_test_no_output "set debug remote 1" "" - set test "${cmd}: vCont;s=${exp_vCont_s} vCont;r=${exp_vCont_r}" + set test "${cmd}: vCont;r=${exp_vCont_r}" set r_counter 0 set s_counter 0 set ret 1 @@ -40,7 +39,7 @@ proc exec_cmd_expect_vCont_count { cmd exp_vCont_s exp_vCont_r } { exp_continue } -re "$gdb_prompt $" { - if { $r_counter == ${exp_vCont_r} && $s_counter == ${exp_vCont_s} } { + if { $r_counter == ${exp_vCont_r} } { pass $test set ret 0 } else { |