[OpenOCD-devel] [PATCH]: a6a6c96 Fix Semihosting FileIO for targets using vcont packet

[<ge...@op...>]

This is an automated email from Gerrit.

Muhammad Omair Javaid (oma...@li...) just uploaded a new patch set to Gerrit, which you can find at http://openocd.zylin.com/4539

-- gerrit

commit a6a6c968e3f90e07db0d5f00518416cae60d7f5f
Author: Omair Javaid <oma...@li...>
Date:   Thu May 31 05:36:53 2018 +0500

    Fix Semihosting FileIO for targets using vcont packet
    
    This patch fixes a bug where target fails to resume after completing GDB FileIO.
    We need to update target last run control information to decide resumption. This
    was not being done for vcont packets.
    
    Change-Id: I44bea31720f8b877dba97d77a202303d546ea5bd
    Signed-off-by: Omair Javaid <oma...@li...>

diff --git a/src/server/gdb_server.c b/src/server/gdb_server.c
index d329bfe..4f721aa 100644
--- a/src/server/gdb_server.c
+++ b/src/server/gdb_server.c
@@ -2706,6 +2706,7 @@ static bool gdb_handle_vcont_packet(struct connection *connection, const char *p
 
 	/* simple case, a continue packet */
 	if (parse[0] == 'c') {
+		gdb_running_type = 'c';
 		LOG_DEBUG("target %s continue", target_name(target));
 		log_add_callback(gdb_log_callback, connection);
 		retval = target_resume(target, 1, 0, 0, 0);
@@ -2732,6 +2733,7 @@ static bool gdb_handle_vcont_packet(struct connection *connection, const char *p
 
 	/* single-step or step-over-breakpoint */
 	if (parse[0] == 's') {
+		gdb_running_type = 's';
 		bool fake_step = false;
 
 		if (strncmp(parse, "s:", 2) == 0) {

--