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

[entropy_src,verilator] Tune AST model of noise for Verilator simulation #24585

Open
AlexJones0 opened this issue Sep 17, 2024 · 0 comments
Open
Labels
Component:DV DV issue: testbench, test case, etc. IP:ast

Comments

@AlexJones0
Copy link
Contributor

Description

As discovered in #24184, the sw/device/tests:entropy_src_fw_observe_many_contiguous_test_sim_verilator and sw/device/tests:entropy_src_fw_override_test_sim_verilator Verilator test targets are failing if given enough timeout. Even though these entropy tests are primarily defined for FPGA, it would be preferable for these tests to still function in the Verilator simulation environment for debugging purposes.

Currently, to allow these tests to pass we must largely reduce the amount of entropy observed, and conditionally provide a much longer timeout to Verilator (see #24528). Verilator is running at a clock frequency of 500 kHz whereas FPGA runs at 24 Mhz, meaning that a 48x increase in timeout is needed to observe similar amounts of entropy. That is, the large increase in timeout is because the rate of the raw noise source (inside the AST) was never tuned for the Verilator simulation.

It would be better if the AST model was tuned for the Verilator simulation so that these tests can pass and be used for debugging purposes without requiring as many device-specific changes to the tests themselves.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component:DV DV issue: testbench, test case, etc. IP:ast
Projects
None yet
Development

No branches or pull requests

2 participants