-
Notifications
You must be signed in to change notification settings - Fork 653
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
Add support for cosimulation with Spike #1323
Conversation
|
Verilator takes issue with a parameter named "interrupt" in DPI calls
There's some bug with spike's |
I agree with your concerns, I’ll think of a way around this.
I do want to avoid having to serialize all the stuff into a verilog or c
header. I think there’s a way.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Pending CI
Thanks! |
Hi @jerryz123 @abejgonzalez @kabylkas, sorry to open an already closed thread. This is the latest dromajo discussion I could find. I tried dromajo with chipyard 1.6.2, 1.7.0 and 1.7.1. I see same issue, simulation fails with exit code 8191.
There were few old threads on dromajo fixes: #917 and #1204. I applied the fixes suggested there, but still i see same issue. Also, one other thing i noticed is chipyard stopped to track dromajo after
Here is the log:
/root/chipyard-olddromajo/sims/vcs/output/chipyard.TestHarness.DromajoBoomConfig/hello.out
can you please suggest any work around for this? or an older chipyard release that tests dromajo? Thanks. |
Hi All, |
Since spike is actively maintained and kept up-to-date with the spec, it makes more sense to use spike instead of dromajo for cosimulation.
This PR changes the chipyard-boom test to run MediumBoom+cospike (instead of SmallBoom).
This PR also removes testing esp-tools in CI. We cannot continue to guarantee that simulators will compile when using esp-isa-sim.
UPDATE: We're going to refactor the gemmini-spike fork into a dynamically loadable standalone extension that works with upstream spike. Soon we can remove the gemmini->esp-tools dependency.
Related PRs / Issues:
Type of change:
Impact:
Contributor Checklist:
main
as the base branch?changelog:<topic>
label?changelog:
label?.conda-lock.yml
file if you updated the conda requirements file?Please Backport
?