-
-
Notifications
You must be signed in to change notification settings - Fork 303
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
Are there some bugs in clockDomain.waitSampling() API? #1382
Comments
To answer your question directly: maybe, but none that we know of ATM. One question that I'd have is: what is driving the clock in the CD? Is it driven from
|
This is weird, there hasn't been a waitSampling since quite long in the StreamMonitor, which version of SpinalHDL are you using ? Else, can you send a minimal self contained example for us to reproduce ? (verilator is the reference implementation of SpinalSim) |
SpinalHDL Version:1.10.1 |
I have two stream monitor in my simualtion enviroment,one is ok. |
I would say, would need to first clockDomain.waitSampling() in the loop, then check for the bus status But overall would need a small fully self contained example of weird behaviour to reproduce. |
So, maybe this's the same as my problem in issues 1334? |
This discussion has been last long, I am trying to find the difference between StreamMonitor and class you describe. waitSampling is different from the onSampling, it waits specified number of valid samplings. onSampling triggered each valid sampling. Another difference is that StreamMonitor is not to create a new process by fork. It uses the same process of simulation by default. |
Get Some Error Info: |
Again: please post a self contained example that reproduces the issue - if not we can't help you. |
I have a StreamMonitor Written by myself,the core function is running this function by fork:
In a design of my company, I need two StreamMonitors. When running the case, one of the cases failed. By adding printing, I found that the clockDomain.waitSampling() in the StreamMonitor of an interface missed a sample, which caused the simulation fail.
Since it is the company's code, the complete code cannot be posted here. I tried to test in VCS and Verilator. This case failed due to the same error reason, while other cases were successful.
The text was updated successfully, but these errors were encountered: