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

[rom_ctrl,dv] Tweak randomisation of bad mubi4_t values #21335

Merged
merged 1 commit into from
Feb 14, 2024

Conversation

rswarbrick
Copy link
Contributor

The code in rom_ctrl_corrupt_sig_fatal_chk_vseq.sv was not quite right: it ignored the return value of mubi4_test_invalid, and never actually assigned anything to rand_var.

This change switches things to do what I think was originally intended.

(@msfschaffner: I'm adding you as a reviewer because you touched these lines recently. As far as I can tell, they were silly before you touched them! But I thought you might know some context that I'm missing)

@rswarbrick rswarbrick added Component:DV DV issue: testbench, test case, etc. IP:rom_ctrl labels Feb 13, 2024
@rswarbrick rswarbrick requested a review from a team as a code owner February 13, 2024 13:55
@rswarbrick rswarbrick requested review from matutem and removed request for a team February 13, 2024 13:55
@rswarbrick
Copy link
Contributor Author

Huh. My initial version of this code doesn't actually work with VCS. The problem is that VCS takes the strong-typed enum rules in the 1800 spec a bit more seriously than Xcelium (and, presumably, us).

I've just force-pushed with a version that uses logic [3:0] instead, and generally makes more sense.

The code in rom_ctrl_corrupt_sig_fatal_chk_vseq.sv was not quite
right: it ignored the return value of mubi4_test_invalid, and never
actually assigned anything to rand_var.

This change switches things to do what I think was originally
intended.

Signed-off-by: Rupert Swarbrick <[email protected]>
@msfschaffner
Copy link
Contributor

Huh, I think when I updated that I misread what this function was doing (absolutely nothing in this context :)).
Thanks for patching this!

@rswarbrick rswarbrick merged commit e4f6249 into lowRISC:master Feb 14, 2024
32 of 33 checks passed
@rswarbrick rswarbrick deleted the rom-ctrl-bad-mubi4 branch February 14, 2024 09:26
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:rom_ctrl
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants