interesting
/
riscv-openocd
mirror of https://github.com/riscv/riscv-openocd.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Merge pull request #1165 from aap-sc/aap-sc/resume_debug_errors 

target/riscv: detailed error messages for cases when resume operation fails
This commit is contained in:
Evgeniy Naydanov 2024-11-18 13:17:25 +03:00 committed by GitHub
parent 463d1b0866 faffae0493
commit f51900b4a2
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 18 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
src/target/riscv/riscv-013.c
View File

@ -5392,7 +5392,9 @@ static int riscv013_step_or_resume_current_hart(struct target *target,
LOG_TARGET_ERROR(target, "Hart is not halted!");
return ERROR_FAIL;
}
LOG_TARGET_DEBUG(target, "resuming (for step?=%d)", step);
LOG_TARGET_DEBUG(target, "resuming (operation=%s)",
step ? "single-step" : "resume");
if (riscv_reg_flush_all(target) != ERROR_OK)
return ERROR_FAIL;
@ -5425,16 +5427,26 @@ static int riscv013_step_or_resume_current_hart(struct target *target,
return ERROR_OK;
}
dm_write(target, DM_DMCONTROL, dmcontrol);
LOG_TARGET_ERROR(target, "Failed to %s. dmstatus=0x%08x",
step ? "single-step" : "resume", dmstatus);
dm_write(target, DM_DMCONTROL, dmcontrol);
LOG_TARGET_ERROR(target,
" cancelling the resume request (dmcontrol.resumereq <- 0)");
LOG_TARGET_ERROR(target, "unable to resume");
if (dmstatus_read(target, &dmstatus, true) != ERROR_OK)
return ERROR_FAIL;
LOG_TARGET_ERROR(target, " dmstatus=0x%08x", dmstatus);
LOG_TARGET_ERROR(target, " dmstatus after cancellation=0x%08x", dmstatus);
if (step) {
LOG_TARGET_ERROR(target, " was stepping, halting");
riscv_halt(target);
LOG_TARGET_ERROR(target,
" trying to recover from a failed single-step, by requesting halt");
if (riscv_halt(target) == ERROR_OK)
LOG_TARGET_ERROR(target, " halt completed after failed single-step");
else
LOG_TARGET_ERROR(target, " could not halt, something is wrong with the taget");
// TODO: returning ERROR_OK is questionable, this code needs to be revised
return ERROR_OK;
}