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.

Sign up for GitHub

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

Jump to bottom

Integrity bit is not forwarded to the alignment buffer #473

Closed
silabs-krdosvik opened this issue Jun 28, 2023 · 1 comment
Closed

Integrity bit is not forwarded to the alignment buffer #473

silabs-krdosvik opened this issue Jun 28, 2023 · 1 comment
Labels
Component:RTL For issues in the RTL (e.g. for files in the rtl directory) Status:Resolved Issue has been resolved, but closure is pending on git merge and/or issuer confirmation Type:Bug For bugs in any content (RTL, Documentation, etc.)

Comments

@silabs-krdosvik
Copy link
Contributor

Template for Bug Issue

Forwarding of Integrity Bit

A clear and concise description of what the bug is:
OBI rew has integrity (see screenshot 1), but this is not registered in the alignment buffer (see screenshot 2)

What seems to be the problem:
The integrity bit is computed according to the core_tras_i.addr (see screenshot 3), but the core_tras_i.addr is connectred to the prefetcher_trans_addr (see screenshot 4), which dont correspond with the req address (see screenshot 5 and 6).

Component

Indicate the component in which you found a problem (and we will add the appropriate label):
Component:RTL: cv32e40s core

Steps to Reproduce

Please provide:

  1. git hash: 29d4417
  2. Command line: -
  3. Logfile and/or wave-dump info (screen shots can be useful)
    image
    image
    image
    image
    image
    image
@silabs-krdosvik silabs-krdosvik added the Type:Bug For bugs in any content (RTL, Documentation, etc.) label Jun 28, 2023
@Silabs-ArjanB Silabs-ArjanB added the Component:RTL For issues in the RTL (e.g. for files in the rtl directory) label Jun 28, 2023
silabs-oysteink added a commit to silabs-oysteink/cv32e40s that referenced this issue Jun 28, 2023
@silabs-oysteink
Fix for issue openhwgroup#473. In addition to the reported issue this…
7d8a926 
… bug could also cause false major alerts.

Integrity fifo in instruction obi used signals with wrong timing. Used trans_i.integrity instead of the actual signals used with req-timing.

Signed-off-by: Oystein Knauserud <Oystein.Knauserud@silabs.com>
@silabs-oysteink
Copy link
Contributor

Issue should be fixed with PR #474

Silabs-ArjanB added a commit that referenced this issue Jun 28, 2023
@Silabs-ArjanB
Merge pull request #474 from silabs-oysteink/silabs-oysteink_instr-in…
847fe0e 
…tegrity-fix

Fix for issue #473.
@Silabs-ArjanB Silabs-ArjanB added the Status:Resolved Issue has been resolved, but closure is pending on git merge and/or issuer confirmation label Jun 28, 2023
@silabs-oysteink silabs-oysteink mentioned this issue Sep 26, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Component:RTL For issues in the RTL (e.g. for files in the rtl directory) Status:Resolved Issue has been resolved, but closure is pending on git merge and/or issuer confirmation Type:Bug For bugs in any content (RTL, Documentation, etc.)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Silabs-ArjanB @silabs-oysteink @silabs-krdosvik