Merged
Conversation
Member
Author
|
Once lowRISC/opentitan#27408 is merged, I will rebase the https://github.com/lowRISC/opentitan/tree/ot-sca_cw310 branch, create a tag, and build from there & link the tag in the commit message here in this PR. |
nasahlpa
force-pushed
the
update_pentest_binaries
branch
from
62e240e to
01ec33f
Compare
nasahlpa
force-pushed
the
update_pentest_binaries
branch
from
01ec33f to
2abfadd
Compare
nasahlpa
force-pushed
the
update_pentest_binaries
branch
3 times, most recently
from
0c2af73 to
ef14b81
Compare
Member
Author
|
The AES CI job will pass again after merging #397. For context, the failing AES SCA job uses the generic bitstream & pentest binary instead of the specific AES bitstream and binary. |
vogelpi
approved these changes
Jul 21, 2025
Collaborator
vogelpi
left a comment
vogelpi
left a comment
There was a problem hiding this comment.
LGTM with one question.
This commit updates the cw310 pentest binaries to the latest commit (lowRISC/opentitan@c0fece9) in the ot-sca_cw310 branch. Build commands used: ``` ./bazelisk.sh build //sw/device/tests/penetrationtests/\ firmware:pen_test_fi_otbn_fpga_cw310_test_rom ./bazelisk.sh build //sw/device/tests/penetrationtests/\ firmware:pen_test_fi_fpga_cw310_test_rom ./bazelisk.sh build //sw/device/tests/penetrationtests/\ firmware:pen_test_fi_ibex_fpga_cw310_test_rom ``` Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org>
Update the CW310 bitstream used for the FI experiments to commit lowRISC/opentitan@c0fece9 of the ot-sca_cw310 branch of OpenTitan. The following command was used to generate the bitstream: ``` ./bazelisk.sh build //hw/bitstream/vivado:fpga_cw310_test_rom ``` Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org>
This commit updates the Init function that is used by all FI commands. Now, the SRAM readback feature as well as the jittery clock can be configured. Moreover, in the init function, the device configuration now is received and stored into the database. Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org>
In the latest pentest framework, the IbexFi tests return a register file dump back to ot-sca. As the RF dump could be different for each compilation, filter it. Nevertheless, the RF dump is still stored into the database for further evaluation. Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org>
As the FI pentest framework changed quite substantially, this commit updates the golden responses of the tests. Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org>
This commit excludes the "data" field from the FI evaluation as it contains random data. The field is stored in the database for further evaluation. Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org>
Streamline the icache and dummy instructions enable/disable configuration. Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org>
As in the OpenTitan repository the compilation target names for the pentest binaries changes, this commit adapts them accordingly. Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org>
This commit renames jittery_clock_enable to enable_jittery_clock to streamline the naming with the other config parameters. Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org>
nasahlpa
force-pushed
the
update_pentest_binaries
branch
from
ef14b81 to
e1290f2
Compare
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
2 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.
This PR updates the pentest binaries to the latest binaries from lowRISC/opentitan#27408.