-
Notifications
You must be signed in to change notification settings - Fork 578
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
Cosim mismatch in some tests with PMP enabled using QuestaSim #2124
Comments
Hmm, this might be a problem with Questa support. At lowRISC, we don't currently have Questa access so it's rather hard for us to help. Can you reproduce the problem with another simulator (Xcelium / VCS / Verilator) as well? If not, I think you might have to do a bit more digging to figure out what's coming unstuck. |
Unfortunately, I can't run using another simulator at the time. vendor/google_riscv-dv/src/riscv_asm_program_gen.sv if (riscv_instr_pkg::support_pmp) begin
str = {indent, "j main"};
instr_stream.push_back(str);
end If I comment it out the test passes. |
Another follow-up is that the traces between ISS and RTL are erroring out at only the last instruction where on ISS it is giving the instruction "wfi" while on the RTL side, it is not giving anything the last instruction is the one before it.
|
The code you've pinpointed is part of Your note about the traces makes sense. I suspect that this means the "there was a WFI instruction" message isn't getting to the DV code properly. @GregAC knows lots more about this area. Greg: any ideas? |
All tests mentioned don't generate the "WFI" instructions in the rtl_trace. |
Observed Behavior
I am getting the following error when running the tests using the default configuration (OpenTitan).
Expected Behavior
It was expected for these tests to pass with the default configuration as of here Ibex Regression Results
When I set the PMPEnable to 0 in ibex configuration file it passes gracefully.
Steps to reproduce the issue
The test passes when PMPEnable is set to 0 in ibex_configs.yaml
My Environment
EDA tool and version: Questasim 2023.1
Operating system: Linux-64 CentOS 7.9
Version of the Ibex source code: df88055
The text was updated successfully, but these errors were encountered: