Skip to content
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

Halt reason after singlestep - CI test #3

Draft
wants to merge 15 commits into
base: riscv
Choose a base branch
from

Conversation

JanMatCodasip
Copy link
Owner

Dummy pull request, created just to test this change: riscv-collab#618

Do not merge, this is to be closed after the testing.

Jan Matyas and others added 15 commits June 8, 2021 09:51
Adding an automation that runs riscv-tests/debug on the build
of riscv-openocd.

The steps performed are:
- Build of OpenOCD
- Build of Spike
- Download a pre-built RISC-V toolchain
- Run riscv-tests/debug
- Collect test results
- Collect OpenOCD code coverage

Change-Id: Id10cfa26828c22d9759399bee6c2f24def6e4318
Signed-off-by: Jan Matyas <[email protected]>
fix
Change-Id: I3f818a2f438856231f18da2b535eee1e23bba03c
Signed-off-by: Jan Matyas <[email protected]>
Change-Id: Ib52fc36ed44af1cd259f6698be90acb7e0c9fbba
Signed-off-by: Jan Matyas <[email protected]>
Change-Id: If85f59f88001780b615e5f33af501478bd3ad838
Signed-off-by: Jan Matyas <[email protected]>
Change-Id: Id3417e4b8b9a4dd4063e387d3c5ca025bafff9d1
Signed-off-by: Jan Matyas <[email protected]>
Change-Id: I5939ed01ded16879dae441a2b2af80b769cf44e1
Signed-off-by: Jan Matyas <[email protected]>
Change-Id: Idc997c4ebbf83aebf4d767343c59e19abc4fb761
Signed-off-by: Jan Matyas <[email protected]>
Change-Id: I2030d95d4d7fad5759ba8afb65171e7cecbe7a6b
Signed-off-by: Jan Matyas <[email protected]>
Change-Id: I7fa2af93e7595244e66f4ae1bfcc4b9dbe718fb6
Signed-off-by: Jan Matyas <[email protected]>
Change-Id: I10c692c9f86eb185feb17e2744e98dc3dffc0049
Signed-off-by: Jan Matyas <[email protected]>
Change-Id: Ibe5e0f08514a640b84691f9521883533bdde9159
Signed-off-by: Jan Matyas <[email protected]>
Change-Id: I0005603ad60d8e104f7bec4fe16d2a9ae1342df3
Change-Id: Ic285cd0147fb9285adb5edb20f27b0a3e06ff82a
Signed-off-by: Jan Matyas <[email protected]>
When it doesn't do anything (most of the time) it has negligible
performance impact. With slower remote bitbang, and multiple spike
instances being tested in a single chain, things are sufficiently slow
that if a computer is busy then this is required to pass
riscv-tests/debug.

Change-Id: I8816efedaa0cc3b25734ba8fdc979ee4502284a1
Signed-off-by: Tim Newsome <[email protected]>
After single step operation, we should not assume that the halt
reason is single-step. There can be a higher-priority halt cause,
e.g. a breakpoint.

The real halt reason should be obtained from the target (dcsr.cause).

Change-Id: I3f1649aa46b91ea456d642d31ec2c50788b96084
Signed-off-by: Jan Matyas <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants