Skip to content

Hook up the error signals in the L0 cache#30

Merged
colluca merged 1 commit intopulp-platform:mainfrom
mosaic-soc:pr/err-signals
Jul 28, 2025
Merged

Hook up the error signals in the L0 cache#30
colluca merged 1 commit intopulp-platform:mainfrom
mosaic-soc:pr/err-signals

Conversation

@LucaRufer
Copy link
Contributor

The current implementation of the L0 has an in_error_o signal, which is just tied to zero. The error signal provided by the refill/L1 is ignored.
In this PR, the error signal is hooked up in the L0 the same way it is hooked up in the L1.

Changes:

  • The icache handler forwards error signals on cache line hits from the L1 to L0.
  • The L0 gets an error bit per cache line indicating if an error occured while fetching the cache line (same as in L1).
  • On an L0 hit, the error signal is provided together with the data.

Copy link
Collaborator

@colluca colluca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution @LucaRufer!

Tomorrow I will test it in Snitch before merging, but otherwise looks good to me.

@colluca colluca merged commit bc6582b into pulp-platform:main Jul 28, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants