[OpenOCD-commit] Main OpenOCD repository branch, master, updated. v0.10.0-1167-g25efc1506

[OpenOCD-Gerrit <ope...@us...>]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Main OpenOCD repository".

The branch, master has been updated
       via  25efc150694042b349b8df1ff7c41f16955c5288 (commit)
      from  0a804222da63c5f849efa23b019a59e2dea76842 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 25efc150694042b349b8df1ff7c41f16955c5288
Author: Jan Matyas <ma...@co...>
Date:   Wed Apr 1 11:58:20 2020 +0200

    target: added events TARGET_EVENT_STEP_START and _END
    
    Events TARGET_EVENT_STEP_START and TARGET_EVENT_STEP_END
    have been added - analogous to already existing events
    TARGET_EVENT_RESUME_*.
    
    This is an example of a concrete use case where having
    these events is important:
    
    In RISC-V processors without Debug Program Buffer, OpenOCD
    cannot execute fence/fence.i when resuming or single-
    stepping. With these events implemented, the user can
    instead provide custom operations to achieve that same
    effect prior to resuming the processor.
    
    Change-Id: I786348ff08940759d99b0f24e9e0ed5a44581094
    Signed-off-by: Jan Matyas <ma...@co...>
    Reviewed-on: http://openocd.zylin.com/5551
    Tested-by: jenkins
    Reviewed-by: Tim Newsome <ti...@si...>

diff --git a/doc/openocd.texi b/doc/openocd.texi
index 250db326f..0c58a682c 100644
--- a/doc/openocd.texi
+++ b/doc/openocd.texi
@@ -4924,6 +4924,10 @@ when reset disables PLLs needed to use a fast clock.
 @* After all targets have resumed
 @item @b{resumed}
 @* Target has resumed
+@item @b{step-start}
+@* Before a target is single-stepped
+@item @b{step-end}
+@* After single-step has completed
 @item @b{trace-config}
 @* After target hardware trace configuration was changed
 @end itemize
diff --git a/src/target/target.c b/src/target/target.c
index 50dd1488f..24fa416f8 100644
--- a/src/target/target.c
+++ b/src/target/target.c
@@ -205,6 +205,8 @@ static const Jim_Nvp nvp_target_event[] = {
 	{ .value = TARGET_EVENT_RESUMED, .name = "resumed" },
 	{ .value = TARGET_EVENT_RESUME_START, .name = "resume-start" },
 	{ .value = TARGET_EVENT_RESUME_END, .name = "resume-end" },
+	{ .value = TARGET_EVENT_STEP_START, .name = "step-start" },
+	{ .value = TARGET_EVENT_STEP_END, .name = "step-end" },
 
 	{ .name = "gdb-start", .value = TARGET_EVENT_GDB_START },
 	{ .name = "gdb-end", .value = TARGET_EVENT_GDB_END },
@@ -1256,7 +1258,17 @@ bool target_supports_gdb_connection(struct target *target)
 int target_step(struct target *target,
 		int current, target_addr_t address, int handle_breakpoints)
 {
-	return target->type->step(target, current, address, handle_breakpoints);
+	int retval;
+
+	target_call_event_callbacks(target, TARGET_EVENT_STEP_START);
+
+	retval = target->type->step(target, current, address, handle_breakpoints);
+	if (retval != ERROR_OK)
+		return retval;
+
+	target_call_event_callbacks(target, TARGET_EVENT_STEP_END);
+
+	return retval;
 }
 
 int target_get_gdb_fileio_info(struct target *target, struct gdb_fileio_info *fileio_info)
@@ -3145,7 +3157,7 @@ COMMAND_HANDLER(handle_step_command)
 
 	struct target *target = get_current_target(CMD_CTX);
 
-	return target->type->step(target, current_pc, addr, 1);
+	return target_step(target, current_pc, addr, 1);
 }
 
 void target_handle_md_output(struct command_invocation *cmd,
diff --git a/src/target/target.h b/src/target/target.h
index b954ec22d..ddeb00b57 100644
--- a/src/target/target.h
+++ b/src/target/target.h
@@ -258,6 +258,8 @@ enum target_event {
 	TARGET_EVENT_RESUMED,		/* target resumed to normal execution */
 	TARGET_EVENT_RESUME_START,
 	TARGET_EVENT_RESUME_END,
+	TARGET_EVENT_STEP_START,
+	TARGET_EVENT_STEP_END,
 
 	TARGET_EVENT_GDB_START, /* debugger started execution (step/run) */
 	TARGET_EVENT_GDB_END, /* debugger stopped execution (step/run) */

-----------------------------------------------------------------------

Summary of changes:
 doc/openocd.texi    |  4 ++++
 src/target/target.c | 16 ++++++++++++++--
 src/target/target.h |  2 ++
 3 files changed, 20 insertions(+), 2 deletions(-)


hooks/post-receive
-- 
Main OpenOCD repository