diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 4a60c08192..25626832a8 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,10 @@ +2012-09-28 Yao Qi + + * mi/mi-main.c (mi_cmd_data_write_memory): Call + write_memory_with_notification instead of write_memory. + (mi_cmd_data_write_memory_bytes): Call write_memory_with_notification + instead of target_write_memory. + 2012-09-28 Yao Qi * mi/mi-main.c (mi_cmd_data_write_memory_bytes): Emit error diff --git a/gdb/mi/mi-main.c b/gdb/mi/mi-main.c index f73d4f8796..9fa1eaac86 100644 --- a/gdb/mi/mi-main.c +++ b/gdb/mi/mi-main.c @@ -1648,7 +1648,7 @@ mi_cmd_data_write_memory (char *command, char **argv, int argc) old_chain = make_cleanup (xfree, buffer); store_signed_integer (buffer, word_size, byte_order, value); /* Write it down to memory. */ - write_memory (addr, buffer, word_size); + write_memory_with_notification (addr, buffer, word_size); /* Free the buffer. */ do_cleanups (old_chain); } @@ -1688,9 +1688,7 @@ mi_cmd_data_write_memory_bytes (char *command, char **argv, int argc) data[i] = (gdb_byte) x; } - r = target_write_memory (addr, data, len); - if (r != 0) - error (_("Could not write memory")); + write_memory_with_notification (addr, data, len); do_cleanups (back_to); } diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index a68f69d2b5..d196a35827 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2012-09-28 Yao Qi + + * gdb.mi/pr11022.exp: New. + * gdb.mi/pr11022.c: New. Copied from gdb.base/. + 2012-09-27 Tom Tromey * gdb.cp/derivation.exp: Add regression test. diff --git a/gdb/testsuite/gdb.mi/pr11022.c b/gdb/testsuite/gdb.mi/pr11022.c new file mode 100644 index 0000000000..ef6746d73c --- /dev/null +++ b/gdb/testsuite/gdb.mi/pr11022.c @@ -0,0 +1,32 @@ +/* This test is part of GDB, the GNU debugger. + + Copyright 2009-2012 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 . + */ + +int x; + +int +main () +{ + int i, j; + + for (i = 0; i < 500; ++i) { + j = 0; /* break here */ + x = 42; + j = i; /* expect HW watchpoint stop */ + } + return 0; +} diff --git a/gdb/testsuite/gdb.mi/pr11022.exp b/gdb/testsuite/gdb.mi/pr11022.exp new file mode 100644 index 0000000000..157a9e3020 --- /dev/null +++ b/gdb/testsuite/gdb.mi/pr11022.exp @@ -0,0 +1,67 @@ +# Copyright 2009-2012 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 . + +if {[skip_hw_watchpoint_tests]} { + return 0 +} + +load_lib mi-support.exp +set MIFLAGS "-i=mi" + +standard_testfile .c + +if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } { + untested pr11022.exp + return +} + +proc test_memory_changed_observer { mi_command } { with_test_prefix "${mi_command}" { + global srcfile + + if [mi_gdb_start] { + return + } + mi_run_to_main + + set line_number [gdb_get_line_number "break here"] + mi_gdb_test "-break-insert ${srcfile}:${line_number}" \ + {\^done,bkpt=.number="2",type="breakpoint".*\}} \ + "set breakpoint" + + mi_gdb_test "watch x" ".*" "set watchpoint" + + mi_execute_to "exec-continue" "breakpoint-hit" "main" "" ".*" ".*" \ + { "" "disp=\"keep\"" } "breakpoint hit" + + mi_execute_to "exec-continue" "watchpoint-trigger" "main" "" ".*" ".*" \ + { "" "wpt=\{number=\"3\",exp=\"x\"\},value=\{old=\"0\",new=\"42\"\}" } \ + "watchpoint hit" + + mi_execute_to "exec-continue" "breakpoint-hit" "main" "" ".*" ".*" \ + { "" "disp=\"keep\"" } "breakpoint hit 2" + + mi_gdb_test "-${mi_command}" {\^done} "set x" + + mi_execute_to "exec-continue" "watchpoint-trigger" "main" "" ".*" ".*" \ + { "" "wpt=\{number=\"3\",exp=\"x\"\},value=\{old=\"1\",new=\"42\"\}" } \ + "watchpoint hit 2" + + mi_gdb_exit +}} + +test_memory_changed_observer "data-write-memory-bytes &x \"01\"" +# We want to trigger a target memory write here, so the word size (4 or 8) +# is not that important. +test_memory_changed_observer "data-write-memory &x x 4 \"01\""