mirrors/binutils-gdb
0
0
Fork 0
mirror of https://sourceware.org/git/binutils-gdb.git synced 2024-11-28 12:33:36 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Handle memory write errors on gdb.base/break-always.exp

Browse Source 
This is another case of the testcase not handling memory write errors that
happen on some targets (QEMU) when GDB attempts to modify an address that
should contain a breakpoint, for example.

The following patch handles this and prevents spurious failures from
happening. It also adds a foreach loop to avoid duplication of code
and hardcoded patterns.

gdb/testsuite/ChangeLog:

2015-04-29  Luis Machado  <lgustavo@codesourcery.com>

	* gdb.base/break-always.exp: Abort testing if writing to memory
	causes an error.
This commit is contained in:
Luis Machado 2015-04-29 12:09:40 -03:00
parent 0a13382c8e
commit ecbf2b3c4f
2 changed files with 23 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
gdb/testsuite/ChangeLog
View File

@ -1,3 +1,8 @@
2015-04-29 Luis Machado <lgustavo@codesourcery.com>
* gdb.base/break-always.exp: Abort testing if writing to memory
causes an error.
2015-04-28 Doug Evans <dje@google.com>
* gdb.python/py-pp-maint.py: Move "replace" testing to ...

30
gdb/testsuite/gdb.base/break-always.exp
View File

@ -69,19 +69,25 @@ gdb_test "p /x \$shadow = *(char *) $bp_address" \
# and still leave the breakpoint insn planted. Try twice with
# different values, in case we happen to be writting exactly what was
# there already.
gdb_test "p /x *(char *) $bp_address = 0" \
" = 0x0" \
"write 0 to breakpoint's address"
gdb_test "p /x *(char *) $bp_address" \
" = 0x0" \
"read back 0 from the breakpoint's address"
foreach test_value {0 1} {
set write_test "write $test_value to breakpoint's address"
gdb_test "p /x *(char *) $bp_address = 1" \
" = 0x1" \
"write 1 to breakpoint's address"
gdb_test "p /x *(char *) $bp_address" \
" = 0x1" \
"read back 1 from the breakpoint's address"
gdb_test_multiple "p /x *(char *) $bp_address = $test_value" $write_test {
-re "Cannot access memory at address $hex.*$gdb_prompt $" {
# Some targets do not allow manually writing a breakpoint to a
# certain memory address, like QEMU. In that case, just bail out.
unsupported "Cannot write to address $bp_address"
return -1
}
-re " = .*$gdb_prompt $" {
pass $write_test
}
}
set read_test "read back $test_value from the breakpoint's address"
gdb_test "p /x *(char *) $bp_address" " = 0x$test_value" $read_test
}
# Restore the original contents.
gdb_test "p /x *(char *) $bp_address = \$shadow" "" \