From: Anton Kolesov Date: Tue, 27 Aug 2013 09:38:50 +0000 (+0400) Subject: gdb_server: Do not set gdb_con->sync to true for new connections X-Git-Tag: v0.8.0-rc1~288 X-Git-Url: https://review.openocd.org/gitweb?p=openocd.git;a=commitdiff_plain;h=b8a23f1c43fd87f36ce454fbc7a6ea2534dcf52d gdb_server: Do not set gdb_con->sync to true for new connections In GDB connected to OpenOCD there is a command "monitor gdb_sync" which makes next stepi command to be ignored while GDB still will get an updated target state. This command sets gdb_connection->sync field to true to notify that stepi should be ignored. This field is set to true for all new connection and is set to false after first "continue" command. However if first resume command is stepi/nexti then it will be ignored and result will confuse GDB client, it will report that target received signal SIGINT. This patch sets this field to false for new connections, thus stepi/nexti will work properly when it is a first resume command. Change-Id: I7c9ebd69c3dc35f3e316041aa99f4e9d3425c0b6 Signed-off-by: Anton Kolesov Reviewed-on: http://openocd.zylin.com/1587 Tested-by: jenkins Reviewed-by: Andreas Fritiofson --- diff --git a/src/server/gdb_server.c b/src/server/gdb_server.c index ab7d40d796..b9c8e5e83e 100644 --- a/src/server/gdb_server.c +++ b/src/server/gdb_server.c @@ -906,7 +906,7 @@ static int gdb_new_connection(struct connection *connection) gdb_connection->closed = 0; gdb_connection->busy = 0; gdb_connection->noack_mode = 0; - gdb_connection->sync = true; + gdb_connection->sync = false; gdb_connection->mem_write_error = false; gdb_connection->attached = true;