lowRISC / lowRISC/opentitan

[sram_ctrl,dv] Check d_user for sram_ctrl_tl_errors

Open Beginner friendly
#25,927 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Component:DV IP:sram_ctrl
Dominant language
SystemVerilog
Stars
3.6k
Forks
1.1k
Avg merge
2d 22h
Merged PRs (30d)
141

Description

Description

For sram_ctrl_tl_errors, we currently only check whether d_error is set when we inject an error. However, we should also check whether the content of the d_user field matches our expectation. This could be done in the get_sram_predict_tl_err function. Detected in #25925.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in hw/ip/sram_ctrl/dv/env/sram_ctrl_scoreboard.sv at get_sram_predict_tl_err, where sram_ctrl_tl_errors are checked after error injection. Inspect the existing d_error check and the expected d_user value, then run the relevant SRAM controller DV tests. Done means the check also verifies d_user matches its expectation.

Written by the indexing model from the issue text.

Assessment

Domain
testing
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
72/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.