Skip to content

[icache/rtl] Tweak infection#2376

Merged
vogelpi merged 1 commit into
lowRISC:masterfrom
nasahlpa:icache_tweak_infection
Apr 9, 2026
Merged

[icache/rtl] Tweak infection#2376
vogelpi merged 1 commit into
lowRISC:masterfrom
nasahlpa:icache_tweak_infection

Conversation

@nasahlpa
Copy link
Copy Markdown
Contributor

@nasahlpa nasahlpa commented Apr 9, 2026

Before writing data and tag into the data and tag banks, XOR a tweak on top. When reading back the data and tag from the banks, un-XOR them using the tweak.

Before writing data and tag into the data and tag banks, XOR a tweak on
top. When reading back the data and tag from the banks, un-XOR them using
the tweak.

Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org>
@nasahlpa nasahlpa requested review from johannheyszl and vogelpi April 9, 2026 07:00
Copy link
Copy Markdown
Contributor

@vogelpi vogelpi left a comment

Choose a reason for hiding this comment

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

This PR is identical to the one that I've already reviewed in depth previously.

LGTM, thanks @nasahlpa !

@vogelpi vogelpi added this pull request to the merge queue Apr 9, 2026
Merged via the queue into lowRISC:master with commit 556dff1 Apr 9, 2026
12 checks passed
@nasahlpa nasahlpa deleted the icache_tweak_infection branch April 9, 2026 14:25
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