-
Notifications
You must be signed in to change notification settings - Fork 307
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
What version of the RISC-V Debug Specification is used? #999
Comments
The issue I am trying to deal with is the backwards-incompatible change to allow access to The hardware I am working with does not support this modification. |
You don't need to build it for any/each specific debug specification. The one build supports most or all of all relevant debug specs:
|
To add to what I posted above - that does not rule out the possibility of bugs where, for example, something that works in one version of the debug spec may not work in another. I already had one today :-) If you can explain in more detail what you understand the issue to be here it might help others to figure out if there's a possible explanation or fix. |
Also - it might be worth collecting a verbose ( |
I had a quick skim of the 0.13.2 debug spec and can't see anything that says that a trigger must be enabled for |
Thanks for the info! Yes I am seeing that now. Here is the debug log with the write of 0 to
|
@mateoconlechuga could you double check this issue? #870 Does this have relevance to your case? |
@aap-sc Yes that issue is relevant - the hardware I am using is 0.13 specification compliant and doesn't accept writes to |
Can you clarify where in the 0.13 spec it states that |
Regarding the supported version. @mateoconlechuga , @TommyMurphyTM1234 the current OpenOCD codebase does not make a distinction between 1.0 and 0.13. The intention is to support both. There may be indeed be inconsistencies between 1.0 and 0.13 - it's just that no one encountered such case before (at least there was no pending issue to indicate that).
|
Under section
When OpenOCD writes 0 to the The 1.0 specification clarifies the intended behavior a lot better in the updated specification section
|
Yes - but I still don't see how that mandates that accesses/writes to |
I don't think I read your comment here correctly the first time:
The behavior I am experiencing is the other way around - Section 5.7 of the 1.0 specification clarifies this a lot more. This clarification about writing a 0 to disable the register and allow writes to tdata2 is nonexistent in the 0.13 specification, which the hardware was built against.
|
@mateoconlechuga a note regarding the spec interpretation. Such questions are better to be raised here (if there is no agreement): https://github.com/riscv/riscv-debug-spec . I believe RISCV debug workgroup still provides support for both versions of the spec. |
@aap-sc Thanks! I don't believe that this is a specification interpretation issue though. This is the backwards-incompatible specification change that is called out in the 1.0.0 specification that OpenOCD currently does not support:
|
Yes, the note you've mentioned looks convincing enough to me. So if you are going to submit a patch - please, consider creating a patch for spike too, so we could test this case. |
Hello, I am wondering if there is a way to know either through a tag, command line argument, or documentation which RISC-V debug specification is implemented? Currently it seems like 1.0, but I would like to target hardware which uses 0.13, and I am unsure where/how to get/compile a version of OpenOCD that works with different versions of the specification.
Thanks for the help.
The text was updated successfully, but these errors were encountered: