This repository was archived by the owner on May 27, 2026. It is now read-only.
[acc,rtl] Masked the AppIntf for KMAC connection#241
Closed
apinise wants to merge 12 commits into
Closed
Conversation
apinise
force-pushed
the
acc-masked-app
branch
17 times, most recently
from
b9096bd to
3767ea0
Compare
apinise
force-pushed
the
acc-masked-app
branch
2 times, most recently
from
ba9f64a to
7a47e10
Compare
Signed-off-by: Evan Apinis <eapinis@zerorisc.com>
Renamed existing digest WSR to digest0 and created a new digest1 register for the second share. Made a state machine to stall the eager KMAC refresh until both WSR have been read. Signed-off-by: Evan Apinis <eapinis@zerorisc.com>
Signed-off-by: Evan Apinis <eapinis@zerorisc.com>
Signed-off-by: Evan Apinis <eapinis@zerorisc.com>
Signed-off-by: Evan Apinis <eapinis@zerorisc.com>
Changed rig to generate a clean base KMAC transaction and extend it to inject errors. Added fatal and recoverable error flags for the AppIntf to the ISS and RTL. Created a new test sequence to verify transactions. Signed-off-by: Evan Apinis <eapinis@zerorisc.com>
Signed-off-by: Evan Apinis <eapinis@zerorisc.com>
Signed-off-by: Evan Apinis <eapinis@zerorisc.com>
Signed-off-by: Jade Philipoom <jadep@zerorisc.com>
Signed-off-by: Jade Philipoom <jadep@zerorisc.com>
Signed-off-by: Jade Philipoom <jadep@zerorisc.com>
Added masked message support to the verilator test and updated the expected final register values. A fixed mask has been used on the KMAC output to synchronize it with the ISS value. Signed-off-by: Evan Apinis <eapinis@zerorisc.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds masking support to the ACC side of the KMAC AppIntf, which was masked in #242.
kmac_cfgCSR to enable/disable masking for the next KMAC transaction.kmac_msgandkmac_digestWSR registers are now used for Share 0 or the unmasked value if masking is not enabled.kmac_msg1andkmac_digest1WSR with independent datapaths for Share 1.accsimISS andkmac.pyto implement masked KMAC transactions.app_req.pyRIG and created a newbad_app_req.pyto generate first-order masked transactions for random binary testing.bad_app_req.pygenerated snippets.