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

Fix #35 (verilator getting stuck) and add ci test #37

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

andreaskuster
Copy link

@andreaskuster andreaskuster commented Mar 4, 2022

This PR should fix #35. In addition, we should add a test case to the CI to ensure that verilator support persists. Right now, I set the environment variable SIM=verilator, however, we should test icarus and verilator before we merge this.

@andreaskuster andreaskuster marked this pull request as ready for review March 4, 2022 10:11
@alexforencich
Copy link
Owner

Thanks for the PR, but I'm not interested in making any changes here to support verilator until verilator/verilator#2778 is fixed and cocotb works with the latest release version of verilator.

@andreaskuster
Copy link
Author

Fair enough. Let´s keep this open, since I still think there is the buggy verilator issue you mentioned, however there is a good chance that something broke in cocotb-bus > 0.1.1

@benreynwar
Copy link

Just hit this same issue trying to run test_axi_interconnect.py using a new version of verilator (with the above mentioned bug fixed)
Verilator 5.007 devel rev v5.006-7-g4a8cfe367

Switching to using cocotb-bus==0.1.1 fixed the issue for me.

@alexforencich
Copy link
Owner

Upstream verilator issue is verilator/verilator#3919

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.

Simulation gets stuck with SIM=verilator 4.106
3 participants